Open Bug 1441831 Opened 5 years ago Updated 2 months ago
Potential memory leak in mozilla-unified/media/mtransport/nr
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/63.0.3239.108 Safari/537.36 Steps to reproduce: Scan Code with SourceBrella Pinpoint Actual results: File mozilla-unified/media/mtransport/nr_timer.cpp:216 The allocated variable callback on line 210 is leaked when the program returns on line 217. I've uploaded a screenshot with source code and comments for a better description.
Based on the source code location, I believe this should belong to the core/WebRTC.
Component: Untriaged → WebRTC
Product: Firefox → Core
Looks legit. Thanks for filing! Nils, who maintains mtransport these days?
Status: UNCONFIRMED → NEW
Component: WebRTC → WebRTC: Networking
Ever confirmed: true
Priority: -- → P3
Looks like we can't delete it directly. And callback->Cancel() assumes that _timer has been set before. I guess we need to modify nrappkitTimerCallback one way or the other. Byron, any thoughts on this?
Flags: needinfo?(drno) → needinfo?(docfaraday)
(In reply to Nils Ohlmeier [:drno] from comment #3) > Looks like we can't delete it directly. And callback->Cancel() assumes that > _timer has been set before. I guess we need to modify nrappkitTimerCallback > one way or the other. > > Byron, any thoughts on this? I would think we could put it in a RefPtr when we create it.
You need to log in before you can comment on or make changes to this bug.