Changeset 45317 in vbox for trunk/src/VBox/Installer/win/Stub

Timestamp:

Apr 3, 2013 5:46:20 PM (12 years ago)

Author:

vboxsync

Message:

ARG - installer stub exits successfully on failure. -> windows box

File:

• trunk/src/VBox/Installer/win/Stub/VBoxStub.cpp (modified) (8 diffs)

trunk/src/VBox/Installer/win/Stub/VBoxStub.cpp

@@ -5 +5 @@
 
 /*
- * Copyright (C) 2010-2012 Oracle Corporation
+ * Copyright (C) 2010-2013 Oracle Corporation
  *
  * This file is part of VirtualBox Open Source Edition (OSE), as
@@ -348 +348 @@
     /* Do not use a global namespace ("Global\\") for mutex name here, will blow up NT4 compatibility! */
     HANDLE hMutexAppRunning = CreateMutex(NULL, FALSE, "VBoxStubInstaller");
-    if (   (hMutexAppRunning != NULL)
-        && (GetLastError() == ERROR_ALREADY_EXISTS))
+    if (   hMutexAppRunning != NULL
+        && GetLastError() == ERROR_ALREADY_EXISTS)
     {
         /* Close the mutex for this application instance. */
         CloseHandle(hMutexAppRunning);
         hMutexAppRunning = NULL;
-        return 1;
+        return RTEXITCODE_FAILURE;
     }
 
@@ -360 +360 @@
     int vrc = RTR3InitExe(argc, &argv, 0);
     if (RT_FAILURE(vrc))
-        return vrc;
+        return RTMsgInitFailure(vrc);
 
     /*
@@ -494 +494 @@
     }
 
+    /** @todo The rest of this function should be done in smaller functions,
+     *        we've lost the overview here! Too much state going around! */
+
     HRESULT hr = S_OK;
 
     do /* break loop */
     {
-        /* Get/create our temp path (only if not already set). */
+        /*
+         * Determine and create our temp path (only if not already set).
+         */
         if (szExtractPath[0] == '\0')
         {
@@ -684 +689 @@
                                     if (hModule)
                                         FreeLibrary(hModule);
+                                    /** @todo program exit code needs to be set */
                                     break;
                                 }
@@ -690 +696 @@
                             vrc = VERR_NO_CHANGE; /* No change done to the system. */
                         }
+                        else
+                            /** @todo program exit code needs to be for ERROR_INSTALL_USEREXIT. */
                     }
                     RTStrFree(pszTempFile);
@@ -700 +708 @@
             && RT_SUCCESS(vrc))
         {
-            for (int i=0; i<5; i++)
+            for (int i = 0; i < 5; i++)
             {
                 vrc = RTDirRemoveRecursive(szExtractPath, 0 /*fFlags*/);
@@ -729 +737 @@
     }
 
-    /* Set final exit (return) code (error level). */
+    /*
+     * Figure the exit code (not very difficult at the moment).
+     */
+    RTEXITCODE rcExit = RTEXITCODE_SUCCESS;
     if (RT_FAILURE(vrc))
-    {
-        switch(vrc)
-        {
-            case VERR_NO_CHANGE:
-            default:
-                vrc = 1;
-        }
-    }
-    else /* Always set to (VINF_SUCCESS), even if we got something else (like a VWRN etc). */
-        vrc = VINF_SUCCESS;
-    return vrc;
-}
-
+        rcExit = RTEXITCODE_FAILURE;
+    return rcExit;
+}
+