Closed Bug 1281408 Opened 3 years ago Closed 3 years ago

Crash producing memory report [@ mozilla::dom::MediaStreamAudioSourceNode::SizeOfIncludingThis]

Categories

(Core :: Audio/Video: MediaStreamGraph, defect, P2, critical)

defect

Tracking

()

RESOLVED FIXED
mozilla50
Tracking Status
firefox50 --- fixed

People

(Reporter: jruderman, Assigned: padenot)

References

(Blocks 2 open bugs)

Details

(Keywords: assertion, crash, testcase)

Crash Data

Attachments

(4 files)

1. Load the testcase
2. Open about:memory
3. Click "Measure"

Crashes with a RefPtr null deref in debug builds, but does not crash in non-debug ASan builds.
Attached file stack
Rank: 21
Priority: -- → P2
Comment on attachment 8764313 [details]
Bug 1281408 - Nullcheck `mInputPort` in MediaStreamAudioSourceNode to prevent crashing when getting memory reports after the underlying MediaStream has been destroyed.

https://reviewboard.mozilla.org/r/60276/#review57098
Attachment #8764313 - Flags: review?(pehrson) → review+
Assignee: nobody → padenot
Pushed by paul@paul.cx:
https://hg.mozilla.org/integration/mozilla-inbound/rev/f35504e8fc9d
Nullcheck `mInputPort` in MediaStreamAudioSourceNode to prevent crashing when getting memory reports after the underlying MediaStream has been destroyed. r=pehrsons
https://hg.mozilla.org/mozilla-central/rev/f35504e8fc9d
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla50
You need to log in before you can comment on or make changes to this bug.