pauljako/yambar
1
0
Fork 0
forked from external/yambar
Code Pull requests Activity

module/script: restore signal handlers and mask in child process

Browse source 
This fixes the issue where `killpg()` didn’t manage to kill the
sub-process tree.
This commit is contained in:
Daniel Eklöf 2020-10-25 16:06:16 +01:00
parent c911d20e73
commit 8702378c74
No known key found for this signature in database
GPG key ID: 5BBD4992C116573F
1 changed files with 14 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
modules/script.c
View file

@ -377,6 +377,20 @@ run(struct module *mod)
if (pid == 0) { if (pid == 0) {
/* Child */ /* Child */
/* Restore signal handlers */
sigset_t mask;
sigemptyset(&mask);
const struct sigaction sa = {.sa_handler = SIG_DFL};
if (sigaction(SIGINT, &sa, NULL) < 0 ||
sigaction(SIGTERM, &sa, NULL) < 0 ||
sigaction(SIGCHLD, &sa, NULL) < 0 ||
sigprocmask(SIG_SETMASK, &mask, NULL) < 0)
{
goto fail;
}
setpgid(0, 0); setpgid(0, 0);
/* Close pipe read ends */ /* Close pipe read ends */