81777 2012-03-21 07:22:35 -0700 [GTK] ASSERT bug in WebAudio (AudioFileReader) 2012-03-22 08:31:07 -0700 1 1 1 Unclassified WebKit WebKitGTK 528+ (Nightly build) PC Linux RESOLVED FIXED P2 Normal --- 0 gouache95 pnormand donggwan.kim dw.im gouache95 pnormand s.choi oldest_to_newest 584114 0 gouache95 2012-03-21 07:22:35 -0700 there is ASSERT bug in <decodeAudioForBusCreation() "Source/WebCore/platform/audio/gstreamer/AudioFileReaderGStreamer.cpp:283">. the code is below. GRefPtr<GstBus> bus = adoptGRef(gst_pipeline_get_bus(GST_PIPELINE(m_pipeline))); gst_pipeline_get_bus() returns FLOATING GstBus object. i think it's sane. but in adoptGRef, ASSERT(!GST_OBJECT_IS_FLOATING(GST_OBJECT(ptr))) then always i meet the segfault. [O/S] Linux 3.2.0-19-generic #30-Ubuntu SMP Fri Mar 16 16:27:15 UTC 2012 x86_64 x86_64 x86_64 GNU/Linux [Build Option] '--gtk --debug --web-audio --fullscreen-api --no-svg' [command] GtkLauncher --enable-webaudio=true --enable-webgl=true http://chromium.googlecode.com/svn/trunk/samples/audio/shiny-drum-machine.html [Result] ASSERTION FAILED: !GST_OBJECT_IS_FLOATING(GST_OBJECT(ptr)) ../../Source/WebCore/platform/graphics/gstreamer/GRefPtrGStreamer.cpp(130) : WTF::GRefPtr<T> WTF::adoptGRef(T*) [with T = _GstBus] 1 0x7fd8d28aff69 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x1c6cf69) [0x7fd8d28aff69] 2 0x7fd8d24ddcf6 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x189acf6) [0x7fd8d24ddcf6] 3 0x7fd8d24dd3c7 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x189a3c7) [0x7fd8d24dd3c7] 4 0x7fd8cfb6e53b /o/WebKit/WebKitBuild/Dependencies/Root/lib64/libglib-2.0.so.0(+0x4753b) [0x7fd8cfb6e53b] 5 0x7fd8cfb6d8fa /o/WebKit/WebKitBuild/Dependencies/Root/lib64/libglib-2.0.so.0(g_main_context_dispatch+0x16a) [0x7fd8cfb6d8fa] 6 0x7fd8cfb6dcc0 /o/WebKit/WebKitBuild/Dependencies/Root/lib64/libglib-2.0.so.0(+0x46cc0) [0x7fd8cfb6dcc0] 7 0x7fd8cfb6e0ba /o/WebKit/WebKitBuild/Dependencies/Root/lib64/libglib-2.0.so.0(g_main_loop_run+0x6a) [0x7fd8cfb6e0ba] 8 0x7fd8d24de0c4 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x189b0c4) [0x7fd8d24de0c4] 9 0x7fd8d24de447 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x189b447) [0x7fd8d24de447] 10 0x7fd8d24e0950 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x189d950) [0x7fd8d24e0950] 11 0x7fd8d24cf718 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x188c718) [0x7fd8d24cf718] 12 0x7fd8d24cfbfc /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x188cbfc) [0x7fd8d24cfbfc] 13 0x7fd8d24d00ce /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x188d0ce) [0x7fd8d24d00ce] 14 0x7fd8d24cdeca /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x188aeca) [0x7fd8d24cdeca] 15 0x7fd8d24cde04 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x188ae04) [0x7fd8d24cde04] 16 0x7fd8d24cf1e1 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x188c1e1) [0x7fd8d24cf1e1] 17 0x7fd8d24cf160 /o/WebKit/WebKitBuild/Debug/.libs/libwebkitgtk-3.0.so.0(+0x188c160) [0x7fd8d24cf160] 18 0x7fd8cf0029fd /o/WebKit/WebKitBuild/Debug/.libs/libjavascriptcoregtk-3.0.so.0(+0x68d9fd) [0x7fd8cf0029fd] 19 0x7fd8cf003063 /o/WebKit/WebKitBuild/Debug/.libs/libjavascriptcoregtk-3.0.so.0(+0x68e063) [0x7fd8cf003063] 20 0x7fd8cf911e9a /lib/x86_64-linux-gnu/libpthread.so.0(+0x7e9a) [0x7fd8cf911e9a] 21 0x7fd8cf63f74d /lib/x86_64-linux-gnu/libc.so.6(clone+0x6d) [0x7fd8cf63f74d] 584172 1 pnormand 2012-03-21 08:51:03 -0700 Hum, yeah I indeed didn't test a Debug build before landing the nested-loop-support patch of the AudioFileReader. I'll look into this issue, thanks for reporting! 585319 2 133270 pnormand 2012-03-22 08:26:03 -0700 Created attachment 133270 Patch 585322 3 pnormand 2012-03-22 08:31:07 -0700 Committed r111706: <http://trac.webkit.org/changeset/111706> 133270 2012-03-22 08:26:03 -0700 2012-03-22 08:27:56 -0700 Patch bug-81777-20120322162606.patch text/plain 2306 pnormand U3VidmVyc2lvbiBSZXZpc2lvbjogMTExNzA0CmRpZmYgLS1naXQgYS9Tb3VyY2UvV2ViQ29yZS9D aGFuZ2VMb2cgYi9Tb3VyY2UvV2ViQ29yZS9DaGFuZ2VMb2cKaW5kZXggYjlkNDMyYWI3MmJkMmE1 MDNkZjMyOWU0ZmU4NzFhMjg3MDE4ODJlNC4uODUyMDViY2MwZjBiZDQwNGExMTc4OTI1NjIzZDUw NzY5YTM2NmUxZSAxMDA2NDQKLS0tIGEvU291cmNlL1dlYkNvcmUvQ2hhbmdlTG9nCisrKyBiL1Nv dXJjZS9XZWJDb3JlL0NoYW5nZUxvZwpAQCAtMSwzICsxLDE1IEBACisyMDEyLTAzLTIyICBQaGls aXBwZSBOb3JtYW5kICA8cG5vcm1hbmRAaWdhbGlhLmNvbT4KKworICAgICAgICBbR1RLXSBBU1NF UlQgYnVnIGluIFdlYkF1ZGlvIChBdWRpb0ZpbGVSZWFkZXIpCisgICAgICAgIGh0dHBzOi8vYnVn cy53ZWJraXQub3JnL3Nob3dfYnVnLmNnaT9pZD04MTc3NworCisgICAgICAgIFJldmlld2VkIGJ5 IE5PQk9EWSAoT09QUyEpLgorCisgICAgICAgICogcGxhdGZvcm0vYXVkaW8vZ3N0cmVhbWVyL0F1 ZGlvRmlsZVJlYWRlckdTdHJlYW1lci5jcHA6CisgICAgICAgIChXZWJDb3JlOjpBdWRpb0ZpbGVS ZWFkZXI6OmRlY29kZUF1ZGlvRm9yQnVzQ3JlYXRpb24pOiBEb24ndCBzdGVhbAorICAgICAgICB0 aGUgZmxvYXQgR3N0QnVzIHJlZmVyZW5jZS4KKyAgICAgICAgKFdlYkNvcmU6OkF1ZGlvRmlsZVJl YWRlcjo6Y3JlYXRlQnVzKTogRGl0dG8uCisKIDIwMTItMDMtMjIgIElseWEgVGlraG9ub3Zza3kg IDxsb2lzbG9AY2hyb21pdW0ub3JnPgogCiAgICAgICAgIFVucmV2aWV3ZWQ6IFdlYiBJbnNwZWN0 b3I6IGZpeCBzeW50YXggZXJyb3IgaW4gdGV4dC4KZGlmZiAtLWdpdCBhL1NvdXJjZS9XZWJDb3Jl L3BsYXRmb3JtL2F1ZGlvL2dzdHJlYW1lci9BdWRpb0ZpbGVSZWFkZXJHU3RyZWFtZXIuY3BwIGIv U291cmNlL1dlYkNvcmUvcGxhdGZvcm0vYXVkaW8vZ3N0cmVhbWVyL0F1ZGlvRmlsZVJlYWRlckdT dHJlYW1lci5jcHAKaW5kZXggY2Y2MmVhODg1YmNlYjQxMDY3OWQxZGJhMTIxZmJhMmRkMzNmNDM4 Ni4uMWNmMGE5YmE2MmRlYTRhNzRhZmEwNzQyMjZjNzUyZWNkYTdhZjI1YyAxMDA2NDQKLS0tIGEv U291cmNlL1dlYkNvcmUvcGxhdGZvcm0vYXVkaW8vZ3N0cmVhbWVyL0F1ZGlvRmlsZVJlYWRlckdT dHJlYW1lci5jcHAKKysrIGIvU291cmNlL1dlYkNvcmUvcGxhdGZvcm0vYXVkaW8vZ3N0cmVhbWVy L0F1ZGlvRmlsZVJlYWRlckdTdHJlYW1lci5jcHAKQEAgLTI4MCw3ICsyODAsNyBAQCB2b2lkIEF1 ZGlvRmlsZVJlYWRlcjo6ZGVjb2RlQXVkaW9Gb3JCdXNDcmVhdGlvbigpCiAgICAgLy8gQSBkZWlu dGVybGVhdmUgZWxlbWVudCBpcyBhZGRlZCBvbmNlIGEgc3JjIHBhZCBiZWNvbWVzIGF2YWlsYWJs ZSBpbiBkZWNvZGViaW4uCiAgICAgbV9waXBlbGluZSA9IGdzdF9waXBlbGluZV9uZXcoMCk7CiAK LSAgICBHUmVmUHRyPEdzdEJ1cz4gYnVzID0gYWRvcHRHUmVmKGdzdF9waXBlbGluZV9nZXRfYnVz KEdTVF9QSVBFTElORShtX3BpcGVsaW5lKSkpOworICAgIEdSZWZQdHI8R3N0QnVzPiBidXMgPSBn c3RfcGlwZWxpbmVfZ2V0X2J1cyhHU1RfUElQRUxJTkUobV9waXBlbGluZSkpOwogICAgIGdzdF9i dXNfYWRkX3NpZ25hbF93YXRjaChidXMuZ2V0KCkpOwogICAgIGdfc2lnbmFsX2Nvbm5lY3QoYnVz LmdldCgpLCAibWVzc2FnZSIsIEdfQ0FMTEJBQ0sobWVzc2FnZUNhbGxiYWNrKSwgdGhpcyk7CiAK QEAgLTMzNCw3ICszMzQsNyBAQCBQYXNzT3duUHRyPEF1ZGlvQnVzPiBBdWRpb0ZpbGVSZWFkZXI6 OmNyZWF0ZUJ1cyhmbG9hdCBzYW1wbGVSYXRlLCBib29sIG1peFRvTW9ubwogICAgIGdfbWFpbl9s b29wX3J1bihtX2xvb3AuZ2V0KCkpOwogICAgIGdfbWFpbl9jb250ZXh0X3BvcF90aHJlYWRfZGVm YXVsdChjb250ZXh0LmdldCgpKTsKIAotICAgIEdSZWZQdHI8R3N0QnVzPiBidXMgPSBhZG9wdEdS ZWYoZ3N0X3BpcGVsaW5lX2dldF9idXMoR1NUX1BJUEVMSU5FKG1fcGlwZWxpbmUpKSk7CisgICAg R1JlZlB0cjxHc3RCdXM+IGJ1cyA9IGdzdF9waXBlbGluZV9nZXRfYnVzKEdTVF9QSVBFTElORSht X3BpcGVsaW5lKSk7CiAgICAgZ19zaWduYWxfaGFuZGxlcnNfZGlzY29ubmVjdF9ieV9mdW5jKGJ1 cy5nZXQoKSwgcmVpbnRlcnByZXRfY2FzdDxncG9pbnRlcj4obWVzc2FnZUNhbGxiYWNrKSwgdGhp cyk7CiAKICAgICBnX3NpZ25hbF9oYW5kbGVyc19kaXNjb25uZWN0X2J5X2Z1bmMobV9kZWNvZGVi aW4uZ2V0KCksIHJlaW50ZXJwcmV0X2Nhc3Q8Z3BvaW50ZXI+KG9uR1N0cmVhbWVyRGVjb2RlYmlu UGFkQWRkZWRDYWxsYmFjayksIHRoaXMpOwo=