]> git.ipfire.org Git - thirdparty/asterisk.git/commitdiff
res_pjsip_session: Fix memory leak of direct media format capabilities
authorMatthew Jordan <mjordan@digium.com>
Thu, 21 Nov 2013 19:21:20 +0000 (19:21 +0000)
committerMatthew Jordan <mjordan@digium.com>
Thu, 21 Nov 2013 19:21:20 +0000 (19:21 +0000)
The direct media format capabilities are always allocated in
ast_sip_session_alloc and were not freed in the session destructor. Whoops.

(This being the third whoops caught by Scott and Nitesh's valgrind work for
the Asterisk Test Suite. Nifty!)

git-svn-id: https://origsvn.digium.com/svn/asterisk/branches/12@402968 65c4cc65-6c06-0410-ace0-fbb531ad65f3

res/res_pjsip_session.c

index 4dae4b4830f8c875f3a820506ead90e61a64e3bb..f44ec4422cc36721d4b7fa94825c8299c34a6514 100644 (file)
@@ -1026,6 +1026,7 @@ static void session_destructor(void *obj)
        ast_party_id_free(&session->id);
        ao2_cleanup(session->endpoint);
        ast_format_cap_destroy(session->req_caps);
+       ast_format_cap_destroy(session->direct_media_cap);
 
        if (session->dsp) {
                ast_dsp_free(session->dsp);