Changeset 33044 in vbox for trunk/src/libs/xpcom18a4/ipc/ipcd/client

Timestamp:

Oct 11, 2010 4:30:54 PM (14 years ago)

Author:

vboxsync

Message:

iprt/process: eliminate RTPROC_FLAGS_DAEMONIZE_DEPRECATED, rework starting a detached process on posix platforms and eliminate one useless fork(), avoid running

atexit functions in the temporary process

XPCOM: more detached process rework, block anything not going through IPRT, clos
e a few file descriptor inheritance leaks, and clean up the file descriptor pass
ing to VBoxXPCOMIPCD

File:

• trunk/src/libs/xpcom18a4/ipc/ipcd/client/src/ipcdclient.cpp (modified) (2 diffs)

trunk/src/libs/xpcom18a4/ipc/ipcd/client/src/ipcdclient.cpp

@@ -1216 +1216 @@
   PRProcessAttr *attr = nsnull;
   nsresult rv = NS_ERROR_FAILURE;
+  PRFileDesc *devNull;
   char *const argv[] = { (char *const) path, nsnull };
   char c;
@@ -1233 +1234 @@
 
   if (PR_ProcessAttrSetInheritableFD(attr, writable, IPC_STARTUP_PIPE_NAME) != PR_SUCCESS)
+  goto end;
+
+  devNull = PR_Open("/dev/null", PR_RDWR, 0);
+  if (!devNull)
     goto end;
+
+  PR_ProcessAttrSetStdioRedirect(attr, PR_StandardInput, devNull);
+  PR_ProcessAttrSetStdioRedirect(attr, PR_StandardOutput, devNull);
+  PR_ProcessAttrSetStdioRedirect(attr, PR_StandardError, devNull);
 
   if (PR_CreateProcessDetached(path, argv, nsnull, attr) != PR_SUCCESS)
     goto end;
 
+  // Close /dev/null
+  PR_Close(devNull);
   // close the child end of the pipe in order to get notification on unexpected
   // child termination instead of being infinitely blocked in PR_Read().