diff options
| author | Daniel Eklöf <daniel@ekloef.se> | 2021-06-02 19:52:10 +0200 |
|---|---|---|
| committer | Simon Ser <contact@emersion.fr> | 2021-06-05 11:50:11 +0200 |
| commit | 1dd6df6a5df3924d42a25bd17dab0ef8beecad61 (patch) | |
| tree | 085430957c6112e52bfda6ee07109757898308b8 | |
| parent | b997147284b9d5632f1f929f02a17936a77cd7ec (diff) | |
sway: commands: exec: restore SIGPIPE before exec:ing
Sway ignores SIGPIPE (by installing a SIG_IGN handler), in order to
“prevent IPC from crashing Sway”.
SIG_IGN handlers are the *only* signal handlers inherited in
sub-processes. As such, we should be a good citizen and restore
the SIGPIPE handler to its default handler.
Original bug report:
https://www.mail-archive.com/debian-bugs-dist@lists.debian.org/msg1806907.html
| -rw-r--r-- | sway/commands/exec_always.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/sway/commands/exec_always.c b/sway/commands/exec_always.c index e18e2c22..fce337d5 100644 --- a/sway/commands/exec_always.c +++ b/sway/commands/exec_always.c @@ -57,6 +57,7 @@ struct cmd_results *cmd_exec_process(int argc, char **argv) { sigset_t set; sigemptyset(&set); sigprocmask(SIG_SETMASK, &set, NULL); + signal(SIGPIPE, SIG_DFL); close(fd[0]); if ((child = fork()) == 0) { close(fd[1]); |