[asterisk-commits] file: trunk r377022 - in /trunk: ./ channels/chan_motif.c

[SVN commits to the Asterisk project]

Author: file
Date: Mon Dec  3 08:56:36 2012
New Revision: 377022

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=377022
Log:
Fix an RTP instance reference count leak in chan_motif.

When setting up an RTP instance the RTCP portion of the instance
keeps a reference to the instance itself. In order to release this
reference and stop RTCP the stop API call must be called before
destroying the instance.

(closes issue ASTERISK-20751)
Reported by: joshoa
........

Merged revisions 377021 from http://svn.asterisk.org/svn/asterisk/branches/11

Modified:
    trunk/   (props changed)
    trunk/channels/chan_motif.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-11-merged' - no diff available.

Modified: trunk/channels/chan_motif.c
URL: http://svnview.digium.com/svn/asterisk/trunk/channels/chan_motif.c?view=diff&rev=377022&r1=377021&r2=377022
==============================================================================
--- trunk/channels/chan_motif.c (original)
+++ trunk/channels/chan_motif.c Mon Dec  3 08:56:36 2012
@@ -434,10 +434,12 @@
 	}
 
 	if (session->rtp) {
+		ast_rtp_instance_stop(session->rtp);
 		ast_rtp_instance_destroy(session->rtp);
 	}
 
 	if (session->vrtp) {
+		ast_rtp_instance_stop(session->vrtp);
 		ast_rtp_instance_destroy(session->vrtp);
 	}