[asterisk-commits] sgriepentrog: branch 1.8 r408142 - /branches/1.8/main/pbx.c

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


Author: sgriepentrog
Date: Fri Feb 14 15:52:33 2014
New Revision: 408142

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


Modified:
    branches/1.8/main/pbx.c

Modified: branches/1.8/main/pbx.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/main/pbx.c?view=diff&rev=408142&r1=408141&r2=408142
==============================================================================
--- branches/1.8/main/pbx.c (original)
+++ branches/1.8/main/pbx.c Fri Feb 14 15:52:33 2014
@@ -3794,6 +3794,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