[asterisk-commits] mmichelson: trunk r248946 - /trunk/main/acl.c

[SVN commits to the Asterisk project]

Author: mmichelson
Date: Thu Feb 25 16:41:48 2010
New Revision: 248946

URL: http://svnview.digium.com/svn/asterisk?view=rev&rev=248946
Log:
Fix incorrect ACL behavior when CIDR notation of "/0" is used.

AST-2010-003


Modified:
    trunk/main/acl.c

Modified: trunk/main/acl.c
URL: http://svnview.digium.com/svn/asterisk/trunk/main/acl.c?view=diff&rev=248946&r1=248945&r2=248946
==============================================================================
--- trunk/main/acl.c (original)
+++ trunk/main/acl.c Thu Feb 25 16:41:48 2010
@@ -298,7 +298,14 @@
 
 		if (!strchr(nm, '.')) {
 			if ((sscanf(nm, "%30d", &x) == 1) && (x >= 0) && (x <= 32)) {
-				ha->netmask.s_addr = htonl(0xFFFFFFFF << (32 - x));
+				if (x == 0) {
+					/* This is special-cased to prevent unpredictable
+					 * behavior of shifting left 32 bits
+					 */
+					ha->netmask.s_addr = 0;
+				} else {
+					ha->netmask.s_addr = htonl(0xFFFFFFFF << (32 - x));
+				}
 			} else {
 				ast_log(LOG_WARNING, "Invalid CIDR in %s\n", stuff);
 				ast_free(ha);