Correctly detect SOCK_CLOEXEC

SOCK_CLOEXEC expands to an enum, so it cannot be used in preprocessor conditionals.
QubesOS · Apr 26, 2023 · b7851c8 · b7851c8
1 parent af828db
commit b7851c8
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/libqrexec/exec.c b/libqrexec/exec.c
@@ -127,10 +127,10 @@ static int do_fork_exec(const char *user,
                 fix_fds(inpipe[0], outpipe[1], 2);
 
             close(statuspipe[0]);
-#if !SOCK_CLOEXEC
-            status = fcntl(statuspipe[1], F_GETFD);
-            fcntl(statuspipe[1], F_SETFD, status | FD_CLOEXEC);
-#endif
+            if (SOCK_CLOEXEC == (0)) {
+                status = fcntl(statuspipe[1], F_GETFD);
+                fcntl(statuspipe[1], F_SETFD, status | FD_CLOEXEC);
+            }
             if (exec_func != NULL)
                 exec_func(cmdline, user);
             else