rde and eigrpe should use exit(3) instead of _exit(2)
authorrenato <renato@openbsd.org>
Mon, 8 Aug 2016 16:48:53 +0000 (16:48 +0000)
committerrenato <renato@openbsd.org>
Mon, 8 Aug 2016 16:48:53 +0000 (16:48 +0000)
Since recently these processes call exec() after fork(), so they should
stop using _exit(2) and use exit(3) instead when shutting down.

ok claudio@

usr.sbin/eigrpd/eigrpe.c
usr.sbin/eigrpd/log.c
usr.sbin/eigrpd/rde.c

index 1a4a143..56b2908 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: eigrpe.c,v 1.24 2016/06/05 17:07:41 renato Exp $ */
+/*     $OpenBSD: eigrpe.c,v 1.25 2016/08/08 16:48:53 renato Exp $ */
 
 /*
  * Copyright (c) 2015 Renato Westphal <renato@openbsd.org>
@@ -195,7 +195,7 @@ eigrpe_shutdown(void)
        free(pkt_ptr);
 
        log_info("eigrp engine exiting");
-       _exit(0);
+       exit(0);
 }
 
 /* imesg */
index 895f698..b8ee06b 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: log.c,v 1.3 2016/04/15 13:10:56 renato Exp $ */
+/*     $OpenBSD: log.c,v 1.4 2016/08/08 16:48:53 renato Exp $ */
 
 /*
  * Copyright (c) 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -157,10 +157,7 @@ fatal(const char *emsg)
                        logit(LOG_CRIT, "fatal in %s: %s",
                            procnames[eigrpd_process], emsg);
 
-       if (eigrpd_process == PROC_MAIN)
-               exit(1);
-       else                            /* parent copes via SIGCHLD */
-               _exit(1);
+       exit(1);
 }
 
 void
index 1e30220..bb9d32b 100644 (file)
@@ -1,4 +1,4 @@
-/*     $OpenBSD: rde.c,v 1.17 2016/06/05 03:36:41 renato Exp $ */
+/*     $OpenBSD: rde.c,v 1.18 2016/08/08 16:48:53 renato Exp $ */
 
 /*
  * Copyright (c) 2015 Renato Westphal <renato@openbsd.org>
@@ -142,7 +142,7 @@ rde_shutdown(void)
        free(iev_main);
 
        log_info("route decision engine exiting");
-       _exit(0);
+       exit(0);
 }
 
 int