Actually fix problem with assigning number of interfaces greater than 10 or 20.

1 files changed, 4 insertions, 9 deletions

diff --git a/usr/local/www/interfaces_assign.php b/usr/local/www/interfaces_assign.php
index 12bd653..aef0c03 100755
--- a/usr/local/www/interfaces_assign.php
+++ b/usr/local/www/interfaces_assign.php
@@ -241,15 +241,10 @@ if ($_GET['act'] == "add") {
 		$config['interfaces'][$newifname] = array();
 		$config['interfaces'][$newifname]['descr'] = $descr;
 	} else {
-		/* 
-		 * Since we account above for LAN and we start counting 
-		 * optional(opt#) interfaces from 1 the formula below
-		 * gives the next, to be added, opt# interface.
-		 *
-		 * XXX: The best of all i getting rid of lan and wan
-		 *	from interface keys and just keeping their descriptions.
-		 */
-		$i = count($config['interfaces']) - 1;
+		for ($i = 1; $i <= count($config['interfaces']); $i++) {
+			if (!$config['interfaces']["opt{$i}"])
+				break;
+		}
 		$newifname = 'opt' . $i;
 		$descr = "OPT{$i}";
 		$config['interfaces'][$newifname] = array();