[asterisk-commits] file: branch 11 r377021 - /branches/11/channels/chan_motif.c

[SVN commits to the Asterisk project]

Author: file
Date: Mon Dec  3 08:54:54 2012
New Revision: 377021

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

Modified:
    branches/11/channels/chan_motif.c

Modified: branches/11/channels/chan_motif.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/channels/chan_motif.c?view=diff&rev=377021&r1=377020&r2=377021
==============================================================================
--- branches/11/channels/chan_motif.c (original)
+++ branches/11/channels/chan_motif.c Mon Dec  3 08:54:54 2012
@@ -426,10 +426,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);
 	}