Moved the signal handler initialization so it gets run in the child, not the parent..

diff --git a/src/collectd.c b/src/collectd.c
index a9b1ac80d847834bfff7836f2ca721ffb80ac5e7..247a185f08c753de4c11294b94e84ffd48c5591e 100644 (file)
--- a/src/collectd.c
+++ b/src/collectd.c
@@ -262,9 +262,11 @@ static int pidfile_remove (const char *file)
 
 int main (int argc, char **argv)
 {
+#if COLLECT_DAEMON
+       struct sigaction sigChldAction;
+#endif
        struct sigaction sigIntAction;
        struct sigaction sigTermAction;
-       struct sigaction sigChldAction;
        char *configfile = CONFIGFILE;
        char *plugindir  = PLUGINDIR;
        char *datadir    = PKGLOCALSTATEDIR;
@@ -383,21 +385,12 @@ int main (int argc, char **argv)
        }
 
        /*
-        * install signal handlers
+        * fork off child
         */
-       sigIntAction.sa_handler = sigIntHandler;
-       sigaction (SIGINT, &sigIntAction, NULL);
-
-       sigIntAction.sa_handler = sigTermHandler;
-       sigaction (SIGTERM, &sigTermAction, NULL);
-
+#if COLLECT_DAEMON
        sigChldAction.sa_handler = SIG_IGN;
        sigaction (SIGCHLD, &sigChldAction, NULL);
 
-       /*
-        * fork off child
-        */
-#if COLLECT_DAEMON
        if (daemonize)
        {
                if ((pid = fork ()) == -1)
@@ -443,6 +436,15 @@ int main (int argc, char **argv)
        } /* if (daemonize) */
 #endif /* COLLECT_DAEMON */
 
+       /*
+        * install signal handlers
+        */
+       sigIntAction.sa_handler = sigIntHandler;
+       sigaction (SIGINT, &sigIntAction, NULL);
+
+       sigIntAction.sa_handler = sigTermHandler;
+       sigaction (SIGTERM, &sigTermAction, NULL);
+
        /*
         * run the actual loops
         */