From 7a701b1efc8b63acb5ad6930bb0d398ac8526ed0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ermal=20Lu=C3=A7i?= Date: Sun, 31 Aug 2008 15:11:40 +0000 Subject: Fix error checking for wireless interfaces being part of a bridge which is allowed only in hostap mode. --- usr/local/www/interfaces_bridge_edit.php | 11 +++++++++++ 1 file changed, 11 insertions(+) (limited to 'usr/local/www/interfaces_bridge_edit.php') diff --git a/usr/local/www/interfaces_bridge_edit.php b/usr/local/www/interfaces_bridge_edit.php index a1b9fdc..05ddc2d 100644 --- a/usr/local/www/interfaces_bridge_edit.php +++ b/usr/local/www/interfaces_bridge_edit.php @@ -123,6 +123,17 @@ if ($_POST) { $input_errors[] = "{$ifdescr} interface path cost for STP needs to be an interger between 1 and 200000000."; $i++; } + + if (!is_array($_POST['members']) || count($_POST['members'] < 2) + $input_errors[] = "You must select at least 2 member interfaces for a bridge."; + + if (is_array($_POST['members'])) { + foreach($_POST['members'] as $ifmembers) + if (is_array($config['interfaces'][$ifmembers]['wireless']) && + $config['interfaces'][$ifmembers]['wireless']['mode'] != "hostap") + $input_errors[] = "Bridging a wireless interface is only possible in hostap mode."; + } + if (!$input_errors) { $bridge = array(); $bridge['members'] = implode(',', $_POST['members']); -- cgit v1.1