Bug 190578 - [EME][GTK] media/encrypted-media/mock-MediaKeySystemAccess.html crashes in CDM::createInstance
Summary: [EME][GTK] media/encrypted-media/mock-MediaKeySystemAccess.html crashes in CD...
Status: RESOLVED DUPLICATE of bug 181225
Alias: None
Product: WebKit
Classification: Unclassified
Component: WebKitGTK (show other bugs)
Version: WebKit Nightly Build
Hardware: Unspecified Unspecified
: P2 Normal
Assignee: Charlie Turner
URL:
Keywords:
Depends on: 99024 190577
Blocks:
  Show dependency treegraph
 
Reported: 2018-10-15 07:07 PDT by Charlie Turner
Modified: 2020-05-04 07:03 PDT (History)
3 users (show)

See Also:


Attachments
Patch (1.54 KB, patch)
2018-10-15 07:27 PDT, Charlie Turner
no flags Details | Formatted Diff | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Charlie Turner 2018-10-15 07:07:54 PDT
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
Comment 1 Charlie Turner 2018-10-15 07:27:56 PDT
Created attachment 352318 [details]
Patch
Comment 2 Xabier Rodríguez Calvar 2018-10-15 22:45:39 PDT
Comment on attachment 352318 [details]
Patch

Shouldn't this be skipped instead?
Comment 3 Charlie Turner 2018-10-16 08:16:59 PDT
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.
Comment 4 WebKit Commit Bot 2018-10-17 04:00:21 PDT
Comment on attachment 352318 [details]
Patch

Clearing flags on attachment: 352318

Committed r237216: <https://trac.webkit.org/changeset/237216>
Comment 5 WebKit Commit Bot 2018-10-17 04:00:22 PDT
All reviewed patches have been landed.  Closing bug.
Comment 6 Charlie Turner 2020-05-04 07:03:56 PDT
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 ***