diff options
author | Chris Down <chris@chrisdown.name> | 2023-01-28 12:53:48 +0100 |
---|---|---|
committer | Hiltjo Posthuma <hiltjo@codemadness.org> | 2023-01-28 13:34:43 +0100 |
commit | 712d6639ff8e863560328131bbb92b248dc9cde7 (patch) | |
tree | c563872871c1e3e947d40caae45614498140e8dd /util.o%252525253fid%252525253da5ef81cd3600306c4b2af8a5dad69fab4d6cad17%2525253fid%2525253d271e1a5f3445e8ab70255fcf9b1da8c233efa1d2%25253fid%25253df81ac905a0940e0779fe5207e5eec4ac42956b98%253fid%253de81f17d4c196aaed6893fd4beed49991caa3e2a4%3fid%3d348f6559ab0d4793db196ffa56ba96ab95a594a6?id=712d6639ff8e863560328131bbb92b248dc9cde7 | |
parent | 89f9905714c1c1b2e8b09986dfbeca15b68d8af8 (diff) |
Use sigaction(SA_NOCLDWAIT) for SIGCHLD handling
signal() semantics are pretty unclearly specified. For example, depending on OS
kernel and libc, the handler may be returned to SIG_DFL (hence the inner call
to read the signal handler). Moving to sigaction() means the behaviour is
consistently defined.
Using SA_NOCLDWAIT also allows us to avoid calling the non-reentrant function
die() in the handler.
Some addditional notes for archival purposes:
* NRK pointed out errno of waitpid could also theoretically get clobbered.
* The original patch was iterated on and modified by NRK and Hiltjo:
* SIG_DFL was changed to SIG_IGN, this is required, atleast on older systems
such as tested on Slackware 11.
* signals are not blocked using sigprocmask, because in theory it would
briefly for example also ignore a SIGTERM signal. It is OK if waitpid() is (in
theory interrupted).
POSIX reference:
"Consequences of Process Termination":
https://pubs.opengroup.org/onlinepubs/9699919799/functions/_Exit.html#tag_16_01_03_01
Diffstat (limited to 'util.o%252525253fid%252525253da5ef81cd3600306c4b2af8a5dad69fab4d6cad17%2525253fid%2525253d271e1a5f3445e8ab70255fcf9b1da8c233efa1d2%25253fid%25253df81ac905a0940e0779fe5207e5eec4ac42956b98%253fid%253de81f17d4c196aaed6893fd4beed49991caa3e2a4%3fid%3d348f6559ab0d4793db196ffa56ba96ab95a594a6?id=712d6639ff8e863560328131bbb92b248dc9cde7')
0 files changed, 0 insertions, 0 deletions