diff options
author | netblue30 <netblue30@yahoo.com> | 2016-11-27 18:10:50 -0500 |
---|---|---|
committer | netblue30 <netblue30@yahoo.com> | 2016-11-27 18:10:50 -0500 |
commit | 4ea68a4e03592d1c685f760f66eebe3018536416 (patch) | |
tree | 950649bfa28b6f56bb4991c46eed8bd2ca3eae40 /src/firejail/x11.c | |
parent | fixes (diff) | |
download | firejail-4ea68a4e03592d1c685f760f66eebe3018536416.tar.gz firejail-4ea68a4e03592d1c685f760f66eebe3018536416.tar.zst firejail-4ea68a4e03592d1c685f760f66eebe3018536416.zip |
cleanup
Diffstat (limited to 'src/firejail/x11.c')
-rw-r--r-- | src/firejail/x11.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/firejail/x11.c b/src/firejail/x11.c index 807f2d5f0..d9b3b23d1 100644 --- a/src/firejail/x11.c +++ b/src/firejail/x11.c | |||
@@ -252,7 +252,8 @@ void x11_start_xephyr(int argc, char **argv) { | |||
252 | } | 252 | } |
253 | 253 | ||
254 | for (i = 0; i < (int) strlen(xephyr_extra_params)-1; i++) { | 254 | for (i = 0; i < (int) strlen(xephyr_extra_params)-1; i++) { |
255 | if (pos >= (sizeof(server_argv)/sizeof(*server_argv))) { | 255 | //todo: if working , add a -2 also in 0.9.44-bugfix |
256 | if (pos >= (sizeof(server_argv)/sizeof(*server_argv)) - 2) { | ||
256 | fprintf(stderr, "Error: arg count limit exceeded while parsing xephyr_extra_params\n"); | 257 | fprintf(stderr, "Error: arg count limit exceeded while parsing xephyr_extra_params\n"); |
257 | exit(1); | 258 | exit(1); |
258 | } | 259 | } |
@@ -716,6 +717,7 @@ void x11_xorg(void) { | |||
716 | } | 717 | } |
717 | if (set_perms(RUN_XAUTHORITY_SEC_FILE, getuid(), getgid(), 0600)) | 718 | if (set_perms(RUN_XAUTHORITY_SEC_FILE, getuid(), getgid(), 0600)) |
718 | errExit("set_perms"); | 719 | errExit("set_perms"); |
720 | /* coverity[toctou] */ | ||
719 | unlink(tmpfname); | 721 | unlink(tmpfname); |
720 | 722 | ||
721 | // mount | 723 | // mount |