Changeset 34709 in vbox

Timestamp:

Dec 3, 2010 5:38:01 PM (14 years ago)

Author:

vboxsync

Message:

Guest execution: Implemented ability to start guest processes hidden.

Location:

trunk/src/VBox

Files:

• Additions/common/VBoxService/VBoxServiceControlExec.cpp (modified) (2 diffs)
• Frontends/VBoxManage/VBoxManageGuestCtrl.cpp (modified) (1 diff)
• Main/GuestImpl.cpp (modified) (5 diffs)
• Main/idl/VirtualBox.xidl (modified) (1 diff)

trunk/src/VBox/Additions/common/VBoxService/VBoxServiceControlExec.cpp

@@ -1210 +1210 @@
         if (RT_SUCCESS(rc))
         {
-            /* If no user name specified run with current credentials.
-             * This is prohibited via official Main API! */
-            if (!strlen(pszAsUser))
-                fFlags &= ~RTPROC_FLAGS_SERVICE;
+            uint32_t uProcFlags = 0;
+            if (fFlags)
+            {
+                /* Process Main flag "ExecuteProcessFlag_Hidden". */
+                if (fFlags & RT_BIT(2))
+                    uProcFlags = RTPROC_FLAGS_HIDDEN;
+            }
+
+            /* If no user name specified run with current credentials (e.g.
+             * full service/system rights). This is prohibited via official Main API!
+             *
+             * Otherwise use the RTPROC_FLAGS_SERVICE to use some special authentication
+             * code (at least on Windows) for running processes as different users
+             * started from our system service. */
+            if (strlen(pszAsUser))
+                uProcFlags |= RTPROC_FLAGS_SERVICE;
 
             /* Do normal execution. */
-            rc = RTProcCreateEx(szExecExp, papszArgsExp, hEnv, fFlags,
+            rc = RTProcCreateEx(szExecExp, papszArgsExp, hEnv, uProcFlags,
                                 phStdIn, phStdOut, phStdErr,
                                 strlen(pszAsUser) ? pszAsUser : NULL,
@@ -1308 +1320 @@
                             {
                                 RTPROCESS hProcess;
-                                rc = VBoxServiceControlExecCreateProcess(pData->pszCmd, pData->papszArgs, hEnv, RTPROC_FLAGS_SERVICE,
+                                rc = VBoxServiceControlExecCreateProcess(pData->pszCmd, pData->papszArgs, hEnv, pData->uFlags,
                                                                          phStdIn, phStdOut, phStdErr,
                                                                          pData->pszUser, pData->pszPassword,

trunk/src/VBox/Frontends/VBoxManage/VBoxManageGuestCtrl.cpp

@@ -349 +349 @@
             case 'f': /* Flags */
                 /** @todo Needs a bit better processing as soon as we have more flags. */
+                /** @todo Add a hidden flag. */
                 if (!RTStrICmp(ValueUnion.psz, "ignoreorphanedprocesses"))
                     uFlags |= ExecuteProcessFlag_IgnoreOrphanedProcesses;

trunk/src/VBox/Main/GuestImpl.cpp

@@ -263 +263 @@
                      */
                     rc = pGuest->executeProcessInternal(Bstr(VBOXSERVICE_TOOL_CAT).raw(),
-                                                        ExecuteProcessFlag_WaitForProcessStartOnly,
+                                                          ExecuteProcessFlag_Hidden
+                                                        | ExecuteProcessFlag_WaitForProcessStartOnly,
                                                         ComSafeArrayAsInParam(args),
                                                         ComSafeArrayAsInParam(env),
@@ -410 +411 @@
                 /*
                  * Start the just copied over installer with system rights
-                 * in silent mode on the guest.
+                 * in silent mode on the guest. Don't use the hidden flag since there
+                 * may be pop ups the user has to process.
                  */
                 ComPtr<IProgress> progressInstaller;
@@ -1597 +1599 @@
     {
         if (   !(aFlags & ExecuteProcessFlag_IgnoreOrphanedProcesses)
-            && !(aFlags & ExecuteProcessFlag_WaitForProcessStartOnly))
+            && !(aFlags & ExecuteProcessFlag_WaitForProcessStartOnly)
+            && !(aFlags & ExecuteProcessFlag_Hidden))
         {
             if (pRC)
@@ -2434 +2437 @@
                          */
                         rc = ExecuteProcess(Bstr(VBOXSERVICE_TOOL_CAT).raw(),
-                                            ExecuteProcessFlag_WaitForProcessStartOnly,
+                                              ExecuteProcessFlag_Hidden
+                                            | ExecuteProcessFlag_WaitForProcessStartOnly,
                                             ComSafeArrayAsInParam(args),
                                             ComSafeArrayAsInParam(env),
@@ -2591 +2595 @@
         {
             rc = ExecuteProcess(Bstr(VBOXSERVICE_TOOL_MKDIR).raw(),
-                                ExecuteProcessFlag_None,
+                                ExecuteProcessFlag_Hidden,
                                 ComSafeArrayAsInParam(args),
                                 ComSafeArrayAsInParam(env),

trunk/src/VBox/Main/idl/VirtualBox.xidl

@@ -7871 +7871 @@
       <desc>Do not report an error when executed processes are still alive when VBoxService or the guest OS is shutting down.</desc>
     </const>
+
+    <const name="Hidden"                  value="4">
+      <desc>Don't show the started process according to the guest OS guidelines.</desc>
+    </const>
   </enum>