Closed
Bug 1270329
Opened 9 years ago
Closed 9 years ago
Use-after-forget in WebMTrackDemuxer::SkipToNextRandomAccessPoint
Categories
(Core :: Audio/Video: Playback, defect)
Core
Audio/Video: Playback
Tracking
()
RESOLVED
DUPLICATE
of bug 1261900
| Tracking | Status | |
|---|---|---|
| firefox49 | --- | affected |
People
(Reporter: jld, Assigned: jld)
References
Details
Attachments
(1 file)
|
1.46 KB,
patch
|
Details | Diff | Splinter Review |
It looks like this will be a null pointer crash if logging is enabled:
mSamples.PushFront(sample.forget());
followed by:
WEBM_DEBUG("next sample: %f (parsed: %d)",
media::TimeUnit::FromMicroseconds(sample->mTime).ToSeconds(),
parsed);
It looks like bug 1190472 regressed this. Found with the static analysis patches in bug 1186706 (after un-bit-rotting them to test bug 1191452).
|
Assignee | |
Comment 1•9 years ago
|
||
Attachment #8748973 -
Flags: review?(kinetik)
Component: Audio/Video → Audio/Video: Playback
|
||
Comment 2•9 years ago
|
||
I've fixed that one in the pending bug 1261900 (patch 9)
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → DUPLICATE
|
|
||
Updated•9 years ago
|
Attachment #8748973 -
Flags: review?(kinetik)
You need to log in
before you can comment on or make changes to this bug.
Description
•