[systemd-devel] [PATCH] core: fix detection of dead process v2.

Lennart Poettering lennart at poettering.net
Sat Feb 15 09:18:40 PST 2014

On Sat, 15.02.14 02:38, Yuxuan Shui (yshuiv7 at gmail.com) wrote:

> Commit 5ba6985b moves the UNIT_VTABLE(u)->sigchld_event before systemd
> actually reap the zombie. Which leads to service_load_pid_file accept
> zombie as a valid pid.

service_load_pid_file() actually checks the validity of all PIDs before
making use of them, with kill(pid, 0). Zombies actually result in ESRCH
when one tries to kill them, and we will not accept the PID.

Is this a real problem you encountered? Are you suggesting the kill()
check doesn't work? If so, we should probably fix that rather than
adding some new code...


Lennart Poettering, Red Hat

More information about the systemd-devel mailing list