120159 2013-08-22 06:11:57 -0700 Missing null-check in HTMLFormElement::rendererIsNeeded() 2013-08-23 01:27:48 -0700 1 1 1 Unclassified WebKit Forms 528+ (Nightly build) Unspecified Unspecified RESOLVED FIXED P2 Normal --- 116980 1 rhodovan.u-szeged rhodovan.u-szeged darin kling koivisto rniwa oldest_to_newest 919828 0 rhodovan.u-szeged 2013-08-22 06:11:57 -0700 The following test is crashing both with release and debug build since parentRenderer is null in WebCore::HTMLFormElement::rendererIsNeeded: <html> <blockquote style="-webkit-flow-from: thread;"/> <table/> <form/> </html> Backtrace: Program received signal SIGSEGV, Segmentation fault. 0x00007ffff439825a in WebCore::HTMLFormElement::rendererIsNeeded (this=0x8eb5b0, context=...) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/html/HTMLFormElement.cpp:106 warning: Source file is more recent than executable. 106 || (parentRenderer->isTableRow() && node->hasTagName(trTag)) (gdb) bt #0 0x00007ffff439825a in WebCore::HTMLFormElement::rendererIsNeeded (this=0x8eb5b0, context=...) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/html/HTMLFormElement.cpp:106 #1 0x00007ffff4258900 in WebCore::NodeRenderingContext::elementInsideRegionNeedsRenderer (this=0x7fffffffc790) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/dom/NodeRenderingContext.cpp:216 #2 0x00007ffff4258b6e in WebCore::NodeRenderingContext::createRendererForElementIfNeeded (this=0x7fffffffc790) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/dom/NodeRenderingContext.cpp:257 #3 0x00007ffff42058eb in WebCore::Element::createRendererIfNeeded (this=0x8eb5b0, context=...) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/dom/Element.cpp:1390 #4 0x00007ffff420594f in WebCore::Element::attach (this=0x8eb5b0, context=...) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/dom/Element.cpp:1399 #5 0x00007ffff4406792 in WebCore::executeTask (task=...) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/html/parser/HTMLConstructionSite.cpp:103 #6 0x00007ffff4406ae3 in WebCore::HTMLConstructionSite::executeQueuedTasks (this=0x8261b8) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/html/parser/HTMLConstructionSite.cpp:146 #7 0x00007ffff442fecc in WebCore::HTMLTreeBuilder::constructTree (this=0x8261a0, token=0x7fffffffc920) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/html/parser/HTMLTreeBuilder.cpp:382 #8 0x00007ffff440eb06 in WebCore::HTMLDocumentParser::constructTreeFromHTMLToken (this=0x7e2db0, rawToken=...) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/html/parser/HTMLDocumentParser.cpp:597 #9 0x00007ffff440e73b in WebCore::HTMLDocumentParser::pumpTokenizer (this=0x7e2db0, mode=WebCore::HTMLDocumentParser::AllowYield) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/html/parser/HTMLDocumentParser.cpp:551 #10 0x00007ffff440df03 in WebCore::HTMLDocumentParser::pumpTokenizerIfPossible (this=0x7e2db0, mode=WebCore::HTMLDocumentParser::AllowYield) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/html/parser/HTMLDocumentParser.cpp:235 #11 0x00007ffff440f0a2 in WebCore::HTMLDocumentParser::append (this=0x7e2db0, inputSource=...) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/html/parser/HTMLDocumentParser.cpp:747 #12 0x00007ffff41a1e4f in WebCore::DecodedDataDocumentParser::flush (this=0x7e2db0, writer=0x6942b0) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/dom/DecodedDataDocumentParser.cpp:60 #13 0x00007ffff45a71d9 in WebCore::DocumentWriter::end (this=0x6942b0) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/loader/DocumentWriter.cpp:245 #14 0x00007ffff4599d52 in WebCore::DocumentLoader::finishedLoading (this=0x694210, finishTime=0) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/loader/DocumentLoader.cpp:402 #15 0x00007ffff4599ac0 in WebCore::DocumentLoader::notifyFinished (this=0x694210, resource=0x7dd950) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/loader/DocumentLoader.cpp:344 #16 0x00007ffff4580db6 in WebCore::CachedResource::checkNotify (this=0x7dd950) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/loader/cache/CachedResource.cpp:369 #17 0x00007ffff4580e8c in WebCore::CachedResource::finishLoading (this=0x7dd950) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/loader/cache/CachedResource.cpp:385 #18 0x00007ffff457d5de in WebCore::CachedRawResource::finishLoading (this=0x7dd950, data=0x7636b0) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/loader/cache/CachedRawResource.cpp:94 #19 0x00007ffff45e3c41 in WebCore::SubresourceLoader::didFinishLoading (this=0x7c7f80, finishTime=0) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/loader/SubresourceLoader.cpp:282 #20 0x00007ffff45da52b in WebCore::ResourceLoader::didFinishLoading (this=0x7c7f80, finishTime=0) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/loader/ResourceLoader.cpp:488 #21 0x00007ffff4a85729 in WebCore::QNetworkReplyHandler::finish (this=0x7dc580) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/platform/network/qt/QNetworkReplyHandler.cpp:516 #22 0x00007ffff4a84448 in WebCore::QNetworkReplyHandlerCallQueue::flush (this=0x7dc5b8) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/platform/network/qt/QNetworkReplyHandler.cpp:250 #23 0x00007ffff4a84145 in WebCore::QNetworkReplyHandlerCallQueue::push (this=0x7dc5b8, method=(void (WebCore::QNetworkReplyHandler::*)(WebCore::QNetworkReplyHandler * const)) 0x7ffff4a8556e <WebCore::QNetworkReplyHandler::finish()>) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/platform/network/qt/QNetworkReplyHandler.cpp:216 #24 0x00007ffff4a85092 in WebCore::QNetworkReplyWrapper::didReceiveFinished (this=0x7bbc10) at /home/reni/Data/REPOS/webkit_sec/Source/WebCore/platform/network/qt/QNetworkReplyHandler.cpp:409 #25 0x00007ffff4a87a24 in WebCore::QNetworkReplyWrapper::qt_static_metacall (_o=0x7bbc10, _c=QMetaObject::InvokeMetaMethod, _id=1, _a=0x7fffffffcf60) at .moc/release-shared/moc_QNetworkReplyHandler.cpp:176 #26 0x00007ffff220f5cb in QMetaObject::activate(QObject*, int, int, void**) () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Core.so.5 #27 0x00007ffff221084e in QObject::event(QEvent*) () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Core.so.5 #28 0x00007ffff3056dbc in QApplicationPrivate::notify_helper(QObject*, QEvent*) () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Widgets.so.5 ---Type <return> to continue, or q <return> to quit--- #29 0x00007ffff305a075 in QApplication::notify(QObject*, QEvent*) () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Widgets.so.5 #30 0x00007ffff21eadbe in QCoreApplication::notifyInternal(QObject*, QEvent*) () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Core.so.5 #31 0x00007ffff21eca76 in QCoreApplicationPrivate::sendPostedEvents(QObject*, int, QThreadData*) () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Core.so.5 #32 0x00007ffff2232333 in ?? () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Core.so.5 #33 0x00007fffee377166 in g_main_dispatch (context=0x6632f0) at /build/buildd/glib2.0-2.37.5/./glib/gmain.c:3065 #34 g_main_context_dispatch (context=context@entry=0x6632f0) at /build/buildd/glib2.0-2.37.5/./glib/gmain.c:3641 #35 0x00007fffee3774b8 in g_main_context_iterate (context=context@entry=0x6632f0, block=block@entry=1, dispatch=dispatch@entry=1, self=<optimized out>) at /build/buildd/glib2.0-2.37.5/./glib/gmain.c:3712 #36 0x00007fffee37755c in g_main_context_iteration (context=0x6632f0, may_block=1) at /build/buildd/glib2.0-2.37.5/./glib/gmain.c:3773 #37 0x00007ffff22324bc in QEventDispatcherGlib::processEvents(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Core.so.5 #38 0x00007ffff21e9d3b in QEventLoop::exec(QFlags<QEventLoop::ProcessEventsFlag>) () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Core.so.5 #39 0x00007ffff21ed120 in QCoreApplication::exec() () from /usr/local/Trolltech/Qt5/Qt-5.0.0-r40/lib/libQt5Core.so.5 #40 0x0000000000421ba0 in launcherMain (app=...) at /home/reni/Data/REPOS/webkit_sec/Tools/QtTestBrowser/qttestbrowser.cpp:49 #41 0x0000000000423680 in main (argc=2, argv=0x7fffffffdc38) at /home/reni/Data/REPOS/webkit_sec/Tools/QtTestBrowser/qttestbrowser.cpp:318 919831 1 209356 rhodovan.u-szeged 2013-08-22 06:14:00 -0700 Created attachment 209356 Test case 919836 2 209359 rhodovan.u-szeged 2013-08-22 06:16:27 -0700 Created attachment 209359 Proposed patch 920147 3 rhodovan.u-szeged 2013-08-23 01:27:48 -0700 Committed r154476: <http://trac.webkit.org/changeset/154476> 209356 2013-08-22 06:14:00 -0700 2013-08-22 06:14:00 -0700 Test case lastCrash.html text/html 90 rhodovan.u-szeged PGh0bWw+CiAgIDxibG9ja3F1b3RlIHN0eWxlPSItd2Via2l0LWZsb3ctZnJvbTogdGhyZWFkOyIv PgogICA8dGFibGUvPgogICA8Zm9ybS8+CjwvaHRtbD4K 209359 2013-08-22 06:16:27 -0700 2013-08-22 09:28:40 -0700 Proposed patch 0001-fix-for-null-renderer.patch text/plain 3933 rhodovan.u-szeged RnJvbSA2NDNmOGEyNjFkMzJiNThkMzNjMjdmNjExZDNhMzA5MWUyODM2MzM5IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBSZW5hdGEgSG9kb3ZhbiA8cmVuaUB3ZWJraXQub3JnPgpEYXRl OiBUaHUsIDIyIEF1ZyAyMDEzIDE1OjEzOjI0ICswMjAwClN1YmplY3Q6IFtQQVRDSF0gZml4IGZv ciBudWxsIHJlbmRlcmVyCgotLS0KIExheW91dFRlc3RzL0NoYW5nZUxvZyAgICAgICAgICAgICAg ICAgICAgICAgICAgICAgIHwgICAxMiArKysrKysrKysrKysKIC4uLi9taXNzaW5nLXBhcmVudHJl bmRlcmVyLWNyYXNoLWV4cGVjdGVkLnR4dCAgICAgIHwgICAgMSArCiAuLi4vZmFzdC9mb3Jtcy9t aXNzaW5nLXBhcmVudHJlbmRlcmVyLWNyYXNoLmh0bWwgICB8ICAgMTIgKysrKysrKysrKysrCiBT b3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cgICAgICAgICAgICAgICAgICAgICAgICAgICB8ICAgMTUg KysrKysrKysrKysrKysrCiBTb3VyY2UvV2ViQ29yZS9odG1sL0hUTUxGb3JtRWxlbWVudC5jcHAg ICAgICAgICAgICB8ICAgIDQgKysrKwogNSBmaWxlcyBjaGFuZ2VkLCA0NCBpbnNlcnRpb25zKCsp CiBjcmVhdGUgbW9kZSAxMDA2NDQgTGF5b3V0VGVzdHMvZmFzdC9mb3Jtcy9taXNzaW5nLXBhcmVu dHJlbmRlcmVyLWNyYXNoLWV4cGVjdGVkLnR4dAogY3JlYXRlIG1vZGUgMTAwNjQ0IExheW91dFRl c3RzL2Zhc3QvZm9ybXMvbWlzc2luZy1wYXJlbnRyZW5kZXJlci1jcmFzaC5odG1sCgpkaWZmIC0t Z2l0IGEvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nIGIvTGF5b3V0VGVzdHMvQ2hhbmdlTG9nCmluZGV4 IDk3YzM1OWUuLmE1NTBlNDAgMTAwNjQ0Ci0tLSBhL0xheW91dFRlc3RzL0NoYW5nZUxvZworKysg Yi9MYXlvdXRUZXN0cy9DaGFuZ2VMb2cKQEAgLTEsNSArMSwxNyBAQAogMjAxMy0wOC0yMiAgUmVu YXRhIEhvZG92YW4gIDxyZW5pQHdlYmtpdC5vcmc+CiAKKyAgICAgICAgTWlzc2luZyBudWxsLWNo ZWNrIGluIEhUTUxGb3JtRWxlbWVudDo6cmVuZGVyZXJJc05lZWRlZCgpCisgICAgICAgIGh0dHBz Oi8vYnVncy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD0xMjAxNTkKKworICAgICAgICBSZXZp ZXdlZCBieSBOT0JPRFkgKE9PUFMhKS4KKworICAgICAgICBUZXN0IGZvciB0aGUgaGFuZGxpbmcg b2YgbnVsbCBwYXJlbnRSZWRlcmVyLgorCisgICAgICAgICogZmFzdC9mb3Jtcy9taXNzaW5nLXBh cmVudHJlbmRlcmVyLWNyYXNoLWV4cGVjdGVkLnR4dDogQWRkZWQuCisgICAgICAgICogZmFzdC9m b3Jtcy9taXNzaW5nLXBhcmVudHJlbmRlcmVyLWNyYXNoLmh0bWw6IEFkZGVkLgorCisyMDEzLTA4 LTIyICBSZW5hdGEgSG9kb3ZhbiAgPHJlbmlAd2Via2l0Lm9yZz4KKwogICAgICAgICBBU1NFUlRJ T04gRkFJTEVEOiBleHRyYWN0ZWRTdHlsZSBpbiBXZWJDb3JlOjpBcHBseVN0eWxlQ29tbWFuZDo6 cmVtb3ZlSW5saW5lU3R5bGVGcm9tRWxlbWVudAogICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0 Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTE5NjcyCiAKZGlmZiAtLWdpdCBhL0xheW91dFRlc3RzL2Zh c3QvZm9ybXMvbWlzc2luZy1wYXJlbnRyZW5kZXJlci1jcmFzaC1leHBlY3RlZC50eHQgYi9MYXlv dXRUZXN0cy9mYXN0L2Zvcm1zL21pc3NpbmctcGFyZW50cmVuZGVyZXItY3Jhc2gtZXhwZWN0ZWQu dHh0Cm5ldyBmaWxlIG1vZGUgMTAwNjQ0CmluZGV4IDAwMDAwMDAuLjJhZmEwYmYKLS0tIC9kZXYv bnVsbAorKysgYi9MYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL21pc3NpbmctcGFyZW50cmVuZGVyZXIt Y3Jhc2gtZXhwZWN0ZWQudHh0CkBAIC0wLDAgKzEgQEAKK1BBU1MuIFdlYktpdCBkaWRuJ3QgY3Jh c2guCmRpZmYgLS1naXQgYS9MYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL21pc3NpbmctcGFyZW50cmVu ZGVyZXItY3Jhc2guaHRtbCBiL0xheW91dFRlc3RzL2Zhc3QvZm9ybXMvbWlzc2luZy1wYXJlbnRy ZW5kZXJlci1jcmFzaC5odG1sCm5ldyBmaWxlIG1vZGUgMTAwNjQ0CmluZGV4IDAwMDAwMDAuLjdi MWY5NjUKLS0tIC9kZXYvbnVsbAorKysgYi9MYXlvdXRUZXN0cy9mYXN0L2Zvcm1zL21pc3Npbmct cGFyZW50cmVuZGVyZXItY3Jhc2guaHRtbApAQCAtMCwwICsxLDEyIEBACis8aHRtbD4KKyAgICA8 YmxvY2txdW90ZSBzdHlsZT0iLXdlYmtpdC1mbG93LWZyb206IHRocmVhZDsiLz4KKyAgICA8dGFi bGUvPgorICAgIDxmb3JtLz4KKworICAgIDxzY3JpcHQ+CisgICAgICAgIGlmICh3aW5kb3cudGVz dFJ1bm5lcikKKyAgICAgICAgICAgIHRlc3RSdW5uZXIuZHVtcEFzVGV4dCgpOworCisgICAgICAg IGRvY3VtZW50LmJvZHkuaW5uZXJIVE1MID0gIlBBU1MuIFdlYktpdCBkaWRuJ3QgY3Jhc2guIjsK KyAgICA8L3NjcmlwdD4KKzwvaHRtbD4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL0NoYW5n ZUxvZyBiL1NvdXJjZS9XZWJDb3JlL0NoYW5nZUxvZwppbmRleCAxNDc3ZTkyLi5iYzE2YTQyIDEw MDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKKysrIGIvU291cmNlL1dlYkNvcmUv Q2hhbmdlTG9nCkBAIC0xLDUgKzEsMjAgQEAKIDIwMTMtMDgtMjIgIFJlbmF0YSBIb2RvdmFuICA8 cmVuaUB3ZWJraXQub3JnPgogCisgICAgICAgIE1pc3NpbmcgbnVsbC1jaGVjayBpbiBIVE1MRm9y bUVsZW1lbnQ6OnJlbmRlcmVySXNOZWVkZWQoKQorICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0 Lm9yZy9zaG93X2J1Zy5jZ2k/aWQ9MTIwMTU5CisKKyAgICAgICAgUmV2aWV3ZWQgYnkgTk9CT0RZ IChPT1BTISkuCisKKyAgICAgICAgTnVsbC1jaGVjayBwYXJlbnRSZW5kZXJlciBpbiBIVE1MRm9y bUVsZW1lbnQ6OnJlbmRlcmVySXNOZWVkZWQoKQorICAgICAgICBhbmQgZWFybHkgcmV0dXJuLgor CisgICAgICAgIFRlc3Q6IGZhc3QvZm9ybXMvbWlzc2luZy1wYXJlbnRyZW5kZXJlci1jcmFzaC5o dG1sCisKKyAgICAgICAgKiBodG1sL0hUTUxGb3JtRWxlbWVudC5jcHA6CisgICAgICAgIChXZWJD b3JlOjpIVE1MRm9ybUVsZW1lbnQ6OnJlbmRlcmVySXNOZWVkZWQpOgorCisyMDEzLTA4LTIyICBS ZW5hdGEgSG9kb3ZhbiAgPHJlbmlAd2Via2l0Lm9yZz4KKwogICAgICAgICBBU1NFUlRJT04gRkFJ TEVEOiBleHRyYWN0ZWRTdHlsZSBpbiBXZWJDb3JlOjpBcHBseVN0eWxlQ29tbWFuZDo6cmVtb3Zl SW5saW5lU3R5bGVGcm9tRWxlbWVudAogICAgICAgICBodHRwczovL2J1Z3Mud2Via2l0Lm9yZy9z aG93X2J1Zy5jZ2k/aWQ9MTE5NjcyCiAKZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3JlL2h0bWwv SFRNTEZvcm1FbGVtZW50LmNwcCBiL1NvdXJjZS9XZWJDb3JlL2h0bWwvSFRNTEZvcm1FbGVtZW50 LmNwcAppbmRleCA0M2ViZDNjLi4wODBiNmEzIDEwMDY0NAotLS0gYS9Tb3VyY2UvV2ViQ29yZS9o dG1sL0hUTUxGb3JtRWxlbWVudC5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvaHRtbC9IVE1MRm9y bUVsZW1lbnQuY3BwCkBAIC0xMDEsNiArMTAxLDEwIEBAIGJvb2wgSFRNTEZvcm1FbGVtZW50Ojpy ZW5kZXJlcklzTmVlZGVkKGNvbnN0IFJlbmRlclN0eWxlJiBzdHlsZSkKIAogICAgIENvbnRhaW5l ck5vZGUqIG5vZGUgPSBwYXJlbnROb2RlKCk7CiAgICAgUmVuZGVyT2JqZWN0KiBwYXJlbnRSZW5k ZXJlciA9IG5vZGUtPnJlbmRlcmVyKCk7CisKKyAgICBpZiAoIXBhcmVudFJlbmRlcmVyKQorICAg ICAgICByZXR1cm4gZmFsc2U7CisKICAgICAvLyBGSVhNRTogU2hvdWxkbid0IHdlIGFsc28gY2hl Y2sgZm9yIHRhYmxlIGNhcHRpb24gKHNlZSB8Zm9ybUlzVGFibGVQYXJ0fCBiZWxvdykuCiAgICAg Ym9vbCBwYXJlbnRJc1RhYmxlRWxlbWVudFBhcnQgPSAocGFyZW50UmVuZGVyZXItPmlzVGFibGUo KSAmJiBpc0hUTUxUYWJsZUVsZW1lbnQobm9kZSkpCiAgICAgICAgIHx8IChwYXJlbnRSZW5kZXJl ci0+aXNUYWJsZVJvdygpICYmIG5vZGUtPmhhc1RhZ05hbWUodHJUYWcpKQotLSAKMS43LjkuNQoK