Crash in mozilla::MediaCodecDataDecoder::DecoderLoop

RESOLVED FIXED

Status

()

Firefox for Android
Audio/Video
--
critical
RESOLVED FIXED
a year ago
4 months ago

People

(Reporter: calixte, Unassigned)

Tracking

({crash})

Firefox Tracking Flags

(Not tracked)

Details

(crash signature)

(Reporter)

Description

a year ago
This bug was filed from the Socorro interface and is 
report bp-7d2bad9d-99c3-4804-8b33-439182170125.
=============================================================

There were 2565 crashes in release 50.1.0 and 877 in 50.0.2.
Jean-yves, does it ring a bell?
 0 	libxul.so	mozilla::MediaCodecDataDecoder::DecoderLoop	dom/media/platforms/android/AndroidDecoderModule.cpp:615
1 	libxul.so	mozilla::detail::RunnableMethodImpl<void (mozilla::MediaCodecDataDecoder::*)(), true, false>::Run	xpcom/glue/nsThreadUtils.h:729
2 	libxul.so	nsThread::ProcessNextEvent	xpcom/threads/nsThread.cpp:1076
3 	libxul.so	NS_ProcessNextEvent	xpcom/glue/nsThreadUtils.cpp:290
4 	libxul.so	mozilla::ipc::MessagePumpForNonMainThreads::Run	ipc/glue/MessagePump.cpp:354
5 	libxul.so	MessageLoop::Run	ipc/chromium/src/base/message_loop.cc:225
6 	libxul.so	nsThread::ThreadFunc	xpcom/threads/nsThread.cpp:469
7 	libnss3.so	_pt_root	nsprpub/pr/src/pthreads/ptthread.c:216
Flags: needinfo?(jyavenard)
:jhlin know about Android more than I do
Flags: needinfo?(jyavenard) → needinfo?(jolin)
72% of the urls are a porn site xvideos dot com.
FWIW, Jhlin and many Taipei team members will be off from Jan. 27 to Feb. 1 for Chinese new year. 
If anyone is interested in this bug, please feel free to check it.
Read the code and couldn't tell how the non-empty queue assertion would fail.
Esawin, would you by any chance happen to know what the problem could be?
Flags: needinfo?(jolin) → needinfo?(esawin)
(In reply to John Lin [:jolin][:jhlin] from comment #5)
> Read the code and couldn't tell how the non-empty queue assertion would fail.
> Esawin, would you by any chance happen to know what the problem could be?

That's practically impossible unless we have multiple decoder loops running on the same decoder instance.
Flags: needinfo?(esawin)
MediaCodecDataDecoder has been deprecated since 54 in bug 1340096. Close this one as fixed.
Status: NEW → RESOLVED
Last Resolved: 4 months ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.