Changeset 32136 in vbox

Timestamp:

Aug 31, 2010 12:24:38 PM (14 years ago)

Author:

vboxsync

Message:

FE/Qt: Problem Reporter: Fixed segfaults while trying to acquire data from (or just delete) QIMessageBox already deleted by it's parent.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/globals/VBoxProblemReporter.cpp (modified) (3 diffs)

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

@@ -199 +199 @@
     }
 
-    QIMessageBox *box = new QIMessageBox (title, aMessage, icon, aButton1, aButton2,
-                                          aButton3, aParent, aAutoConfirmId);
+    QPointer<QIMessageBox> box = new QIMessageBox (title, aMessage, icon, aButton1, aButton2,
+                                                   aButton3, aParent, aAutoConfirmId);
 
     if (!aText1.isNull())
@@ -222 +222 @@
     if (aAutoConfirmId)
     {
-        if (box->isFlagChecked())
+        if (box && box->isFlagChecked())
         {
             msgs << aAutoConfirmId;
@@ -229 +229 @@
     }
 
-    delete box;
+    if (box)
+        delete box;
 
     return rc;