Closed Bug 1373914 Opened 8 years ago Closed 8 years ago

Crash in mozilla::MediaShutdownManager::Instance

Categories

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

Product:
Core
Component:
Audio/Video: Playback
Version:
55 Branch
Platform:
All
Windows
Type:
defect

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla56
Tracking Flags:
Tracking Status
firefox-esr52 --- unaffected
firefox54 --- unaffected
firefox55 --- wontfix
firefox56 --- fixed

People

(Reporter: philipp, Assigned: jwwang)

References

Details

(Keywords: crash, regression)

Crash Data

Attachments

(3 files)

Bug 1373914. P1 - add a static to track the life cycle of MediaShutdownManager.
59 bytes, text/x-review-board-request
mozbugz
: review+
Details
Bug 1373914. P2 - remove unused members.
59 bytes, text/x-review-board-request
mozbugz
: review+
Details
Bug 1373914. P3 - dump sInitPhase when sInstance is null in Instance() to help debugging.
59 bytes, text/x-review-board-request
mozbugz
: review+
Details
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)
Status: NEW → RESOLVED
Closed: 8 years ago
Flags: needinfo?(jwwang)
Resolution: --- → DUPLICATE
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.
Status: RESOLVED → REOPENED
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. https://reviewboard.mozilla.org/r/150746/#review155530
Attachment #8879421 - Flags: review?(gsquelart) → review+
Comment on attachment 8879423 [details] Bug 1373914. P3 - dump sInitPhase when sInstance is null in Instance() to help debugging. https://reviewboard.mozilla.org/r/150750/#review155534
Attachment #8879423 - Flags: review?(gsquelart) → review+
Thanks!
Keywords: leave-open
Pushed by jwwang@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/34bb2c8317f6 P1 - add a static to track the life cycle of MediaShutdownManager. r=gerald https://hg.mozilla.org/integration/autoland/rev/dfb864562afe P2 - remove unused members. r=gerald https://hg.mozilla.org/integration/autoland/rev/22486af244c1 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.
Status: REOPENED → RESOLVED
Closed: 8 years ago → 8 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.
status-firefox55: affected → fix-optional
Did you want to nominate this for Beta or let it ride the 56 train?
status-firefox54: --- → unaffected
status-firefox-esr52: --- → unaffected
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.

Attachment

General

Creator:
Created:
Updated:
Size: