You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@vcl.apache.org by jf...@apache.org on 2010/01/20 21:42:24 UTC

svn commit: r901370 - /incubator/vcl/trunk/web/.ht-inc/computers.php

Author: jfthomps
Date: Wed Jan 20 20:42:24 2010
New Revision: 901370

URL: http://svn.apache.org/viewvc?rev=901370&view=rev
Log:
(no issue associated with this)

modified processBulkComputerInput to look at startpripaddrss and endpripaddress for anyone instead of just users with ADMIN_DEVELOPER access

Modified:
    incubator/vcl/trunk/web/.ht-inc/computers.php

Modified: incubator/vcl/trunk/web/.ht-inc/computers.php
URL: http://svn.apache.org/viewvc/incubator/vcl/trunk/web/.ht-inc/computers.php?rev=901370&r1=901369&r2=901370&view=diff
==============================================================================
--- incubator/vcl/trunk/web/.ht-inc/computers.php (original)
+++ incubator/vcl/trunk/web/.ht-inc/computers.php Wed Jan 20 20:42:24 2010
@@ -3072,64 +3072,62 @@
 		                          . "w, x, y, and z being between 1 and 255 (inclusive)";
 	}
 	$endpraddrArr = array();
-	if($viewmode == ADMIN_DEVELOPER) {
-		if(! empty($return['startpripaddress']) ||
-		   ! empty($return['endpripaddress'])) {
-			$startpraddrArr = explode('.', $return["startpripaddress"]);
-			if(! ereg('^(([0-9]){1,3}\.){3}([0-9]){1,3}$', $return["startpripaddress"]) ||
-				$startpraddrArr[0] < 1 || $startpraddrArr[0] > 255 ||
-				$startpraddrArr[1] < 0 || $startpraddrArr[1] > 255 ||
-				$startpraddrArr[2] < 0 || $startpraddrArr[2] > 255 ||
-				$startpraddrArr[3] < 1 || $startpraddrArr[3] > 255) {
-				$submitErr |= IPADDRERR3;
-				$submitErrMsg[IPADDRERR3] = "Invalid IP address. Must be w.x.y.z with each of "
-				                          . "w, x, y, and z being between 1 and 255 (inclusive)";
-			}
-			$endpraddrArr = explode('.', $return["endpripaddress"]);
-			if(! ereg('^(([0-9]){1,3}\.){3}([0-9]){1,3}$', $return["endpripaddress"]) ||
-				$endpraddrArr[0] < 1 || $endpraddrArr[0] > 255 ||
-				$endpraddrArr[1] < 0 || $endpraddrArr[1] > 255 ||
-				$endpraddrArr[2] < 0 || $endpraddrArr[2] > 255 ||
-				$endpraddrArr[3] < 1 || $endpraddrArr[3] > 255) {
-				$submitErr |= IPADDRERR4;
-				$submitErrMsg[IPADDRERR4] = "Invalid IP address. Must be w.x.y.z with each of "
-				                          . "w, x, y, and z being between 1 and 255 (inclusive)";
-			}
+	if(! empty($return['startpripaddress']) ||
+		! empty($return['endpripaddress'])) {
+		$startpraddrArr = explode('.', $return["startpripaddress"]);
+		if(! ereg('^(([0-9]){1,3}\.){3}([0-9]){1,3}$', $return["startpripaddress"]) ||
+			$startpraddrArr[0] < 1 || $startpraddrArr[0] > 255 ||
+			$startpraddrArr[1] < 0 || $startpraddrArr[1] > 255 ||
+			$startpraddrArr[2] < 0 || $startpraddrArr[2] > 255 ||
+			$startpraddrArr[3] < 1 || $startpraddrArr[3] > 255) {
+			$submitErr |= IPADDRERR3;
+			$submitErrMsg[IPADDRERR3] = "Invalid IP address. Must be w.x.y.z with each of "
+			                          . "w, x, y, and z being between 1 and 255 (inclusive)";
 		}
-		if(! empty($return['startmac'])) {
-		   if(! ereg('^(([A-Fa-f0-9]){2}:){5}([A-Fa-f0-9]){2}$', $return["startmac"])) {
-				$submitErr |= MACADDRERR;
-				$submitErrMsg[MACADDRERR] = "Invalid MAC address.  Must be XX:XX:XX:XX:XX:XX "
-				                          . "with each pair of XX being from 00 to FF (inclusive)";
-			}
-			elseif(! $submitErr) {
-				$tmp = explode(':', $return['startmac']);
-				$topdec = hexdec($tmp[0] . $tmp[1] . $tmp[2]);
-				$botdec = hexdec($tmp[3] . $tmp[4] . $tmp[5]);
-				$topmac = "{$tmp[0]}:{$tmp[1]}:{$tmp[2]}";
-				$topplus = implode(':', str_split(dechex($topdec + 1), 2));
-				$start = $botdec;
-				$return['macs'] = array();
-				$end = $start + (($endaddrArr[3] - $startaddrArr[3] + 1) * 2);
-				for($i = $start; $i < $end; $i++) {
-					if($i > 16777215) {
-						$val = $i - 16777216;
-						$tmp = sprintf('%06x', $val);
-						$tmp2 = str_split($tmp, 2);
-						$return['macs'][] = $topplus . ':' . implode(':', $tmp2);
-					}
-					else {
-						$tmp = sprintf('%06x', $i);
-						$tmp2 = str_split($tmp, 2);
-						$return['macs'][] = $topmac . ':' . implode(':', $tmp2);
-					}
+		$endpraddrArr = explode('.', $return["endpripaddress"]);
+		if(! ereg('^(([0-9]){1,3}\.){3}([0-9]){1,3}$', $return["endpripaddress"]) ||
+			$endpraddrArr[0] < 1 || $endpraddrArr[0] > 255 ||
+			$endpraddrArr[1] < 0 || $endpraddrArr[1] > 255 ||
+			$endpraddrArr[2] < 0 || $endpraddrArr[2] > 255 ||
+			$endpraddrArr[3] < 1 || $endpraddrArr[3] > 255) {
+			$submitErr |= IPADDRERR4;
+			$submitErrMsg[IPADDRERR4] = "Invalid IP address. Must be w.x.y.z with each of "
+			                          . "w, x, y, and z being between 1 and 255 (inclusive)";
+		}
+	}
+	if(! empty($return['startmac'])) {
+		if(! ereg('^(([A-Fa-f0-9]){2}:){5}([A-Fa-f0-9]){2}$', $return["startmac"])) {
+			$submitErr |= MACADDRERR;
+			$submitErrMsg[MACADDRERR] = "Invalid MAC address.  Must be XX:XX:XX:XX:XX:XX "
+			                          . "with each pair of XX being from 00 to FF (inclusive)";
+		}
+		elseif(! $submitErr) {
+			$tmp = explode(':', $return['startmac']);
+			$topdec = hexdec($tmp[0] . $tmp[1] . $tmp[2]);
+			$botdec = hexdec($tmp[3] . $tmp[4] . $tmp[5]);
+			$topmac = "{$tmp[0]}:{$tmp[1]}:{$tmp[2]}";
+			$topplus = implode(':', str_split(dechex($topdec + 1), 2));
+			$start = $botdec;
+			$return['macs'] = array();
+			$end = $start + (($endaddrArr[3] - $startaddrArr[3] + 1) * 2);
+			for($i = $start; $i < $end; $i++) {
+				if($i > 16777215) {
+					$val = $i - 16777216;
+					$tmp = sprintf('%06x', $val);
+					$tmp2 = str_split($tmp, 2);
+					$return['macs'][] = $topplus . ':' . implode(':', $tmp2);
 				}
-				if($i > 16777215 && $topdec == 16777215) {
-					$submitErr |= MACADDRERR;
-					$submitErrMsg[MACADDRERR] = "Starting MAC address too large for given "
-					                          . "given number of machines";
+				else {
+					$tmp = sprintf('%06x', $i);
+					$tmp2 = str_split($tmp, 2);
+					$return['macs'][] = $topmac . ':' . implode(':', $tmp2);
 				}
 			}
+			if($i > 16777215 && $topdec == 16777215) {
+				$submitErr |= MACADDRERR;
+				$submitErrMsg[MACADDRERR] = "Starting MAC address too large for given "
+				                          . "given number of machines";
+			}
 		}
 	}
 	if($return["ram"] < 32 || $return["ram"] > 20480) {