chiark / gitweb /
tree-wide: introduce new safe_fork() helper and port everything over
authorLennart Poettering <lennart@poettering.net>
Fri, 22 Dec 2017 12:08:14 +0000 (13:08 +0100)
committerSven Eden <yamakuzure@gmx.net>
Wed, 30 May 2018 05:49:44 +0000 (07:49 +0200)
commit45aef547896d1e91b2b2a121566320b5c3a17948
tree4e4684c715e966e9fc6edfdc89def6c31124663d
parent7cfc65cbfb5fb2e06710cab0d5008a0643735d49
tree-wide: introduce new safe_fork() helper and port everything over

This adds a new safe_fork() wrapper around fork() and makes use of it
everywhere. The new wrapper does a couple of things we previously did
manually and separately in a safer, more correct and automatic way:

1. Optionally resets signal handlers/mask in the child

2. Sets a name on all processes we fork off right after forking off (and
   the patch assigns useful names for all processes we fork off now,
   following a systematic naming scheme: always enclosed in () – in order
   to indicate that these are not proper, exec()ed processes, but only
   forked off children, and if the process is long-running with only our
   own code, without execve()'ing something else, it gets am "sd-" prefix.)

3. Optionally closes all file descriptors in the child

4. Optionally sets a PR_SET_DEATHSIG to SIGTERM in the child, in a safe
   way so that the parent dying before this happens being handled
   safely.

5. Optionally reopens the logs

6. Optionally connects stdin/stdout/stderr to /dev/null

7. Debug logs about the forked off processes.
src/basic/exec-util.c
src/basic/process-util.c
src/basic/process-util.h
src/basic/terminal-util.c
src/basic/time-util.c
src/basic/util.c
src/libelogind/sd-bus/bus-container.c
src/libelogind/sd-bus/bus-socket.c
src/login/inhibit.c
src/shared/pager.c
src/test/test-process-util.c