Merge pull request #2120 from martinetd/swayidle-zombies

Swayidle: doublefork to not leave zombies around
This commit is contained in:
emersion 2018-06-08 14:41:22 +01:00 committed by GitHub
commit 231c72a141
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 19 additions and 6 deletions

View File

@ -73,7 +73,7 @@ struct cmd_results *cmd_exec_always(int argc, char **argv) {
}
close(fd[0]);
// cleanup child process
wait(0);
waitpid(pid, NULL, 0);
if (*child > 0) {
wlr_log(L_DEBUG, "Child process created with pid %d", *child);
// TODO: add PID to active workspace

View File

@ -6,6 +6,7 @@
#include <stdlib.h>
#include <errno.h>
#include <string.h>
#include <sys/wait.h>
#include <unistd.h>
#include <wayland-client-protocol.h>
#include <wayland-client.h>
@ -59,13 +60,25 @@ static void cmd_exec(void *data) {
}
char *param = (char *)data;
wlr_log(L_DEBUG, "Cmd exec %s", param);
int pid = fork();
pid_t pid = fork();
if (pid == 0) {
char *const cmd[] = { "sh", "-c", param, NULL, };
execvp(cmd[0], cmd);
exit(1);
pid = fork();
if (pid == 0) {
char *const cmd[] = { "sh", "-c", param, NULL, };
execvp(cmd[0], cmd);
wlr_log_errno(L_ERROR, "execve failed!");
exit(1);
} else if (pid < 0) {
wlr_log_errno(L_ERROR, "fork failed");
exit(1);
}
exit(0);
} else if (pid < 0) {
wlr_log_errno(L_ERROR, "fork failed");
} else {
wlr_log(L_DEBUG, "Spawned process %s", param);
waitpid(pid, NULL, 0);
}
wlr_log(L_DEBUG, "Spawned process %d", pid);
}
#if defined(SWAY_IDLE_HAS_SYSTEMD) || defined(SWAY_IDLE_HAS_ELOGIND)