Changeset 62159 in webkit

Timestamp:

Jun 29, 2010 8:46:11 PM (14 years ago)

Author:

Laszlo Gombos

Message:

2010-06-29 Abhinav Mithal <​abhinav.mithal@nokia.com>

Reviewed by Laszlo Gombos.

[Qt] [Symbian] Fix doube-deallocation while destroying PluginContainerSymbian
​https://bugs.webkit.org/show_bug.cgi?id=37303

• plugins/symbian/PluginViewSymbian.cpp: (WebCore::PluginView::platformDestroy):

Location:

trunk/WebCore

Files:

• ChangeLog (modified) (1 diff)
• plugins/symbian/PluginViewSymbian.cpp (modified) (1 diff)

trunk/WebCore/ChangeLog

@@ +1 @@
+2010-06-29  Abhinav Mithal <abhinav.mithal@nokia.com>
+
+        Reviewed by Laszlo Gombos.
+
+        [Qt] [Symbian] Fix doube-deallocation while destroying PluginContainerSymbian
+        https://bugs.webkit.org/show_bug.cgi?id=37303
+
+        * plugins/symbian/PluginViewSymbian.cpp:
+        (WebCore::PluginView::platformDestroy):
+
 2010-06-29  Zhenyao Mo  <zmo@google.com>
 

trunk/WebCore/plugins/symbian/PluginViewSymbian.cpp

@@ -419 +419 @@
     if (platformPluginWidget()) {
         PluginContainerSymbian* container = static_cast<PluginContainerSymbian*>(platformPluginWidget());
-        delete container->proxy();
-        delete container;
+        if (container && container->proxy())
+            delete container->proxy();
+        else
+            delete container;
     }
 }