kenmirrors/asterisk
1
0
Fork 0
mirror of https://github.com/asterisk/asterisk.git synced 2025-10-31 10:47:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

res_pjsip_sdp_rtp fix leaking astobj2 ast_format

Browse Source 
PR #700 added a preferred_format for the struct ast_rtp_codecs,
but when set the preferred_format it leaks an astobj2 ast_format.
In the next code
ast_rtp_codecs_set_preferred_format(&codecs, ast_format_cap_get_format(joint, 0));
both functions ast_rtp_codecs_set_preferred_format
and ast_format_cap_get_format increases the ao2 reference count.

Fixes: #856
This commit is contained in:
Alexei Gradinari
2024-08-23 16:28:24 -04:00
parent 69b7fa29d7
commit 3e2bb5a01a
2 changed files with 5 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
res/res_pjsip_sdp_rtp.c
View File

@@ -544,7 +544,10 @@ static int set_caps(struct ast_sip_session *session,
ast_format_cap_get_names(caps, &usbuf),
ast_format_cap_get_names(peer, &thembuf));
} else {
ast_rtp_codecs_set_preferred_format(&codecs, ast_format_cap_get_format(joint, 0));
struct ast_format *preferred_fmt = ast_format_cap_get_format(joint, 0);
ast_rtp_codecs_set_preferred_format(&codecs, preferred_fmt);
ao2_ref(preferred_fmt, -1);
}
if (is_offer) {