diff options
author | Chris Buechler <cmb@pfsense.org> | 2011-12-25 16:16:26 -0500 |
---|---|---|
committer | Chris Buechler <cmb@pfsense.org> | 2011-12-25 16:16:26 -0500 |
commit | 8bb88a62dd93a80980c5655e2b8c0bec9fbfa33a (patch) | |
tree | 2d6c16f1f4761924e13f7660cf00e2b1741015ff /usr/local/www/firewall_nat_edit.php | |
parent | c0ee2a015faac1771badec3a4ed3bce1feba8da2 (diff) | |
download | pfsense-8bb88a62dd93a80980c5655e2b8c0bec9fbfa33a.zip pfsense-8bb88a62dd93a80980c5655e2b8c0bec9fbfa33a.tar.gz |
Local port must be specified, fix input validation
Diffstat (limited to 'usr/local/www/firewall_nat_edit.php')
-rwxr-xr-x | usr/local/www/firewall_nat_edit.php | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/usr/local/www/firewall_nat_edit.php b/usr/local/www/firewall_nat_edit.php index 0a524b3..83cfd73 100755 --- a/usr/local/www/firewall_nat_edit.php +++ b/usr/local/www/firewall_nat_edit.php @@ -222,8 +222,8 @@ if ($_POST) { if ($_POST['dstendport'] && !is_portoralias($_POST['dstendport'])) $input_errors[] = sprintf(gettext("%s is not a valid end destination port. It must be a port alias or integer between 1 and 65535."), $_POST['dstendport']); - if (!isset($_POST['nordr']) && $_POST['localbeginport'] && !is_portoralias($_POST['localbeginport'])) { - $input_errors[] = sprintf(gettext("%s is not a valid local port. It must be a port alias or integer between 1 and 65535."), $_POST['localbeginport']); + if (!isset($_POST['nordr']) && !is_portoralias($_POST['localbeginport'])) { + $input_errors[] = sprintf(gettext("A valid local port must be specified. It must be a port alias or integer between 1 and 65535."), $_POST['localbeginport']); } /* if user enters an alias and selects "network" then disallow. */ |