Don't bother fork()ing after closing a ctty if ppp is about to

terminate anyway.

1 files changed, 10 insertions, 0 deletions

diff --git a/usr.sbin/ppp/bundle.c b/usr.sbin/ppp/bundle.c
index c61f388..d55172d 100644
--- a/usr.sbin/ppp/bundle.c
+++ b/usr.sbin/ppp/bundle.c
@@ -1697,6 +1697,16 @@ bundle_setsid(struct bundle *bundle, int holdsession)
   char done;
   struct datalink *dl;
 
+  if (!holdsession && bundle_IsDead(bundle)) {
+    /*
+     * No need to lose our session after all... we're going away anyway
+     *
+     * We should really stop the timer and pause if holdsession is set and
+     * the bundle's dead, but that leaves other resources lying about :-(
+     */
+    return;
+  }
+
   orig = getpid();
   if (pipe(fds) == -1) {
     log_Printf(LogERROR, "pipe: %s\n", strerror(errno));