Closed Bug 1161619 Opened 9 years ago Closed 9 years ago

|RunStatsQuery| leaks |queries|

Categories

(Core :: WebRTC: Signaling, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla40
Tracking Status
firefox40 --- fixed

People

(Reporter: erahm, Assigned: pkerr)

References

(Blocks 1 open bug)

Details

(Keywords: coverity, Whiteboard: [CID 1296139][MemShrink:P3])

Attachments

(1 file)

Missing nsAutoPtr it looks like.
The WebrtcGlobalChild should not be deleted at the end of this sequence of work. It exists until the content process containing the associated PeerConnectionCtx instance is deleted. May need to be ref counted.
The leak is RTCStatsQueries, is it not?
Yes, you are correct. I was focusing on line 401 and the function args. Now I see how the highlighted lines are the exit points.
Whiteboard: [CID 1296139] → [CID 1296139][MemShrink:P3]
Assignee: nobody → pkerr
Status: NEW → ASSIGNED
Attachment #8602262 - Flags: review?(jib)
Attachment #8602262 - Flags: review?(jib) → review+
https://hg.mozilla.org/mozilla-central/rev/b1d38f4141af
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in before you can comment on or make changes to this bug.