Use exit(3) instead of _exit(2) in the child processes.
authorrenato <renato@openbsd.org>
Mon, 8 Aug 2016 21:44:00 +0000 (21:44 +0000)
committerrenato <renato@openbsd.org>
Mon, 8 Aug 2016 21:44:00 +0000 (21:44 +0000)
Since recently the child processes call exec() after fork(), so they
should stop using _exit(2) and use exit(3) instead when shutting down.

Ok claudio@

usr.sbin/bgpd/log.c
usr.sbin/bgpd/rde.c
usr.sbin/bgpd/session.c

index f372844..025cf1d 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: log.c,v 1.59 2015/07/18 22:52:39 benno Exp $ */
+/*     $OpenBSD: log.c,v 1.60 2016/08/08 21:44:00 renato Exp $ */
 
 /*
  * Copyright (c) 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -227,10 +227,7 @@ fatal(const char *emsg, ...)
                        logit(LOG_CRIT, "fatal in %s: %s",
                            procnames[bgpd_process], s);
 
-       if (bgpd_process == PROC_MAIN)
-               exit(1);
-       else                            /* parent copes via SIGCHLD */
-               _exit(1);
+       exit(1);
 }
 
 void
index 50f4bc2..0b19f56 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: rde.c,v 1.347 2016/07/21 10:13:58 claudio Exp $ */
+/*     $OpenBSD: rde.c,v 1.348 2016/08/08 21:44:00 renato Exp $ */
 
 /*
  * Copyright (c) 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -329,7 +329,7 @@ rde_main(int debug, int verbose)
        free(ibuf_main);
 
        log_info("route decision engine exiting");
-       _exit(0);
+       exit(0);
 }
 
 struct network_config   netconf_s, netconf_p;
index e70c9db..a99a26a 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: session.c,v 1.351 2016/07/25 14:29:28 claudio Exp $ */
+/*     $OpenBSD: session.c,v 1.352 2016/08/08 21:44:00 renato Exp $ */
 
 /*
  * Copyright (c) 2003, 2004, 2005 Henning Brauer <henning@openbsd.org>
@@ -582,7 +582,7 @@ session_main(int debug, int verbose)
        control_shutdown(csock);
        control_shutdown(rcsock);
        log_info("session engine exiting");
-       _exit(0);
+       exit(0);
 }
 
 void