Changeset 99660 in vbox

Timestamp:

May 8, 2023 9:47:22 AM (19 months ago)

Author:

vboxsync

Message:

Guest Additions/VBoxClient: Repaired setting the g_fDaemonized flag, added better error handling for daemonizing.

File:

• trunk/src/VBox/Additions/x11/VBoxClient/main.cpp (modified) (2 diffs)

trunk/src/VBox/Additions/x11/VBoxClient/main.cpp

@@ -103 +103 @@
 /** Critical section for the signal handler. */
 static RTCRITSECT      g_csSignalHandler;
-/** Flag indicating Whether the service starts in daemonized  mode or not. */
+/** Flag indicating Whether the service starts in daemonized mode or not. */
 bool                   g_fDaemonized = false;
 /** The name of our pidfile.  It is global for the benefit of the cleanup
@@ -787 +787 @@
     if (fDaemonise)
     {
+        VBClLogInfo("Daemonizing service ...\n");
         rc = VbglR3DaemonizeEx(false /* fNoChDir */, false /* fNoClose */, fRespawn, &g_cRespawn,
                                true /* fReturnOnUpdate */, &fUpdateStarted, g_szControlPidFile, &g_hControlPidFile);
-        /* This combination only works in context of parent process. */
-        if (RT_SUCCESS(rc) && fUpdateStarted)
-            vbclHandleUpdateStarted(argv);
-    }
-
-    if (RT_FAILURE(rc))
-        VBClLogFatalError("Daemonizing service failed: %Rrc\n", rc);
+        if (RT_SUCCESS(rc))
+        {
+            g_fDaemonized = true;
+
+            if (fUpdateStarted) /* This combination only works in context of parent process. */
+                vbclHandleUpdateStarted(argv);
+        }
+        else
+            return RTMsgErrorExitFailure("Daemonizing service failed: %Rrc\n", rc);
+    }
 
     if (g_szPidFile[0])