136617 2014-09-08 00:17:28 -0700 [EFL[WK2] MiniBrowser comes to crash on debug mode 2014-09-08 03:01:33 -0700 1 1 1 Unclassified WebKit WebKit EFL 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 136324 1 gyuyoung.kim gyuyoung.kim cdumez commit-queue lucas.de.marchi ossy ryuan.choi sergio oldest_to_newest 1033780 0 gyuyoung.kim 2014-09-08 00:17:28 -0700 There is a crash as soon as Minibrowser runs on debug mode. gyuyoung@gyuyoung-MacBookPro:~/WebKit$ WebKitBuild/Debug/bin/MiniBrowser HTML5 local storage is enabled for this view. ERR<3506>:efreet_cache lib/efreet/efreet_cache.c:1108 on_send_register() org.freedesktop.DBus.Error.ServiceUnknown The name org.enlightenment.Efreet was not provided by any .service files ASSERTION FAILED: !window /home/gyuyoung/WebKit/Source/WebCore/platform/efl/EflScreenUtilities.cpp(113) : void WebCore::applyCursorFromEcoreX(Ecore_X_Window, const char*) 1 0x7f02fabbcfdd WTFCrash 2 0x7f02f65b8a97 WebCore::applyCursorFromEcoreX(unsigned int, char const*) 3 0x7f02fdd47a94 EwkView::updateCursor(unsigned int) 4 0x7f02fdd47c3c EwkView::setCursor(WebCore::Cursor const&) 5 0x7f02fddb95c4 WebKit::WebViewEfl::setCursor(WebCore::Cursor const&) 6 0x7f02fdb2c2ab WebKit::WebPageProxy::setCursor(WebCore::Cursor const&) 7 0x7f02fde05407 void IPC::callMemberFunctionImpl<WebKit::WebPageProxy, void (WebKit::WebPageProxy::*)(WebCore::Cursor const&), std::tuple<WebCore::Cursor>, 0ul>(WebKit::WebPageProxy*, void (WebKit::WebPageProxy::*)(WebCore::Cursor const&), std::tuple<WebCore::Cursor>&&, std::index_sequence<0ul>) 8 0x7f02fde014d8 void IPC::callMemberFunction<WebKit::WebPageProxy, void (WebKit::WebPageProxy::*)(WebCore::Cursor const&), std::tuple<WebCore::Cursor>, std::make_index_sequence<1ul> >(std::tuple<WebCore::Cursor>&&, WebKit::WebPageProxy*, void (WebKit::WebPageProxy::*)(WebCore::Cursor const&)) 9 0x7f02fddf5fdf void IPC::handleMessage<Messages::WebPageProxy::SetCursor, WebKit::WebPageProxy, void (WebKit::WebPageProxy::*)(WebCore::Cursor const&)>(IPC::MessageDecoder&, WebKit::WebPageProxy*, void (WebKit::WebPageProxy::*)(WebCore::Cursor const&)) 10 0x7f02fddf08d2 WebKit::WebPageProxy::didReceiveMessage(IPC::Connection*, IPC::MessageDecoder&) 11 0x7f02fda1f62e IPC::MessageReceiverMap::dispatchMessage(IPC::Connection*, IPC::MessageDecoder&) 12 0x7f02fda3c7e7 WebKit::ChildProcessProxy::dispatchMessage(IPC::Connection*, IPC::MessageDecoder&) 13 0x7f02fdb651c7 WebKit::WebProcessProxy::didReceiveMessage(IPC::Connection*, IPC::MessageDecoder&) 14 0x7f02fda0eae0 IPC::Connection::dispatchMessage(IPC::MessageDecoder&) 15 0x7f02fda0ebac IPC::Connection::dispatchMessage(std::unique_ptr<IPC::MessageDecoder, std::default_delete<IPC::MessageDecoder> >) 16 0x7f02fda0ed6d IPC::Connection::dispatchOneMessage() 17 0x7f02fda1ea1f WTF::FunctionWrapper<void (IPC::Connection::*)()>::operator()(IPC::Connection*) 18 0x7f02fda1e792 WTF::BoundFunctionImpl<WTF::FunctionWrapper<void (IPC::Connection::*)()>, void (IPC::Connection*)>::operator()() 19 0x7f02fda1799d WTF::Function<void ()>::operator()() const 20 0x7f02fda13e65 std::_Function_handler<void (), WTF::Function<void ()> >::_M_invoke(std::_Any_data const&) 21 0x7f02fda275a0 std::function<void ()>::operator()() const 22 0x7f02fde47ba1 WTF::RunLoop::performWork() 23 0x7f02fde48d94 WTF::RunLoop::wakeUpEvent(void*, void*, unsigned int) 24 0x7f02fcb2f07f 25 0x7f02fcb2f80a 26 0x7f02fcb2e811 27 0x7f02fcb2ea17 ecore_main_loop_begin 28 0x40caf5 elm_main 29 0x40cb68 main 30 0x7f02fb6ccec5 __libc_start_main 31 0x4060d9 Error receiving IPC message on socket 27 in process 3537: Connection reset by peer Segmentation fault (core dumped) gyuyoung@gyuyoung-MacBookPro:~/WebKit$ LEAK: 1 WebPage LEAK: 1 WebFrame LEAK: 200 RenderObject LEAK: 1 Page LEAK: 1 Frame LEAK: 8 CachedResource LEAK: 4 SubresourceLoader LEAK: 287 WebCoreNode LEAK: 24 JSLazyEventListener 1033781 1 237763 gyuyoung.kim 2014-09-08 00:23:34 -0700 Created attachment 237763 Patch 1033782 2 gyuyoung.kim 2014-09-08 00:24:40 -0700 In applyCursorFromEcoreX(), ASSERT(!window) makes a crash though window isn't null. This is wrong ASSERT use. This patch fixes it. 1033799 3 237763 ossy 2014-09-08 02:25:21 -0700 Comment on attachment 237763 Patch LGTM, r=me. 1033800 4 ossy 2014-09-08 02:26:31 -0700 Just to note, this buggy assert comes from http://trac.webkit.org/changeset/173354 1033811 5 237763 commit-queue 2014-09-08 03:01:26 -0700 Comment on attachment 237763 Patch Clearing flags on attachment: 237763 Committed r173373: <http://trac.webkit.org/changeset/173373> 1033812 6 commit-queue 2014-09-08 03:01:33 -0700 All reviewed patches have been landed. Closing bug. 237763 2014-09-08 00:23:34 -0700 2014-09-08 03:01:26 -0700 Patch bug-136617-20140908162328.patch text/plain 1470 gyuyoung.kim U3VidmVyc2lvbiBSZXZpc2lvbjogMTczMzY1CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggMDIzODc2YTkxM2JkNzgz MTVkOWY5MGY1YzY4NjIzYzVjZjY2M2Q1My4uNWFhZTFiYWY1YTg4OWE4NDM1OWZiNDk5ZjFlYmFm YTA3ZGM0MmQ4YiAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDE0LTA5LTA4ICBHeXV5 b3VuZyBLaW0gIDxneXV5b3VuZy5raW1Ac2Ftc3VuZy5jb20+CisKKyAgICAgICAgW0VGTFtXSzJd IE1pbmlCcm93c2VyIGNvbWVzIHRvIGNyYXNoIG9uIGRlYnVnIG1vZGUKKyAgICAgICAgaHR0cHM6 Ly9idWdzLndlYmtpdC5vcmcvc2hvd19idWcuY2dpP2lkPTEzNjYxNworCisgICAgICAgIFJldmll d2VkIGJ5IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgIEZpeCB3cm9uZyBBU1NFUlRJT04gdXNl IGluIGFwcGx5Q3Vyc29yRnJvbUVjb3JlWCgpLgorCisgICAgICAgICogcGxhdGZvcm0vZWZsL0Vm bFNjcmVlblV0aWxpdGllcy5jcHA6IENoYW5nZSBBU1NFUlQoIXdpbmRvdykgd2l0aCAqQVNTRVJU KHdpbmRvdykqCisgICAgICAgIChXZWJDb3JlOjphcHBseUN1cnNvckZyb21FY29yZVgpOgorCiAy MDE0LTA5LTA2ICBKZXIgTm9ibGUgIDxqZXIubm9ibGVAYXBwbGUuY29tPgogCiAgICAgICAgIFtG dWxsc2NyZWVuXSBBZGQgYSBzaXRlLXNwZWNpZmljIGhhY2sgdG8gd29yayBhcm91bmQgImZsYXNo IG9uIGV4aXQiIGJlaGF2aW9yIG9mIFlvdVR1YmUuY29tLgpkaWZmIC0tZ2l0IGEvU291cmNlL1dl YkNvcmUvcGxhdGZvcm0vZWZsL0VmbFNjcmVlblV0aWxpdGllcy5jcHAgYi9Tb3VyY2UvV2ViQ29y ZS9wbGF0Zm9ybS9lZmwvRWZsU2NyZWVuVXRpbGl0aWVzLmNwcAppbmRleCBmNjFiYjg2ZjFhZmY0 OTUzNWI5ZTA3YWFiNzE2MDEzOTMyNzFmMGM5Li4zYTJlMzBhMmMwYTNmZWZiNjliYTA2Y2VhNDc4 MmM3YWJhNDA2NjI0IDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9lZmwvRWZs U2NyZWVuVXRpbGl0aWVzLmNwcAorKysgYi9Tb3VyY2UvV2ViQ29yZS9wbGF0Zm9ybS9lZmwvRWZs U2NyZWVuVXRpbGl0aWVzLmNwcApAQCAtMTEwLDcgKzExMCw3IEBAIGludCBnZXRFY29yZUN1cnNv cihjb25zdCBTdHJpbmcmIGN1cnNvclN0cmluZykKIAogdm9pZCBhcHBseUN1cnNvckZyb21FY29y ZVgoRWNvcmVfWF9XaW5kb3cgd2luZG93LCBjb25zdCBjaGFyKiBjdXJzb3JTdHJpbmcpCiB7Ci0g ICAgQVNTRVJUKCF3aW5kb3cpOworICAgIEFTU0VSVCh3aW5kb3cpOwogCiAgICAgaW50IHNoYXBl ID0gZ2V0RWNvcmVDdXJzb3IoY3Vyc29yU3RyaW5nKTsKICAgICBpZiAoc2hhcGUgPCBFQ09SRV9Y X0NVUlNPUl9YIHx8IHNoYXBlID4gRUNPUkVfWF9DVVJTT1JfWFRFUk0pIHsK