"ERROR: Potential deadlock detected" with mozCaptureStreamUntilEnded

RESOLVED FIXED in mozilla15

Status

()

Core
Audio/Video
--
critical
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: Jesse Ruderman, Assigned: roc)

Tracking

(Blocks: 1 bug, {assertion, testcase})

Trunk
mozilla15
x86_64
Mac OS X
assertion, testcase
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

5 years ago
Created attachment 627194 [details]
testcase (assumes it is on jesse's computer)

1. Save the testcase.
2. Fix the path to point at your tree.
3. Load [your copy of the testcase] in a debug build of Firefox.
4. Quit Firefox with Cmd+Q.

Result: Deadlock assertions.


###!!! ERROR: Potential deadlock detected:
$ @@@ ###!!! ERROR: Potential deadlock detected:
=== Cyclical dependency starts at
--- Mutex : mozilla::media::SourceMediaStream calling context
stacks_disabled+0[stacks_disabled +0x0]

--- Next dependency:
--- Mutex : MediaStreamGraphImpl (currently acquired)
 calling context
stacks_disabled+0[stacks_disabled +0x0]

=== Cycle completed at
--- Mutex : mozilla::media::SourceMediaStream calling context
stacks_disabled+0[stacks_disabled +0x0]

Deadlock may happen for some other execution

###!!! ASSERTION: Potential deadlock detected:
$ @@@ ###!!! ASSERTION: Potential deadlock detected:
Cyclical dependency starts at
Mutex : mozilla::media::SourceMediaStream
Next dependency:
Mutex : MediaStreamGraphImpl (currently acquired)
Cycle completed at
Mutex : mozilla::media::SourceMediaStream
Deadlock may happen for some other execution

: 'Error', file BlockingResourceBase.cpp, line 128
Created attachment 627840 [details] [diff] [review]
fix

Not sure how to test this since it requires shutting down Firefox...
Assignee: nobody → roc
Attachment #627840 - Flags: review?(rjesup)

Updated

5 years ago
Attachment #627840 - Flags: review?(rjesup) → review+
https://hg.mozilla.org/integration/mozilla-inbound/rev/cfd001299d34
https://hg.mozilla.org/mozilla-central/rev/cfd001299d34
Status: NEW → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla15
You need to log in before you can comment on or make changes to this bug.