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:56 -0500 |
commit | 406cc860729048babee04898496ac52033419df1 (patch) | |
tree | 4270a16141e3a9008e998b0785074b0eb123ab8c /usr/local/www/firewall_nat_edit.php | |
parent | 4d41d0b8253fb21a28e466e654f1757576898364 (diff) | |
download | pfsense-406cc860729048babee04898496ac52033419df1.zip pfsense-406cc860729048babee04898496ac52033419df1.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 61d534b..ce00580 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. */ |