[svn-commits] dvossel: branch 1.4 r213103 - /branches/1.4/apps/app_mixmonitor.c

SVN commits to the Digium repositories svn-commits at lists.digium.com
Wed Aug 19 16:18:42 CDT 2009


Author: dvossel
Date: Wed Aug 19 16:18:37 2009
New Revision: 213103

URL: http://svn.asterisk.org/svn-view/asterisk?view=rev&rev=213103
Log:
Fixes memory leak caused by incorrectly freeing mixmonitor

(closes issue #15699)
Reported by: edantie
Patches:
      mixmonitor.patch uploaded by edantie (license 862)


Modified:
    branches/1.4/apps/app_mixmonitor.c

Modified: branches/1.4/apps/app_mixmonitor.c
URL: http://svn.asterisk.org/svn-view/asterisk/branches/1.4/apps/app_mixmonitor.c?view=diff&rev=213103&r1=213102&r2=213103
==============================================================================
--- branches/1.4/apps/app_mixmonitor.c (original)
+++ branches/1.4/apps/app_mixmonitor.c Wed Aug 19 16:18:37 2009
@@ -388,7 +388,7 @@
 
 	/* Setup the actual spy before creating our thread */
 	if (ast_audiohook_init(&mixmonitor->audiohook, AST_AUDIOHOOK_TYPE_SPY, mixmonitor_spy_type)) {
-		free(mixmonitor);
+		mixmonitor_free(mixmonitor);
 		return;
 	}
 	
@@ -404,7 +404,7 @@
 			mixmonitor_spy_type, chan->name);
 		/* Since we couldn't add ourselves - bail out! */
 		ast_audiohook_destroy(&mixmonitor->audiohook);
-		free(mixmonitor);
+		mixmonitor_free(mixmonitor);
 		return;
 	}
 




More information about the svn-commits mailing list