Changeset 4100 in vbox

Timestamp:

Aug 9, 2007 5:13:24 PM (17 years ago)

Author:

vboxsync

Message:

Hopefully fix problem with controlling tty (xtracker defect #2166). Just
clearing it doesn't work, as we might automatically be attached to
another controlling tty if we're a session leader. So fork again. See
source of daemon() function in some libc versions (not POSIX).

File:

• trunk/src/VBox/Main/linux/server.cpp (modified) (2 diffs)

trunk/src/VBox/Main/linux/server.cpp

@@ -991 +991 @@
 
     /* nothing to do here, the process is supposed to be already
-     * started daemonized when it is necessary */  
+     * started daemonized when it is necessary */
     NOREF(fDaemonize);
 
@@ -1047 +1047 @@
             printf("ERROR: setsid() failed (errno = %d)\n", errno);
             return 1;
+        }
+
+        /* Need to do another for to get rid of the session leader status.
+         * Otherwise any accidentally opened tty will automatically become a
+         * controlling tty for the daemon process. */
+        childpid = fork();
+        if (childpid == -1)
+        {
+            printf("ERROR: second fork() failed (errno = %d)\n", errno);
+            return 1;
+        }
+
+        if (childpid != 0)
+        {
+            /* we're the parent process, just a dummy so terminate now */
+            exit(0);
         }