Bug 190578 – [EME][GTK] media/encrypted-media/mock-MediaKeySystemAccess.html crashes in CDM::createInstance

• Status: RESOLVED
• Resolution: DUPLICATE
• Component: WebKitGTK

[Charlie Turner]

Crash log,

Thread 1 (Thread 0x7f6a751e9580 (LWP 8232)):
#0  0x00007f6a709e2d6c in _ZN7WebCore3CDM14createInstanceEv (this=0x7f69e64cdc90) at ../../Source/WebCore/Modules/encryptedmedia/CDM.cpp:598
#1  0x00007f6a70689252 in WebCore::MediaKeySystemAccess::<lambda()>::operator()(void) (__closure=0x7f69e648b260) at ../../Source/WebCore/Modules/encryptedmedia/MediaKeySystemAccess.cpp:76
#2  0x00007f6a70689d2e in WTF::Function<void()>::CallableWrapper<WebCore::MediaKeySystemAccess::createMediaKeys(WTF::Ref<WebCore::DeferredPromise>&&)::<lambda()> >::call(void) (this=0x7f69e648b258) at DerivedSources/ForwardingHeaders/wtf/Function.h:101
#3  0x00007f6a6d61a518 in _ZNK3WTF8FunctionIFvvEEclEv (this=0x7f69e648b280) at DerivedSources/ForwardingHeaders/wtf/Function.h:56
#4  0x00007f6a6ec938f5 in _ZZN7WebCore16GenericTaskQueueINS_5TimerEE11enqueueTaskEON3WTF8FunctionIFvvEEEENKUlvE_clEv (__closure=0x7f69e648b278) at ../../Source/WebCore/platform/GenericTaskQueue.h:94
#5  0x00007f6a6ec96746 in _ZN3WTF8FunctionIFvvEE15CallableWrapperIZN7WebCore16GenericTaskQueueINS4_5TimerEE11enqueueTaskEOS2_EUlvE_E4callEv (this=0x7f69e648b270) at DerivedSources/ForwardingHeaders/wtf/Function.h:101
#6  0x00007f6a6d61a518 in _ZNK3WTF8FunctionIFvvEEclEv (this=0x7ffd2690ed30) at DerivedSources/ForwardingHeaders/wtf/Function.h:56
#7  0x00007f6a6fbb952a in _ZN7WebCore14TaskDispatcherINS_5TimerEE15dispatchOneTaskEv (this=0x7f69e647d0e8) at ../../Source/WebCore/platform/GenericTaskQueue.cpp:80
#8  0x00007f6a6fbb93f5 in _ZN7WebCore14TaskDispatcherINS_5TimerEE16sharedTimerFiredEv () at ../../Source/WebCore/platform/GenericTaskQueue.cpp:65
#9  0x00007f6a6fbb924f in WebCore::TaskDispatcher<WebCore::Timer>::<lambda()>::operator()(void) const (__closure=0x7f6a4f62a7c8) at ../../Source/WebCore/platform/GenericTaskQueue.cpp:49
#10 0x00007f6a6fbb9dd6 in WTF::Function<void()>::CallableWrapper<WebCore::TaskDispatcher<WebCore::Timer>::sharedTimer()::<lambda()> >::call(void) (this=0x7f6a4f62a7c0) at DerivedSources/ForwardingHeaders/wtf/Function.h:101
#11 0x00007f6a6d61a518 in _ZNK3WTF8FunctionIFvvEEclEv (this=0x7f6a7501ddf8 <_ZZN7WebCore14TaskDispatcherINS_5TimerEE11sharedTimerEvE5timer+56>) at DerivedSources/ForwardingHeaders/wtf/Function.h:56
#12 0x00007f6a6d61721a in _ZN7WebCore5Timer5firedEv (this=0x7f6a7501ddc0 <_ZZN7WebCore14TaskDispatcherINS_5TimerEE11sharedTimerEvE5timer>) at ../../Source/WebCore/platform/Timer.h:131
#13 0x00007f6a6fbec0c5 in _ZN7WebCore12ThreadTimers24sharedTimerFiredInternalEv (this=0x7f6a4f682438) at ../../Source/WebCore/platform/ThreadTimers.cpp:117
#14 0x00007f6a6fbebb9d in WebCore::ThreadTimers::<lambda()>::operator()(void) const (__closure=0x7f6a4f6fa398) at ../../Source/WebCore/platform/ThreadTimers.cpp:69
#15 0x00007f6a6fbf7824 in WTF::Function<void()>::CallableWrapper<WebCore::ThreadTimers::setSharedTimer(WebCore::SharedTimer*)::<lambda()> >::call(void) (this=0x7f6a4f6fa390) at DerivedSources/ForwardingHeaders/wtf/Function.h:101
#16 0x00007f6a6d61a518 in _ZNK3WTF8FunctionIFvvEEclEv (this=0x7f6a7501dfa8 <_ZZN7WebCore21MainThreadSharedTimer9singletonEvE8instance+8>) at DerivedSources/ForwardingHeaders/wtf/Function.h:56
#17 0x00007f6a6fbc8c87 in _ZN7WebCore21MainThreadSharedTimer5firedEv (this=0x7f6a7501dfa0 <_ZZN7WebCore21MainThreadSharedTimer9singletonEvE8instance>) at ../../Source/WebCore/platform/MainThreadSharedTimer.cpp:54
#18 0x00007f6a6fbcd09a in _ZN3WTF7RunLoop5TimerIN7WebCore21MainThreadSharedTimerEE5firedEv (this=0x7f6a7501dfb0 <_ZZN7WebCore21MainThreadSharedTimer9singletonEvE8instance+16>) at DerivedSources/ForwardingHeaders/wtf/RunLoop.h:149
#19 0x00007f6a613298e5 in WTF::RunLoop::TimerBase::<lambda(gpointer)>::operator()(gpointer) const (__closure=0x0, userData=0x7f6a7501dfb0 <_ZZN7WebCore21MainThreadSharedTimer9singletonEvE8instance+16>) at ../../Source/WTF/wtf/glib/RunLoopGLib.cpp:170
#20 0x00007f6a6132993d in WTF::RunLoop::TimerBase::<lambda(gpointer)>::_FUN(gpointer) () at ../../Source/WTF/wtf/glib/RunLoopGLib.cpp:176
#21 0x00007f6a61328fb2 in WTF::<lambda(GSource*, GSourceFunc, gpointer)>::operator()(GSource *, GSourceFunc, gpointer) const (__closure=0x0, source=0x55d4aec4b3f0, callback=0x7f6a61329920 <WTF::RunLoop::TimerBase::<lambda(gpointer)>::_FUN(gpointer)>, userData=0x7f6a7501dfb0 <_ZZN7WebCore21MainThreadSharedTimer9singletonEvE8instance+16>) at ../../Source/WTF/wtf/glib/RunLoopGLib.cpp:45
#22 0x00007f6a61328fe1 in WTF::<lambda(GSource*, GSourceFunc, gpointer)>::_FUN(GSource *, GSourceFunc, gpointer) () at ../../Source/WTF/wtf/glib/RunLoopGLib.cpp:46
#23 0x00007f6a62f80d35 in g_main_dispatch () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gmain.c:3148
#24 g_main_context_dispatch () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gmain.c:3813
#25 0x00007f6a62f81100 in g_main_context_iterate () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gmain.c:3886
#26 0x00007f6a62f81412 in g_main_loop_run () at /home/charlie/WebKit/WebKitWPE/WebKitBuild/DependenciesGTK/Source/glib-2.54.2/glib/gmain.c:4082
#27 0x00007f6a61329521 in _ZN3WTF7RunLoop3runEv () at ../../Source/WTF/wtf/glib/RunLoopGLib.cpp:96
#28 0x00007f6a6e033a6d in _ZN6WebKit16ChildProcessMainINS_10WebProcessENS_14WebProcessMainEEEiiPPc (argc=3, argv=0x7ffd2690f2f8) at ../../Source/WebKit/Shared/unix/ChildProcessMain.h:61
#29 0x00007f6a6e0322ad in WebKit::WebProcessMainUnix (argc=3, argv=0x7ffd2690f2f8) at ../../Source/WebKit/WebProcess/gtk/WebProcessMainGtk.cpp:67
#30 0x000055d4acfe2a45 in main (argc=3, argv=0x7ffd2690f2f8) at ../../Source/WebKit/WebProcess/EntryPoint/unix/WebProcessMain.cpp:52

[Charlie Turner]

Created attachment 352318 [details]
Patch

[Xabier Rodríguez Calvar]

Comment on attachment 352318 [details]
Patch

Shouldn't this be skipped instead?

[Charlie Turner]

Perhaps, I'm not sure what the purpose of the mock EME classes are, perhaps they're not that useful to use, just assumed we'd want them to work on our ports at some point.

[WebKit Commit Bot]

Comment on attachment 352318 [details]
Patch

Clearing flags on attachment: 352318

Committed r237216: <https://trac.webkit.org/changeset/237216>

[WebKit Commit Bot]

All reviewed patches have been landed.  Closing bug.

[Charlie Turner]

Not sure what I was thinking here... This looks like a simple fix which I'll upload in the original bug.

*** This bug has been marked as a duplicate of bug 181225 ***