From 764c2aff8fa46c370c9c030dc733c73cdecfbce1 Mon Sep 17 00:00:00 2001 From: Pekka Paalanen Date: Fri, 8 Jul 2022 11:07:13 +0300 Subject: [PATCH] xwayland: do not check execve() return value Simplifies the code a bit. Signed-off-by: Pekka Paalanen --- compositor/xwayland.c | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/compositor/xwayland.c b/compositor/xwayland.c index fc3c4bae..84ac7747 100644 --- a/compositor/xwayland.c +++ b/compositor/xwayland.c @@ -285,11 +285,12 @@ spawn_xserver(void *user_data, const char *display, int abstract_fd, int unix_fd if (!ret) _exit(EXIT_FAILURE); - if (execve(xserver, (char *const *)argv, envp) < 0) { - if (exec_failure_msg) { - write(STDERR_FILENO, exec_failure_msg, - strlen(exec_failure_msg)); - } + execve(xserver, (char *const *)argv, envp); + /* execve does not return on success, so it failed */ + + if (exec_failure_msg) { + write(STDERR_FILENO, exec_failure_msg, + strlen(exec_failure_msg)); } _exit(EXIT_FAILURE);