chiark / gitweb /
Revert "tree-wide: use pid_is_valid() at more places"
authorAlan Jenkins <alan.christopher.jenkins@gmail.com>
Tue, 3 Oct 2017 11:05:24 +0000 (12:05 +0100)
committerSven Eden <yamakuzure@gmx.net>
Tue, 3 Oct 2017 11:05:24 +0000 (12:05 +0100)
commit7d729de3a70bc7a7fef69f5cea03fcbf835e118a
treef773ae98d3f151bc290269d21a9e25573f58e1b6
parent50f665df3e070a0d3bdc7d9740986729f4bebc0b
Revert "tree-wide: use pid_is_valid() at more places"

This reverts commit ee043777be58251e7441b4f04594e9e3792d7fb2.

It broke almost everywhere it touched.  The places that
handn't been converted, were mostly followed by special
handling for the invalid PID `0`.  That explains why they
tested for `pid < 0` instead of `pid <= 0`.

I think that one was the first commit I reviewed, heh.
src/basic/process-util.c
src/login/logind-dbus.c