Closed Bug 1031532 Opened 7 years ago Closed 7 years ago
Crash with DASH video and MSE enabled
Omega reported in bug 1031121 that he was experimenting with the youtube dash videos, and when he turned on MSE, he experienced crash bp-4053477d-79b0-4acd-9d56-c063e2140627. This is a null-pointer exception at http://hg.mozilla.org/mozilla-central/annotate/be076357691c/content/media/mediasource/SourceBufferResource.h#l112 within SourceBufferResource::ResourceQueue::GetLength. Filing separately from that bug so we can track the crash and the frame-rate issues separately.
OS: Linux → Windows 7
Hardware: x86_64 → x86
See Also: → 1031121
Just hit this on Linux while seeking on a DASH-enabled YouTube video (Titanfall video from bug 1031121): bp-e4bd8ec7-e212-4ec4-95fe-f0e3c2140715
Looks like a threading issue, GetLength() is trying to iterate over items in an nsDeque and crashing inside a for-loop bounded on the nsDeque's GetSize() with a null pointer, suggesting the size of the nsDeque changed. Evict() removes items from the nsDeque, and is called mainly by SourceBuffer::Append* which is called on the main thread. Both of these crash signatures are on the decoder thread while waiting for data in a SourceBuffer.
Attachment #8455926 - Attachment is obsolete: true
Assignee: nobody → kinetik
Status: NEW → ASSIGNED
Make a bunch of ResourceQueue functions private and remove some that are unused.
Attachment #8455927 - Flags: review?(cajbir.bugzilla)
Attachment #8455927 - Flags: review?(cajbir.bugzilla) → review+
Attachment #8455928 - Flags: review?(cajbir.bugzilla) → review+
Status: ASSIGNED → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla33
You need to log in before you can comment on or make changes to this bug.