diff options
author | John Levon <john.levon@nutanix.com> | 2021-12-22 10:17:11 +0000 |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-12-22 10:17:11 +0000 |
commit | 17769cf1af093dfb4b9bc3347ae39324029989ac (patch) | |
tree | 6e5da3b6b12082e843e979f6702cc18242571bb7 /samples | |
parent | 190dcbde2872b57600088606785b75259b69efce (diff) | |
download | libvfio-user-17769cf1af093dfb4b9bc3347ae39324029989ac.zip libvfio-user-17769cf1af093dfb4b9bc3347ae39324029989ac.tar.gz libvfio-user-17769cf1af093dfb4b9bc3347ae39324029989ac.tar.bz2 |
fix coverity issues (#636)
Fix three remaining low priority coverity issues; they do not represent bugs.
Signed-off-by: John Levon <john.levon@nutanix.com>
Reviewed-by: Swapnil Ingle <swapnil.ingle@nutanix.com>
Diffstat (limited to 'samples')
-rw-r--r-- | samples/client.c | 2 | ||||
-rw-r--r-- | samples/server.c | 3 |
2 files changed, 5 insertions, 0 deletions
diff --git a/samples/client.c b/samples/client.c index 6cd065d..9f90401 100644 --- a/samples/client.c +++ b/samples/client.c @@ -1162,6 +1162,8 @@ int main(int argc, char *argv[]) */ nr_dma_regions = server_max_fds << 1; + umask(0022); + if ((tmpfd = mkstemp(template)) == -1) { err(EXIT_FAILURE, "failed to create backing file"); } diff --git a/samples/server.c b/samples/server.c index a43dc3c..25b4908 100644 --- a/samples/server.c +++ b/samples/server.c @@ -41,6 +41,7 @@ #include <openssl/md5.h> #include <sys/mman.h> #include <sys/param.h> +#include <sys/stat.h> #include <sys/time.h> #include "common.h" @@ -483,6 +484,8 @@ int main(int argc, char *argv[]) err(EXIT_FAILURE, "failed to setup BAR0 region"); } + umask(0022); + /* * Setup BAR1 to be 3 pages in size where only the first and the last pages * are mappable. The client can still mmap the 2nd page, we can't prohibit |