Closed
Bug 1021335
Opened 10 years ago
Closed 10 years ago
leaks in sctp_add_to_readq detected by LSAN
Categories
(Core :: WebRTC: Networking, defect)
Core
WebRTC: Networking
Tracking
()
RESOLVED
FIXED
mozilla32
People
(Reporter: mccr8, Assigned: jesup)
References
Details
(Keywords: memory-leak, Whiteboard: [MemShrink][qa-])
Attachments
(2 files)
3.30 KB,
text/plain
|
Details | |
2.56 KB,
patch
|
tuexen
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
There's a few kb of these. Randell, there's a bunch of leaks reported by Coverity for this SCTP code, so that's probably a good place to start trying to fix these, as that will be easier to turn into a fix than a few random LSAN stacks.
Assignee | ||
Comment 1•10 years ago
|
||
Upstream bug I imagine...
Comment 2•10 years ago
|
||
All "leaks" reported go through the same code path. The SCTP allocates memory which is given to its upper layer via the recv_callback. This memory is expected to be freed by the upper layer in the callback (or later outside). By looking at DataChannel.cc: Where are you freeing the buffer you got? At least not in the receive callback... Best regards Michael
Flags: needinfo?(rjesup)
Assignee | ||
Comment 3•10 years ago
|
||
That would make it my bug; taking
Assignee: nobody → rjesup
Flags: needinfo?(rjesup)
Assignee | ||
Comment 4•10 years ago
|
||
tested under ASAN with no problems
Assignee | ||
Comment 5•10 years ago
|
||
Comment on attachment 8436341 [details] [diff] [review] Fix DataChannel leak of recv buffers from SCTP Given the comment from Michael Tuexen (library author), I just need a sanity check review on this. Tested under ASAN. LSAN rocks
Attachment #8436341 -
Flags: review?(tuexen)
Attachment #8436341 -
Flags: review?(continuation)
Comment 6•10 years ago
|
||
Comment on attachment 8436341 [details] [diff] [review] Fix DataChannel leak of recv buffers from SCTP Review of attachment 8436341 [details] [diff] [review]: ----------------------------------------------------------------- Looks good to me.
Attachment #8436341 -
Flags: review?(tuexen) → review+
Assignee | ||
Comment 7•10 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/1159503d1f38
Target Milestone: --- → mozilla32
Assignee | ||
Updated•10 years ago
|
Whiteboard: [MemShrink] → [MemShrink][webrtc-uplift]
Assignee | ||
Updated•10 years ago
|
Summary: Small leaks in sctp_add_to_readq detected by LSAN → leaks in sctp_add_to_readq detected by LSAN
Assignee | ||
Comment 8•10 years ago
|
||
Comment on attachment 8436341 [details] [diff] [review] Fix DataChannel leak of recv buffers from SCTP [Approval Request Comment] (For 31 Aurora or Beta) Bug caused by (feature/regressing bug #): N/A User impact if declined: major leak using DataChannels Testing completed (on m-c, etc.): On inbound, run datachannel tests locally under ASAN, review by library author Risk to taking this patch (and alternatives if risky): low. ASAN would have shown any problems. Simple mis-use of API; API memory use wasn't documented or I missed it. String or IDL/UUID changes made by this patch: none
Attachment #8436341 -
Flags: review?(continuation) → approval-mozilla-aurora?
Comment 9•10 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/1159503d1f38
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Updated•10 years ago
|
status-firefox31:
--- → affected
status-firefox32:
--- → fixed
Updated•10 years ago
|
Attachment #8436341 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•10 years ago
|
Whiteboard: [MemShrink][webrtc-uplift] → [MemShrink][webrtc-uplift] [qa-]
Assignee | ||
Updated•10 years ago
|
Whiteboard: [MemShrink][webrtc-uplift] [qa-] → [MemShrink][qa-]
You need to log in
before you can comment on or make changes to this bug.
Description
•