Closed
Bug 464197
Opened 17 years ago
Closed 16 years ago
[FIX] Crash [@ nsMediaDecoder::Progress]
Categories
(Core :: Audio/Video, defect)
Core
Audio/Video
Tracking
()
VERIFIED
FIXED
mozilla1.9.2a1
People
(Reporter: jruderman, Assigned: smaug)
References
Details
(Keywords: crash, testcase, verified1.9.1, Whiteboard: [sg:critical])
Crash Data
Attachments
(2 files)
|
382 bytes,
text/html
|
Details | |
|
692 bytes,
patch
|
roc
:
review+
roc
:
superreview+
|
Details | Diff | Splinter Review |
Example stack #1:
0 nsINodeInfo::GetDocument() const + 9 (nsINodeInfo.h:291)
1 nsINode::GetOwnerDoc() const + 28 (nsINode.h:268)
2 nsHTMLMediaElement::DispatchProgressEvent(nsAString_internal const&) + 64 (nsHTMLMediaElement.cpp:813)
3 nsMediaDecoder::Progress() + 73 (nsMediaDecoder.cpp:138)
4 ProgressCallback(nsITimer*, void*) + 30
5 nsTimerImpl::Fire() + 868 (nsTimerImpl.cpp:421)
...
Example stack #2:
0 0 + 3968030437
1 nsMediaDecoder::Progress() + 73 (nsMediaDecoder.cpp:138)
2 ProgressCallback(nsITimer*, void*) + 30
3 nsTimerImpl::Fire() + 868 (nsTimerImpl.cpp:421)
...
Note that nsMediaDecoder::Progress appears in both stacks.
Flags: blocking1.9.1?
|
Reporter | |
Updated•17 years ago
|
Whiteboard: [sg:critical]
|
Assignee | |
Comment 1•17 years ago
|
||
Assignee: nobody → Olli.Pettay
Status: NEW → ASSIGNED
Attachment #347585 -
Flags: superreview?(roc)
Attachment #347585 -
Flags: review?(roc)
Attachment #347585 -
Flags: superreview?(roc)
Attachment #347585 -
Flags: superreview+
Attachment #347585 -
Flags: review?(roc)
Attachment #347585 -
Flags: review+
|
|
Assignee | |
Updated•17 years ago
|
Keywords: checkin-needed
Summary: Crash [@ nsMediaDecoder::Progress] → [FIX] Crash [@ nsMediaDecoder::Progress]
Flags: blocking1.9.1? → blocking1.9.1+
|
|
Assignee | |
Updated•16 years ago
|
Attachment #347585 -
Flags: approval1.9.1?
You don't need approval, since this is a blocker now.
Whiteboard: [sg:critical] → [sg:critical][needs landing]
|
|
Assignee | |
Comment 3•16 years ago
|
||
Comment on attachment 347585 [details] [diff] [review]
remember to shutdown decoder before nulling the variable
Ok.
Attachment #347585 -
Flags: approval1.9.1?
|
|
Assignee | |
Updated•16 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [sg:critical][needs landing] → [sg:critical]
|
||
Updated•16 years ago
|
Keywords: fixed1.9.1
|
||
Updated•16 years ago
|
Keywords: checkin-needed
|
||
Comment 4•16 years ago
|
||
Verified fix on Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.2a1pre) Gecko/20090204 Minefield/3.2a1pre
and Mozilla/5.0 (Macintosh; U; Intel Mac OS X 10.5; en-US; rv:1.9.1b3pre) Gecko/20090204 Shiretoko/3.1b3pre Ubiquity/0.1.5
Status: RESOLVED → VERIFIED
Keywords: fixed1.9.1 → verified1.9.1
|
||
Updated•16 years ago
|
Flags: in-testsuite?
OS: Mac OS X → All
Hardware: x86 → All
Target Milestone: --- → mozilla1.9.2a1
|
||
Updated•16 years ago
|
Flags: wanted1.9.0.x-
Flags: wanted1.8.1.x-
|
|
||
Updated•16 years ago
|
Group: core-security
|
|
Reporter | |
Comment 5•16 years ago
|
||
Flags: in-testsuite? → in-testsuite+
|
||
Updated•14 years ago
|
Crash Signature: [@ nsMediaDecoder::Progress]
You need to log in
before you can comment on or make changes to this bug.
Description
•