diff --git a/usr/local/www/interfaces_lan.php b/usr/local/www/interfaces_lan.php
index ad2867deff..9cbb04e4bb 100755
--- a/usr/local/www/interfaces_lan.php
+++ b/usr/local/www/interfaces_lan.php
@@ -50,15 +50,16 @@ if ($_POST) {
if ($_POST['bridge']) {
/* double bridging? */
- for ($i = 1; isset($config['interfaces']['opt' . $i]); $i++) {
- if ($i != $index) {
- if ($config['interfaces']['opt' . $i]['bridge'] == $_POST['bridge']) {
- //$input_errors[] = "Optional interface {$i} " .
- // "({$config['interfaces']['opt' . $i]['descr']}) is already bridged to " .
+ $iflist = get_configured_interface_list(true, true);
+ foreach ($iflist as $if) {
+ if ($if != $index) { /* XXX: totaly bogus */
+ if ($config['interfaces'][$if]['bridge'] == $_POST['bridge']) {
+ //$input_errors[] = "Optional interface {$if} " .
+ // "({$config['interfaces'][$if]['descr']}) is already bridged to " .
// "the specified interface.";
- } else if ($config['interfaces']['opt' . $i]['bridge'] == "opt{$index}") {
- //$input_errors[] = "Optional interface {$i} " .
- // "({$config['interfaces']['opt' . $i]['descr']}) is already bridged to " .
+ } else if ($config['interfaces'][$if]['bridge'] == $if) {
+ //$input_errors[] = "Optional interface {$if} " .
+ // "({$config['interfaces'][$if]['descr']}) is already bridged to " .
// "this interface.";
}
}
@@ -174,13 +175,10 @@ function enable_change(enable_over) {
|