Changeset 47609 in vbox for trunk/src/VBox/Frontends

Timestamp:

Aug 8, 2013 10:56:04 AM (11 years ago)

Author:

vboxsync

Message:

FE/Qt: Popup-center: Trying to fix crash with mouse-release events handling for stacks/panes which already destroyed.

Location:

trunk/src/VBox/Frontends/VirtualBox/src

Files:

• globals/UIPopupCenter.cpp (modified) (1 diff)
• widgets/UIPopupStackViewport.cpp (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/globals/UIPopupCenter.cpp

@@ -340 +340 @@
     }
 
-    /* Delete popup-stack: */
-    delete m_stacks[strPopupStackID];
+    /* Delete popup-stack asyncronously.
+     * To avoid issues with events which already posted: */
+    UIPopupStack *pPopupStack = m_stacks[strPopupStackID];
     m_stacks.remove(strPopupStackID);
+    pPopupStack->deleteLater();
 }
 

trunk/src/VBox/Frontends/VirtualBox/src/widgets/UIPopupStackViewport.cpp

@@ -140 +140 @@
     emit sigPopupPaneDone(strPopupPaneID, iResultCode);
 
-    /* Cleanup the popup-pane: */
+    /* Delete popup-pane asyncronously.
+     * To avoid issues with events which already posted: */
     m_panes.remove(strPopupPaneID);
-    delete pPopupPane;
+    pPopupPane->deleteLater();
 
     /* Notify listeners about popup-pane removed: */