mirrors/foot
1
0
Fork 0
mirror of https://codeberg.org/dnkl/foot.git synced 2026-03-15 05:33:58 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

slave/spawn: restore SIGHUP handler after fork(), before exec()

Browse source 
Foot installs a SIG_IGN handler for SIGHUP. Ignored signals are
inherited in sub-processes. Thus, we need to restore it to SIG_DFL
before exec:ing.
This commit is contained in:
Daniel Eklöf 2021-06-02 17:48:57 +02:00
parent 2f7e5aed40
commit a72b2688bd
No known key found for this signature in database
GPG key ID: 5BBD4992C116573F
3 changed files with 9 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

6
slave.c
View file

@ -274,10 +274,12 @@ slave_spawn(int ptmx, int argc, const char *cwd, char *const *argv,
_exit(errno_copy);
}
/* Restore signal mask */
/* Restore signal mask, and SIG_IGN'd signals */
sigset_t mask;
sigemptyset(&mask);
if (sigprocmask(SIG_SETMASK, &mask, NULL) < 0) {
if (sigprocmask(SIG_SETMASK, &mask, NULL) < 0 ||
sigaction(SIGHUP, &(struct sigaction){.sa_handler = SIG_DFL}, NULL) < 0)
{
const int errno_copy = errno;
LOG_ERRNO_P(errno, "failed to restore signals");
(void)!write(fork_pipe[1], &errno_copy, sizeof(errno_copy));