Remove pid fron cron serives, reap children in signal handler

Signed-off-by: David Oberhollenzer <david.oberhollenzer@tele2.at>

2 files changed, 9 insertions, 14 deletions

diff --git a/crond/main.c b/crond/main.c
index d7cc33e..613a1ae 100644
--- a/crond/main.c
+++ b/crond/main.c
@@ -79,6 +79,8 @@ static void runjobs(void)
 
 static void sighandler(int signo)
 {
+	pid_t pid;
+
 	switch (signo) {
 	case SIGINT:
 	case SIGTERM:
@@ -87,6 +89,10 @@ static void sighandler(int signo)
 	case SIGHUP:
 		rescan = 1;
 		break;
+	case SIGCHLD:
+		while ((pid = waitpid(-1, NULL, WNOHANG)) != -1)
+			;
+		break;
 	}
 }
 
@@ -94,15 +100,14 @@ int main(void)
 {
 	struct timespec stime;
 	struct sigaction act;
-	crontab_t *t;
 	int timeout;
-	pid_t pid;
 
 	memset(&act, 0, sizeof(act));
 	act.sa_handler = sighandler;
 	sigaction(SIGINT, &act, NULL);
 	sigaction(SIGTERM, &act, NULL);
 	sigaction(SIGHUP, &act, NULL);
+	sigaction(SIGCHLD, &act, NULL);
 
 	while (run) {
 		if (rescan == 1) {
@@ -124,15 +129,6 @@ int main(void)
 				break;
 			}
 		}
-
-		while ((pid = waitpid(-1, NULL, WNOHANG)) != -1) {
-			for (t = jobs; t != NULL; t = t->next) {
-				if (t->pid == pid) {
-					t->pid = -1;
-					break;
-				}
-			}
-		}
 	}
 
 	return EXIT_SUCCESS;
diff --git a/crond/runjob.c b/crond/runjob.c
index 7650748..83c4278 100644
--- a/crond/runjob.c
+++ b/crond/runjob.c
@@ -33,10 +33,8 @@ int runjob(crontab_t *tab)
 		return -1;
 	}
 
-	if (pid != 0) {
-		tab->pid = pid;
+	if (pid != 0)
 		return 0;
-	}
 
 	/* XXX: inside the child process */
 	memset(&act, 0, sizeof(act));
@@ -44,6 +42,7 @@ int runjob(crontab_t *tab)
 	sigaction(SIGINT, &act, NULL);
 	sigaction(SIGTERM, &act, NULL);
 	sigaction(SIGHUP, &act, NULL);
+	sigaction(SIGCHLD, &act, NULL);
 
 	if (setup_tty(tab->ctty, tab->tty_truncate))
 		exit(EXIT_FAILURE);