[asterisk-commits] sgriepentrog: branch 11 r408143 - in /branches/11: ./ main/pbx.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri Feb 14 15:53:40 CST 2014


Author: sgriepentrog
Date: Fri Feb 14 15:53:38 2014
New Revision: 408143

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=408143
Log:
pbx: ast_custom_function_unregister resource leak

In pbx.c ast_custom_function_unregister(), a list
of escalations being removed from the list wasn't
being free'd creating a leak. This patch corrects
that by freeing the records.

Review: https://reviewboard.asterisk.org/r/3213/
Reported by: Corey Farrell
Patches:
     acf_escalating_leak.patch uploaded by coreyfarrell (license 5909)
........

Merged revisions 408142 from http://svn.asterisk.org/svn/asterisk/branches/1.8

Modified:
    branches/11/   (props changed)
    branches/11/main/pbx.c

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

Modified: branches/11/main/pbx.c
URL: http://svnview.digium.com/svn/asterisk/branches/11/main/pbx.c?view=diff&rev=408143&r1=408142&r2=408143
==============================================================================
--- branches/11/main/pbx.c (original)
+++ branches/11/main/pbx.c Fri Feb 14 15:53:38 2014
@@ -3957,6 +3957,7 @@
 	AST_RWLIST_TRAVERSE_SAFE_BEGIN(&escalation_root, cur_escalation, list) {
 		if (cur_escalation->acf == acf) {
 			AST_RWLIST_REMOVE_CURRENT(list);
+			ast_free(cur_escalation);
 			break;
 		}
 	}




More information about the asterisk-commits mailing list