[Asterisk-code-review] DTLS/SRTP: Resolve 2 discrete memory leaks (asterisk[master])

Joshua Colp asteriskteam at digium.com
Tue Apr 28 06:52:35 CDT 2015


Joshua Colp has uploaded a new patch set (#3).

Change subject: DTLS/SRTP: Resolve 2 discrete memory leaks
......................................................................

DTLS/SRTP: Resolve 2 discrete memory leaks

ao2 ref leak in res_rtp_asterisk.c when a DTLS policy is created.
The resources are linked into a table, but the original alloc refs
are never released. ast_strdup leak in rtp_engine.c. If ast_rtp_dtls_cfg_copy() is
called twice on the same destination struct, a pointer to an alloc'd
string is overwritten before the string is free'd.

ASTERISK-25022
Reported by: one47

Change-Id: I62a8ceb8679709f6c3769136dc6aa9a68202ff9b
---
M main/rtp_engine.c
M res/res_rtp_asterisk.c
2 files changed, 6 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/70/270/3
-- 
To view, visit https://gerrit.asterisk.org/270
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I62a8ceb8679709f6c3769136dc6aa9a68202ff9b
Gerrit-PatchSet: 3
Gerrit-Project: asterisk
Gerrit-Branch: master
Gerrit-Owner: Steve Davies <steve at one47.co.uk>
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Reviewer: Steve Davies <steve at one47.co.uk>



More information about the asterisk-code-review mailing list