Closed Bug 1373914 Opened 5 years ago Closed 5 years ago

Crash in mozilla::MediaShutdownManager::Instance


(Core :: Audio/Video: Playback, defect, P1)

55 Branch



Tracking Status
firefox-esr52 --- unaffected
firefox54 --- unaffected
firefox55 --- wontfix
firefox56 --- fixed


(Reporter: philipp, Assigned: jwwang)



(Keywords: crash, regression)

Crash Data


(3 files)

This bug was filed from the Socorro interface and is 
report bp-fd069750-5034-4277-adb2-539a70170617.
Crashing Thread (0)
Frame 	Module 	Signature 	Source
0 	xul.dll 	mozilla::MediaShutdownManager::Instance() 	dom/media/MediaShutdownManager.cpp:42
1 	xul.dll 	mozilla::MediaDecoder::Load(nsIStreamListener**) 	dom/media/MediaDecoder.cpp:626
2 	xul.dll 	mozilla::dom::HTMLMediaElement::FinishDecoderSetup(mozilla::MediaDecoder*, mozilla::MediaResource*, nsIStreamListener**) 	dom/html/HTMLMediaElement.cpp:4714
3 	xul.dll 	mozilla::dom::HTMLMediaElement::InitializeDecoderForChannel(nsIChannel*, nsIStreamListener**) 	dom/html/HTMLMediaElement.cpp:4688
4 	xul.dll 	mozilla::dom::HTMLMediaElement::MediaLoadListener::OnStartRequest(nsIRequest*, nsISupports*) 	dom/html/HTMLMediaElement.cpp:541
5 	xul.dll 	nsJARChannel::OnStartRequest(nsIRequest*, nsISupports*) 	modules/libjar/nsJARChannel.cpp:1022
6 	xul.dll 	nsInputStreamPump::OnStateStart() 	netwerk/base/nsInputStreamPump.cpp:539
7 	xul.dll 	nsInputStreamPump::OnInputStreamReady(nsIAsyncInputStream*) 	netwerk/base/nsInputStreamPump.cpp:441
8 	xul.dll 	nsInputStreamReadyEvent::Run() 	xpcom/io/nsStreamUtils.cpp:96
9 	xul.dll 	nsThread::ProcessNextEvent(bool, bool*) 	xpcom/threads/nsThread.cpp:1418
10 	xul.dll 	XPTC__InvokebyIndex 	xpcom/reflect/xptcall/md/win32/xptcinvoke_asm_x86_64.asm:97
11 		@0x0 	
12 	xul.dll 	XPCWrappedNative::CallMethod(XPCCallContext&, XPCWrappedNative::CallMode) 	js/xpconnect/src/XPCWrappedNative.cpp:1282
13 	xul.dll 	XPC_WN_CallMethod(JSContext*, unsigned int, JS::Value*) 	js/xpconnect/src/XPCWrappedNativeJSOps.cpp:982
14 		@0x34e82779064 	

crash reports with this signature appear to be somewhat frequent in early data from the rollout of the 55.0b devedition. they are occurring with MOZ_RELEASE_ASSERT(sInstance) that got added in bug 1336345.
Flags: needinfo?(jwwang)
Closed: 5 years ago
Flags: needinfo?(jwwang)
Resolution: --- → DUPLICATE
Duplicate of bug: 1366930
the patch for bug 1366930 landed in m-c on 2017-05-25, but the crash signature is still present in 55.0b beta now.
Flags: needinfo?(jwwang)
Sorry, this is not a dup.
Flags: needinfo?(jwwang)
Resolution: DUPLICATE → ---
Assignee: nobody → jwwang
Priority: -- → P1
I can't see how sInstance is null.

sInstance is reset in RemoveBlocker() which happens only after XPCOM shutdown begins. However, seeing Uptime is 5 sec, I don't think this is a shutdown crash.

I will add more logs for debugging.
Attachment #8879421 - Flags: review?(gsquelart)
Attachment #8879422 - Flags: review?(gsquelart)
Attachment #8879423 - Flags: review?(gsquelart)
Comment on attachment 8879421 [details]
Bug 1373914. P1 - add a static to track the life cycle of MediaShutdownManager.
Attachment #8879421 - Flags: review?(gsquelart) → review+
Comment on attachment 8879422 [details]
Bug 1373914. P2 - remove unused members.
Attachment #8879422 - Flags: review?(gsquelart) → review+
Comment on attachment 8879423 [details]
Bug 1373914. P3 - dump sInitPhase when sInstance is null in Instance() to help debugging.
Attachment #8879423 - Flags: review?(gsquelart) → review+
Keywords: leave-open
Pushed by
P1 - add a static to track the life cycle of MediaShutdownManager. r=gerald
P2 - remove unused members. r=gerald
P3 - dump sInitPhase when sInstance is null in Instance() to help debugging. r=gerald
jw, have we learned anything since the patches have landed?
Flags: needinfo?(jwwang)
No. There are no new reports since the patches have landed.
Flags: needinfo?(jwwang)
Feels like we fixed it.
Closed: 5 years ago5 years ago
Resolution: --- → FIXED
Not sure if this is really fixed or by what - since there are no recent crashes for 55 either.
Low volume, fix-optional for 55.
Did you want to nominate this for Beta or let it ride the 56 train?
Flags: needinfo?(jwwang)
No, I will let it ride the train for the patches just dump debugging info when crash happens. Maybe the code changes make the crash less frequent.
Flags: needinfo?(jwwang)
Keywords: leave-open
Target Milestone: --- → mozilla56
You need to log in before you can comment on or make changes to this bug.