[asterisk-commits] mjordan: branch 12 r402968 - /branches/12/res/res_pjsip_session.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Thu Nov 21 13:21:22 CST 2013


Author: mjordan
Date: Thu Nov 21 13:21:20 2013
New Revision: 402968

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=402968
Log:
res_pjsip_session: Fix memory leak of direct media format capabilities

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!)

Modified:
    branches/12/res/res_pjsip_session.c

Modified: branches/12/res/res_pjsip_session.c
URL: http://svnview.digium.com/svn/asterisk/branches/12/res/res_pjsip_session.c?view=diff&rev=402968&r1=402967&r2=402968
==============================================================================
--- branches/12/res/res_pjsip_session.c (original)
+++ branches/12/res/res_pjsip_session.c Thu Nov 21 13:21:20 2013
@@ -1026,6 +1026,7 @@
 	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);




More information about the asterisk-commits mailing list