[asterisk-commits] kmoore: trunk r422647 - in /trunk: ./ menuselect/menuselect.c

SVN commits to the Asterisk project asterisk-commits at lists.digium.com
Fri Sep 5 08:29:40 CDT 2014


Author: kmoore
Date: Fri Sep  5 08:29:38 2014
New Revision: 422647

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=422647
Log:
Menuselect: Fix incorrect enabling on failed deps

This corrects a situation where menuselect can incorrectly enable a
module by default that has defaultenabled set to "no" and has
failed/non-selected dependencies. The bug is due to an inverted test
when checking for whether the given module should be set to enabled by
default on load.

Review: https://reviewboard.asterisk.org/r/3975/
Reported by: John Bigelow
........

Merged revisions 422646 from http://svn.asterisk.org/svn/asterisk/branches/13

Modified:
    trunk/   (props changed)
    trunk/menuselect/menuselect.c

Propchange: trunk/
------------------------------------------------------------------------------
Binary property 'branch-13-merged' - no diff available.

Modified: trunk/menuselect/menuselect.c
URL: http://svnview.digium.com/svn/asterisk/trunk/menuselect/menuselect.c?view=diff&rev=422647&r1=422646&r2=422647
==============================================================================
--- trunk/menuselect/menuselect.c (original)
+++ trunk/menuselect/menuselect.c Fri Sep  5 08:29:38 2014
@@ -459,7 +459,7 @@
 
 	if (!cat->positive_output) {
 		mem->enabled = 1;
-		if (!(mem->defaultenabled && strcasecmp(mem->defaultenabled, "no"))) {
+		if (!mem->defaultenabled || strcasecmp(mem->defaultenabled, "no")) {
 			mem->was_enabled = 1;
 			print_debug("Enabled %s because the category does not have positive output\n", mem->name);
 		}




More information about the asterisk-commits mailing list