Changeset 14999 in vbox

Timestamp:

Dec 4, 2008 5:12:09 PM (16 years ago)

Author:

vboxsync

Message:

IPRT: Extended RTProcCreate with a daemonize flag.

Location:

trunk

Files:

• include/iprt/process.h (modified) (1 diff)
• src/VBox/Runtime/r3/posix/process-posix.cpp (modified) (3 diffs)

trunk/include/iprt/process.h

@@ -131 +131 @@
  * @param   papszArgs   Pointer to an array of arguments to the child. The array terminated by an entry containing NULL.
  * @param   Env         Handle to the environment block for the child.
- * @param   fFlags      Flags. This is currently reserved and must be 0.
+ * @param   fFlags      Flags, one of the RTPROC_FLAGS_* defines.
  * @param   pProcess    Where to store the process identifier on successful return.
  *                      The content is not changed on failure. NULL is allowed.
  */
 RTR3DECL(int)   RTProcCreate(const char *pszExec, const char * const *papszArgs, RTENV Env, unsigned fFlags, PRTPROCESS pProcess);
+
+/** @name RTProcCreate flags
+ * @{ */
+/** Daemonize the child process, without changing the directory.
+ * @remarks Not implemented on all platforms yet... */
+#define RTPROC_FLAGS_DAEMONIZE          RT_BIT(0)
+/** @}  */
+
 
 /**

trunk/src/VBox/Runtime/r3/posix/process-posix.cpp

@@ -64 +64 @@
 RTR3DECL(int)   RTProcCreate(const char *pszExec, const char * const *papszArgs, RTENV Env, unsigned fFlags, PRTPROCESS pProcess)
 {
+    int rc;
+
     /*
      * Validate input.
@@ -69 +71 @@
     AssertPtrReturn(pszExec, VERR_INVALID_POINTER);
     AssertReturn(*pszExec, VERR_INVALID_PARAMETER);
-    AssertReturn(!fFlags, VERR_INVALID_PARAMETER);
+    AssertReturn(fFlags & ~RTPROC_FLAGS_DAEMONIZE, VERR_INVALID_PARAMETER);
     AssertReturn(Env != NIL_RTENV, VERR_INVALID_PARAMETER);
     const char * const *papszEnv = RTEnvGetExecEnvP(Env);
@@ -110 +112 @@
     pid_t pid;
 #ifdef HAVE_POSIX_SPAWN
-    /** @todo check if it requires any of those two attributes, don't remember atm. */
-    int rc = posix_spawn(&pid, pszExec, NULL, NULL, (char * const *)papszArgs,
+    if (!(fFlags & RTPROC_FLAGS_DAEMONIZE))
+    {
+        /** @todo check if it requires any of those two attributes, don't remember atm. */
+        rc = posix_spawn(&pid, pszExec, NULL, NULL, (char * const *)papszArgs,
                          (char * const *)papszEnv);
-    if (!rc)
-    {
-        if (pProcess)
-            *pProcess = pid;
-        return VINF_SUCCESS;
-    }
-
-#else
-
-    pid = fork();
-    if (!pid)
-    {
-        int rc;
-        rc = execve(pszExec, (char * const *)papszArgs, (char * const *)papszEnv);
-        AssertReleaseMsgFailed(("execve returns %d errno=%d\n", rc, errno));
-        exit(127);
-    }
-    if (pid > 0)
-    {
-        if (pProcess)
-            *pProcess = pid;
-        return VINF_SUCCESS;
-    }
-    int rc = errno;
-#endif
+        if (!rc)
+        {
+            if (pProcess)
+                *pProcess = pid;
+            return VINF_SUCCESS;
+        }
+    }
+    else
+#endif
+    {
+        pid = fork();
+        if (!pid)
+        {
+            if (fFlags & RTPROC_FLAGS_DAEMONIZE)
+            {
+                rc = RTProcDaemonize(true /* fNoChDir */, false /* fNoClose */, NULL /* pszPidFile */);
+                AssertReleaseMsgFailed(("RTProcDaemonize returns %Rrc errno=%d\n", rc, errno));
+                exit(127);
+            }
+            rc = execve(pszExec, (char * const *)papszArgs, (char * const *)papszEnv);
+            AssertReleaseMsgFailed(("execve returns %d errno=%d\n", rc, errno));
+            exit(127);
+        }
+        if (pid > 0)
+        {
+            if (pProcess)
+                *pProcess = pid;
+            return VINF_SUCCESS;
+        }
+        int rc = errno;
+    }
 
     /* failure, errno value in rc. */