diff options
author | Seth Mos <seth.mos@dds.nl> | 2011-03-27 13:17:41 +0200 |
---|---|---|
committer | Seth Mos <seth.mos@dds.nl> | 2011-03-27 13:17:41 +0200 |
commit | 404e0009306860284e0df34efdc36eb6fb5f0ed2 (patch) | |
tree | 79bfe5d4f87420fc2a1ccfdfd7b92c0689b5c514 | |
parent | 588402bf991c1caf601852f3187e20fcdcb06216 (diff) | |
download | pfsense-404e0009306860284e0df34efdc36eb6fb5f0ed2.zip pfsense-404e0009306860284e0df34efdc36eb6fb5f0ed2.tar.gz |
Fix the gif edit page, it was checking the subnet bits, not the actual subnet. The check needs to be refined to check actual subnets instead of just the address. but this will do for now.
-rw-r--r-- | usr/local/www/interfaces_gif_edit.php | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/usr/local/www/interfaces_gif_edit.php b/usr/local/www/interfaces_gif_edit.php index 21496fe..04d22bc 100644 --- a/usr/local/www/interfaces_gif_edit.php +++ b/usr/local/www/interfaces_gif_edit.php @@ -82,8 +82,9 @@ if ($_POST) { if (isset($id) && ($a_gifs[$id]) && ($a_gifs[$id] === $gif)) continue; - if (($gif['if'] == $_POST['if']) && ($gif['tunnel-remote-net'] == $_POST['tunnel-remote-net'])) { - $input_errors[] = sprintf(gettext("A gif with the network %s is already defined."), $gif['remote-network']); + /* FIXME: needs to perform proper subnet checks in the feature */ + if (($gif['if'] == $_POST['if']) && ($gif['tunnel-remote-addr'] == $_POST['tunnel-remote-addr'])) { + $input_errors[] = sprintf(gettext("A gif with the network %s is already defined."), $gif['tunnel-remote-addr']); break; } } |