[svn-commits] mjordan: trunk r348160 - /trunk/apps/app_confbridge.c

[SVN commits to the Digium repositories]

Author: mjordan
Date: Wed Dec 14 14:51:39 2011
New Revision: 348160

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=348160
Log:
Improve error message in CONFBRIDGE_INFO

Provided a more descriptive error message when a value supplied for the parameter
type is not one of the acceptable values.

(closes issue ASTERISK-18717)
Reported by: Paul Belanger
Patches:
  __20111103-better-confbridge_info-error-msg.txt (License #4999)

Modified:
    trunk/apps/app_confbridge.c

Modified: trunk/apps/app_confbridge.c
URL: http://svnview.digium.com/svn/asterisk/trunk/apps/app_confbridge.c?view=diff&rev=348160&r1=348159&r2=348160
==============================================================================
--- trunk/apps/app_confbridge.c (original)
+++ trunk/apps/app_confbridge.c Wed Dec 14 14:51:39 2011
@@ -2700,7 +2700,8 @@
 	} else if (!strncasecmp(args.type, "locked", 6)) {
 		count = bridge->locked;
 	} else {
-		ast_log(LOG_ERROR, "Invalid keyword.\n");
+		ast_log(LOG_ERROR, "Invalid keyword '%s' passed to CONFBRIDGE_INFO.  Should be one of: "
+			"parties, admins, marked, or locked.\n", args.type);
 	}
 	snprintf(buf, len, "%d", count);
 	ao2_unlock(bridge);