variable

Used explode to array rather than to a list, and tested array size, so as not to assume it has exactly 2 parts separated by "/".

1 files changed, 4 insertions, 4 deletions

diff --git a/src/etc/inc/util.inc b/src/etc/inc/util.inc
index 252e48f..2cc71f2 100644
--- a/src/etc/inc/util.inc
+++ b/src/etc/inc/util.inc
@@ -818,10 +818,10 @@ function is_subnetoralias($subnet) {
 function subnet_size($subnet, $exact=false) {
 	$parts = explode("/", $subnet);
 	if (count($parts) == 2) {
-		if (is_ipaddrv4($ip)) {
-			return subnet_size_by_netmask(4, $bits, $exact);
-		} elseif (is_ipaddrv6($ip)) {
-			return subnet_size_by_netmask(6, $bits, $exact);
+		if (is_ipaddrv4($parts[0])) {
+			return subnet_size_by_netmask(4, $parts[1], $exact);
+		} elseif (is_ipaddrv6($parts[0])) {
+			return subnet_size_by_netmask(6, $parts[1], $exact);
 		}
 	}
 	return 0;