1453199 - Assertion failure: surface->GetType() == SurfaceType::DATA (The snapshot SourceSurface from WebGL rendering contest is not DataSourceSurface.), at /home/worker/workspace/build/src/dom/canvas/ImageBitmap.cpp:878

Status: RESOLVED DUPLICATE of bug 1358053

(Core :: Graphics: CanvasWebGL, defect, P5)

Description

[Jason Kratzer [:jkratzer]]

Attachment: trigger.html

Testcase found while fuzzing esr52 rev -d61516b059c1.

rax = 0x0000000000625d50   rdx = 0x0000000000000000
rcx = 0x00007f20e06f70c9   rbx = 0x00007ffc87b3f8c0
rsi = 0x00007f20db741770   rdi = 0x00007f20db740540
rbp = 0x00007ffc87b3f930   rsp = 0x00007ffc87b3f860
r8 = 0x00007f20db741770    r9 = 0x00007f20e2cf5c00
r10 = 0x0000000000000043   r11 = 0x0000000000000000
r12 = 0x00007ffc87b3fb00   r13 = 0x00007ffc87b3f978
r14 = 0x00007ffc87b3f9c8   r15 = 0x00007ffc87b3f8a8
rip = 0x00007f20de30196a
OS|Linux|0.0.0 Linux 4.4.0-119-generic #143-Ubuntu SMP Mon Apr 2 16:08:24 UTC 2018 x86_64
CPU|amd64|family 6 model 78 stepping 3|1
GPU|||
Crash|SIGSEGV|0x0|0
0|0|libxul.so|mozilla::dom::ImageBitmap::CreateInternal|hg:hg.mozilla.org/releases/mozilla-esr52:dom/canvas/ImageBitmap.cpp:d61516b059c1|876|0x0
0|1|libxul.so|mozilla::dom::ImageBitmap::Create|hg:hg.mozilla.org/releases/mozilla-esr52:dom/canvas/ImageBitmap.cpp:d61516b059c1|1449|0x14
0|2|libxul.so|nsGlobalWindow::CreateImageBitmap|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsGlobalWindow.cpp:d61516b059c1|14997|0x5
0|3|libxul.so|mozilla::dom::WindowBinding::createImageBitmap|hg:hg.mozilla.org/releases/mozilla-esr52:obj-firefox/dom/bindings/WindowBinding.cpp:d61516b059c1|14814|0x2f
0|4|libxul.so|mozilla::dom::WindowBinding::createImageBitmap_promiseWrapper|hg:hg.mozilla.org/releases/mozilla-esr52:obj-firefox/dom/bindings/WindowBinding.cpp:d61516b059c1|14896|0x5
0|5|libxul.so|mozilla::dom::WindowBinding::genericPromiseReturningMethod|hg:hg.mozilla.org/releases/mozilla-esr52:obj-firefox/dom/bindings/WindowBinding.cpp:d61516b059c1|15195|0x9
0|6|libxul.so|js::CallJSNative|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/jscntxtinlines.h:d61516b059c1|239|0x9
0|7|libxul.so|js::InternalCallOrConstruct|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|459|0xf
0|8|libxul.so|Interpret|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|510|0xf
0|9|libxul.so|js::RunScript|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|405|0xb
0|10|libxul.so|js::InternalCallOrConstruct|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|477|0xb
0|11|libxul.so|js::Call|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/vm/Interpreter.cpp:d61516b059c1|523|0x5
0|12|libxul.so|JS::Call|hg:hg.mozilla.org/releases/mozilla-esr52:js/src/jsapi.cpp:d61516b059c1|2828|0x20
0|13|libxul.so|mozilla::dom::EventListener::HandleEvent|hg:hg.mozilla.org/releases/mozilla-esr52:obj-firefox/dom/bindings/EventListenerBinding.cpp:d61516b059c1|48|0xc
0|14|libxul.so|mozilla::dom::EventListener::HandleEvent<mozilla::dom::EventTarget*>|hg:hg.mozilla.org/releases/mozilla-esr52:obj-firefox/dist/include/mozilla/dom/EventListenerBinding.h:d61516b059c1|64|0x1c
0|15|libxul.so|mozilla::EventListenerManager::HandleEventSubType|hg:hg.mozilla.org/releases/mozilla-esr52:dom/events/EventListenerManager.cpp:d61516b059c1|1131|0x33
0|16|libxul.so|mozilla::EventListenerManager::HandleEventInternal|hg:hg.mozilla.org/releases/mozilla-esr52:dom/events/EventListenerManager.cpp:d61516b059c1|1287|0x23
0|17|libxul.so|mozilla::EventTargetChainItem::HandleEvent|hg:hg.mozilla.org/releases/mozilla-esr52:dom/events/EventListenerManager.h:d61516b059c1|375|0xa
0|18|libxul.so|mozilla::EventTargetChainItem::HandleEventTargetChain|hg:hg.mozilla.org/releases/mozilla-esr52:dom/events/EventDispatcher.cpp:d61516b059c1|380|0xf
0|19|libxul.so|mozilla::EventDispatcher::Dispatch|hg:hg.mozilla.org/releases/mozilla-esr52:dom/events/EventDispatcher.cpp:d61516b059c1|712|0x5
0|20|libxul.so|mozilla::EventDispatcher::DispatchDOMEvent|hg:hg.mozilla.org/releases/mozilla-esr52:dom/events/EventDispatcher.cpp:d61516b059c1|781|0x16
0|21|libxul.so|nsINode::DispatchEvent|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsINode.cpp:d61516b059c1|1309|0x5
0|22|libxul.so|nsContentUtils::DispatchEvent|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsContentUtils.cpp:d61516b059c1|4011|0x17
0|23|libxul.so|nsContentUtils::DispatchTrustedEvent|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsContentUtils.cpp:d61516b059c1|3980|0xf
0|24|libxul.so|nsDocument::DispatchContentLoadedEvents|hg:hg.mozilla.org/releases/mozilla-esr52:dom/base/nsDocument.cpp:d61516b059c1|4971|0x28
0|25|libxul.so|mozilla::detail::RunnableMethodImpl<void (nsDocument::*)(), true, false>::Run|hg:hg.mozilla.org/releases/mozilla-esr52:xpcom/glue/nsThreadUtils.h:d61516b059c1|810|0x5
0|26|libxul.so|nsThread::ProcessNextEvent|hg:hg.mozilla.org/releases/mozilla-esr52:xpcom/threads/nsThread.cpp:d61516b059c1|1216|0x11
0|27|libxul.so|NS_ProcessNextEvent|hg:hg.mozilla.org/releases/mozilla-esr52:xpcom/glue/nsThreadUtils.cpp:d61516b059c1|361|0xd
0|28|libxul.so|mozilla::ipc::MessagePump::Run|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/glue/MessagePump.cpp:d61516b059c1|96|0xa
0|29|libxul.so|MessageLoop::RunInternal|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/chromium/src/base/message_loop.cc:d61516b059c1|232|0x17
0|30|libxul.so|MessageLoop::Run|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/chromium/src/base/message_loop.cc:d61516b059c1|225|0x8
0|31|libxul.so|nsBaseAppShell::Run|hg:hg.mozilla.org/releases/mozilla-esr52:widget/nsBaseAppShell.cpp:d61516b059c1|156|0xd
0|32|libxul.so|XRE_RunAppShell|hg:hg.mozilla.org/releases/mozilla-esr52:toolkit/xre/nsEmbedFunctions.cpp:d61516b059c1|866|0x6
0|33|libxul.so|mozilla::ipc::MessagePumpForChildProcess::Run|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/glue/MessagePump.cpp:d61516b059c1|269|0x5
0|34|libxul.so|MessageLoop::RunInternal|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/chromium/src/base/message_loop.cc:d61516b059c1|232|0x17
0|35|libxul.so|MessageLoop::Run|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/chromium/src/base/message_loop.cc:d61516b059c1|225|0x8
0|36|libxul.so|XRE_InitChildProcess|hg:hg.mozilla.org/releases/mozilla-esr52:toolkit/xre/nsEmbedFunctions.cpp:d61516b059c1|698|0xf
0|37|plugin-container|content_process_main|hg:hg.mozilla.org/releases/mozilla-esr52:ipc/contentproc/plugin-container.cpp:d61516b059c1|197|0xe
0|38|libc-2.23.so||||0x20830
0|39|plugin-container|MOZ_ReportAssertionFailure|hg:hg.mozilla.org/releases/mozilla-esr52:mfbt/Assertions.h:d61516b059c1|170|0x5

Comment 1

[Andrew Osmond [:aosmond] (he/him)]

I think we can consider this a duplicate of bug 1358053, which removed the assert in question from 55+. The assert itself was incorrect, and there were no additional changes required for non-debug builds.