diff options
| author | Ermal <eri@pfsense.org> | 2013-01-24 09:57:22 +0000 |
|---|---|---|
| committer | Ermal <eri@pfsense.org> | 2013-01-24 09:57:22 +0000 |
| commit | 0e22dda58ddc010d12126ed6af6589ff76d03043 (patch) | |
| tree | e7396e447b3f7cb6e4582df870f4f022af9c3bf8 /usr/local/www/interfaces_bridge.php | |
| parent | 7309ff3915c58fd664618f01a91dbe8ac9dd2526 (diff) | |
| download | pfsense-0e22dda58ddc010d12126ed6af6589ff76d03043.zip pfsense-0e22dda58ddc010d12126ed6af6589ff76d03043.tar.gz | |
Put some more sanity checking for various interface configurations
Diffstat (limited to 'usr/local/www/interfaces_bridge.php')
| -rw-r--r-- | usr/local/www/interfaces_bridge.php | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/usr/local/www/interfaces_bridge.php b/usr/local/www/interfaces_bridge.php index 44a21d5..986cd27 100644 --- a/usr/local/www/interfaces_bridge.php +++ b/usr/local/www/interfaces_bridge.php @@ -59,8 +59,12 @@ function bridge_inuse($num) { } if ($_GET['act'] == "del") { + if (!isset($_GET['id'])) + $input_errors[] = getext("Wrong parameters supplied"); + else if (empty($a_bridges[$_GET['id']])) + $input_errors[] = getext("Wrong index supplied"); /* check if still in use */ - if (bridge_inuse($_GET['id'])) { + else if (bridge_inuse($_GET['id'])) { $input_errors[] = gettext("This bridge cannot be deleted because it is assigned as an interface."); } elseif (!does_interface_exist($a_bridges[$_GET['id']]['bridgeif'])) { $input_errors[] = gettext("Invalid bridge interface."); |
