collectdmon: Do not block SIGCHLD.
authorSebastian Harl <sh@tokkee.org>
Thu, 2 Oct 2008 13:25:10 +0000 (15:25 +0200)
committerFlorian Forster <octo@huhu.verplant.org>
Fri, 3 Oct 2008 20:15:43 +0000 (22:15 +0200)
There is no reason to do so as we do not need to synchronize any waitpid()
calls. In fact, doing so was wrong because waitpid() should then return with
an error according to POSIX. This was still working so far since waitpid()
would not return until all children terminated and this is all we really care
about.

Signed-off-by: Sebastian Harl <sh@tokkee.org>
Signed-off-by: Florian Forster <octo@huhu.verplant.org>
src/collectdmon.c

index e496eb0..015344f 100644 (file)
@@ -331,12 +331,6 @@ int main (int argc, char **argv)
                return 1;
        }
 
-       sigaddset (&sa.sa_mask, SIGCHLD);
-       if (0 != sigprocmask (SIG_BLOCK, &sa.sa_mask, NULL)) {
-               syslog (LOG_ERR, "Error: sigprocmask() failed: %s", strerror (errno));
-               return 1;
-       }
-
        while (0 == loop) {
                int status = 0;