Changeset 59888 in vbox for trunk/src/VBox/Frontends/VirtualBox

Timestamp:

Mar 1, 2016 3:16:06 PM (9 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

105792

Message:

FE/Qt: bugref:8200: Runtime UI: Stability fix for PowerUp procedure: Using actual machine state instead of cached one when necessary.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/runtime/UISession.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/runtime/UISession.cpp

@@ -213 +213 @@
             return false;
 
-    /* Check if we missed a really quick termination after successful startup: */
-    if (isTurnedOff())
-        return false;
+    /* Check if we missed a really quick termination after successful powerUp().
+     * We should take into account the fact of async machine-state arrival.
+     * And we don't even want to loop at this point to handle posted
+     * machine-state events, so we are relying onto CMachine getter
+     * to return the fresh state of the thing we want. */
+    const KMachineState ms = machine().GetState();
+    switch (ms)
+    {
+        case KMachineState_PoweredOff:
+        case KMachineState_Saved:
+        case KMachineState_Teleported:
+        case KMachineState_Aborted:
+            return false;
+        default:
+            break;
+    }
 
     /* Postprocess initialization: */