[svn-commits] mjordan: trunk r402969 - in /trunk: ./ res/res_pjsip_session.c

[SVN commits to the Digium repositories]

Author: mjordan
Date: Thu Nov 21 13:22:18 2013
New Revision: 402969

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=402969
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!)
........

Merged revisions 402968 from http://svn.asterisk.org/svn/asterisk/branches/12

Modified:
    trunk/   (props changed)
    trunk/res/res_pjsip_session.c

Propchange: trunk/
------------------------------------------------------------------------------
--- branch-12-merged (original)
+++ branch-12-merged Thu Nov 21 13:22:18 2013
@@ -1,1 +1,1 @@
-/branches/12:1-398558,398560-398577,398579-399305,399307-401390,401392-402738,402755,402757,402767,402769,402787,402793,402804,402817,402838,402864,402891,402926,402940-402941,402944,402956
+/branches/12:1-398558,398560-398577,398579-399305,399307-401390,401392-402738,402755,402757,402767,402769,402787,402793,402804,402817,402838,402864,402891,402926,402940-402941,402944,402956,402968

Modified: trunk/res/res_pjsip_session.c
URL: http://svnview.digium.com/svn/asterisk/trunk/res/res_pjsip_session.c?view=diff&rev=402969&r1=402968&r2=402969
==============================================================================
--- trunk/res/res_pjsip_session.c (original)
+++ trunk/res/res_pjsip_session.c Thu Nov 21 13:22:18 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);