[asterisk-commits] mjordan: branch 1.8 r414880 - /branches/1.8/main/config.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri May 30 06:51:07 CDT 2014


Author: mjordan
Date: Fri May 30 06:50:59 2014
New Revision: 414880

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=414880
Log:
main/config.c: AMI action UpdateConfig EmptyCat clears all categories

When invoking UpdateConfig AMI action with Action set to EmptyCat, Asterisk
will make all categories empty in the config but the one requested with a
Cat variable. This is due to a bug in ast_category_empty (main/config.c)
that makes an incorrect comparison for a category name.

This patch corrects the comparison such that only the requested category
is cleared.

Review: https://reviewboard.asterisk.org/r/3573/

ASTERISK-23803 #close
Reported by: zvision
patches:
  manager.c.diff uploaded by zvision (License 5755)


Modified:
    branches/1.8/main/config.c

Modified: branches/1.8/main/config.c
URL: http://svnview.digium.com/svn/asterisk/branches/1.8/main/config.c?view=diff&rev=414880&r1=414879&r2=414880
==============================================================================
--- branches/1.8/main/config.c (original)
+++ branches/1.8/main/config.c Fri May 30 06:50:59 2014
@@ -994,7 +994,7 @@
 	struct ast_category *cat;
 
 	for (cat = cfg->root; cat; cat = cat->next) {
-		if (!strcasecmp(cat->name, category))
+		if (strcasecmp(cat->name, category))
 			continue;
 		ast_variables_destroy(cat->root);
 		cat->root = NULL;




More information about the asterisk-commits mailing list