diff --git a/client/pool-buffer.c b/client/pool-buffer.c index 836c6b136..fd500c492 100644 --- a/client/pool-buffer.c +++ b/client/pool-buffer.c @@ -11,19 +11,7 @@ #include #include "config.h" #include "pool-buffer.h" - -static bool set_cloexec(int fd) { - long flags = fcntl(fd, F_GETFD); - if (flags == -1) { - return false; - } - - if (fcntl(fd, F_SETFD, flags | FD_CLOEXEC) == -1) { - return false; - } - - return true; -} +#include "util.h" static int create_pool_file(size_t size, char **name) { static const char template[] = "sway-client-XXXXXX"; @@ -46,7 +34,7 @@ static int create_pool_file(size_t size, char **name) { return -1; } - if (!set_cloexec(fd)) { + if (!sway_set_cloexec(fd, true)) { close(fd); return -1; } diff --git a/common/util.c b/common/util.c index 3a807edba..c0324b2fd 100644 --- a/common/util.c +++ b/common/util.c @@ -77,7 +77,7 @@ const char *sway_wl_output_subpixel_to_string(enum wl_output_subpixel subpixel) return NULL; } -bool set_cloexec(int fd, bool cloexec) { +bool sway_set_cloexec(int fd, bool cloexec) { int flags = fcntl(fd, F_GETFD); if (flags == -1) { sway_log_errno(SWAY_ERROR, "fcntl failed"); diff --git a/include/util.h b/include/util.h index 6d9454e05..3cba49f0d 100644 --- a/include/util.h +++ b/include/util.h @@ -32,6 +32,6 @@ float parse_float(const char *value); const char *sway_wl_output_subpixel_to_string(enum wl_output_subpixel subpixel); -bool set_cloexec(int fd, bool cloexec); +bool sway_set_cloexec(int fd, bool cloexec); #endif diff --git a/sway/config/bar.c b/sway/config/bar.c index f90fcdc07..1c7c13b2e 100644 --- a/sway/config/bar.c +++ b/sway/config/bar.c @@ -194,7 +194,7 @@ static void invoke_swaybar(struct bar_config *bar) { sway_log_errno(SWAY_ERROR, "socketpair failed"); return; } - if (!set_cloexec(sockets[0], true) || !set_cloexec(sockets[1], true)) { + if (!sway_set_cloexec(sockets[0], true) || !sway_set_cloexec(sockets[1], true)) { return; } @@ -222,7 +222,7 @@ static void invoke_swaybar(struct bar_config *bar) { sway_log_errno(SWAY_ERROR, "fork failed"); _exit(EXIT_FAILURE); } else if (pid == 0) { - if (!set_cloexec(sockets[1], false)) { + if (!sway_set_cloexec(sockets[1], false)) { _exit(EXIT_FAILURE); } diff --git a/sway/config/output.c b/sway/config/output.c index 50bf1155f..42deec674 100644 --- a/sway/config/output.c +++ b/sway/config/output.c @@ -505,7 +505,7 @@ static bool _spawn_swaybg(char **command) { sway_log_errno(SWAY_ERROR, "socketpair failed"); return false; } - if (!set_cloexec(sockets[0], true) || !set_cloexec(sockets[1], true)) { + if (!sway_set_cloexec(sockets[0], true) || !sway_set_cloexec(sockets[1], true)) { return false; } @@ -529,7 +529,7 @@ static bool _spawn_swaybg(char **command) { sway_log_errno(SWAY_ERROR, "fork failed"); _exit(EXIT_FAILURE); } else if (pid == 0) { - if (!set_cloexec(sockets[1], false)) { + if (!sway_set_cloexec(sockets[1], false)) { _exit(EXIT_FAILURE); } diff --git a/sway/swaynag.c b/sway/swaynag.c index 0fca6c710..db5a919a1 100644 --- a/sway/swaynag.c +++ b/sway/swaynag.c @@ -36,7 +36,7 @@ bool swaynag_spawn(const char *swaynag_command, sway_log(SWAY_ERROR, "Failed to create pipe for swaynag"); return false; } - if (!set_cloexec(swaynag->fd[1], true)) { + if (!sway_set_cloexec(swaynag->fd[1], true)) { goto failed; } } @@ -46,7 +46,7 @@ bool swaynag_spawn(const char *swaynag_command, sway_log_errno(SWAY_ERROR, "socketpair failed"); goto failed; } - if (!set_cloexec(sockets[0], true) || !set_cloexec(sockets[1], true)) { + if (!sway_set_cloexec(sockets[0], true) || !sway_set_cloexec(sockets[1], true)) { goto failed; } @@ -69,7 +69,7 @@ bool swaynag_spawn(const char *swaynag_command, sway_log_errno(SWAY_ERROR, "fork failed"); _exit(EXIT_FAILURE); } else if (pid == 0) { - if (!set_cloexec(sockets[1], false)) { + if (!sway_set_cloexec(sockets[1], false)) { _exit(EXIT_FAILURE); }