[EFL][WK2] Fix sub menu initialization in ewk_context_menu_item
authorch.dumez@sisa.samsung.com <ch.dumez@sisa.samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 May 2013 16:53:55 +0000 (16:53 +0000)
committerch.dumez@sisa.samsung.com <ch.dumez@sisa.samsung.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 17 May 2013 16:53:55 +0000 (16:53 +0000)
https://bugs.webkit.org/show_bug.cgi?id=116321

Patch by Michał Pakuła vel Rutka <m.pakula@samsung.com> on 2013-05-17
Reviewed by Christophe Dumez.

During sub menu creation, after initializing m_parentMenu, parentMenu
pointer became null which may cause segfaults. This patch fixes it by
using m_parentMenu.

* UIProcess/API/efl/ewk_context_menu_item.cpp:
(EwkContextMenuItem::EwkContextMenuItem):

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@150267 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Source/WebKit2/ChangeLog
Source/WebKit2/UIProcess/API/efl/ewk_context_menu_item.cpp

index b24a872..6dd5728 100644 (file)
@@ -1,5 +1,19 @@
 2013-05-17  Michał Pakuła vel Rutka  <m.pakula@samsung.com>
 
+        [EFL][WK2] Fix sub menu initialization in ewk_context_menu_item
+        https://bugs.webkit.org/show_bug.cgi?id=116321
+
+        Reviewed by Christophe Dumez.
+
+        During sub menu creation, after initializing m_parentMenu, parentMenu
+        pointer became null which may cause segfaults. This patch fixes it by
+        using m_parentMenu.
+
+        * UIProcess/API/efl/ewk_context_menu_item.cpp:
+        (EwkContextMenuItem::EwkContextMenuItem):
+
+2013-05-17  Michał Pakuła vel Rutka  <m.pakula@samsung.com>
+
         [EFL][WK2] Support sub menu in ewk context menus
         https://bugs.webkit.org/show_bug.cgi?id=114729
 
index 62afd69..0300f79 100644 (file)
@@ -46,7 +46,7 @@ EwkContextMenuItem::EwkContextMenuItem(WKContextMenuItemRef item, PassRefPtr<Ewk
 {
     if (WKContextMenuItemGetType(item) == kWKContextMenuItemTypeSubmenu) {
         WKRetainPtr<WKArrayRef> menuItems = adoptWK(WKContextMenuCopySubmenuItems(item));
-        m_subMenu = EwkContextMenu::create(parentMenu->ewkView(), menuItems.get());
+        m_subMenu = EwkContextMenu::create(m_parentMenu->ewkView(), menuItems.get());
     }
 }