Properly detach at startup. We could be passing revoked fd's to

child processes or starting children with no fd 0 (stdin) at all etc.
This is currently breaking on hub.freebsd.org.

1 files changed, 2 insertions, 12 deletions

diff --git a/usr.sbin/cron/cron/cron.c b/usr.sbin/cron/cron/cron.c
index e8f8557..8889b72 100644
--- a/usr.sbin/cron/cron/cron.c
+++ b/usr.sbin/cron/cron/cron.c
@@ -100,19 +100,9 @@ main(argc, argv)
 # endif
 		(void) fprintf(stderr, "[%d] cron started\n", getpid());
 	} else {
-		switch (fork()) {
-		case -1:
-			log_it("CRON",getpid(),"DEATH","can't fork");
+		if (daemon(1, 0) == -1) {
+			log_it("CRON",getpid(),"DEATH","can't become daemon");
 			exit(0);
-			break;
-		case 0:
-			/* child process */
-			log_it("CRON",getpid(),"STARTUP","fork ok");
-			(void) setsid();
-			break;
-		default:
-			/* parent process should just die */
-			_exit(0);
 		}
 	}