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_gif.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_gif.php')
-rw-r--r-- | usr/local/www/interfaces_gif.php | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/usr/local/www/interfaces_gif.php b/usr/local/www/interfaces_gif.php index 415c50a..ed6b41b 100644 --- a/usr/local/www/interfaces_gif.php +++ b/usr/local/www/interfaces_gif.php @@ -59,8 +59,12 @@ function gif_inuse($num) { } if ($_GET['act'] == "del") { + if (!isset($_GET['id'])) + $input_errors[] = getext("Wrong parameters supplied"); + else if (empty($a_gifs[$_GET['id']])) + $input_errors[] = getext("Wrong index supplied"); /* check if still in use */ - if (gif_inuse($_GET['id'])) { + else if (gif_inuse($_GET['id'])) { $input_errors[] = gettext("This gif TUNNEL cannot be deleted because it is still being used as an interface."); } else { mwexec("/sbin/ifconfig " . $a_gifs[$_GET['id']]['gifif'] . " destroy"); |