diff options
| author | Renato Botelho <renato.botelho@bluepex.com> | 2010-04-30 15:07:49 -0300 |
|---|---|---|
| committer | Renato Botelho <renato.botelho@bluepex.com> | 2010-04-30 15:07:49 -0300 |
| commit | a457e739f9764d51aada8c3bd3ac4efd943aac9c (patch) | |
| tree | 1ac6b4864cb50f5b261c6d3a67b806438ea3caf6 /usr/local/www/firewall_nat_edit.php | |
| parent | fb5e6dccf031acf7dd8d515590cfeb3331bb92da (diff) | |
| download | pfsense-a457e739f9764d51aada8c3bd3ac4efd943aac9c.zip pfsense-a457e739f9764d51aada8c3bd3ac4efd943aac9c.tar.gz | |
Initialize variable properly, it fixes #557
Diffstat (limited to 'usr/local/www/firewall_nat_edit.php')
| -rwxr-xr-x | usr/local/www/firewall_nat_edit.php | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/usr/local/www/firewall_nat_edit.php b/usr/local/www/firewall_nat_edit.php index bff59ed..8c53421 100755 --- a/usr/local/www/firewall_nat_edit.php +++ b/usr/local/www/firewall_nat_edit.php @@ -668,7 +668,8 @@ include("fbegin.inc"); ?> <td> <select name="dstbeginport" class="formselect" onchange="dst_rep_change();ext_change()"> <option value="">(other)</option> -<?php foreach ($wkports as $wkport => $wkportdesc): ?> +<?php $bfound = 0; + foreach ($wkports as $wkport => $wkportdesc): ?> <option value="<?=$wkport;?>" <?php if ($wkport == $pconfig['dstbeginport']) { echo "selected"; $bfound = 1; }?>><?=htmlspecialchars($wkportdesc);?></option> <?php endforeach; ?> </select> @@ -680,7 +681,8 @@ include("fbegin.inc"); ?> <td> <select name="dstendport" class="formselect" onchange="ext_change()"> <option value="">(other)</option> -<?php foreach ($wkports as $wkport => $wkportdesc): ?> +<?php $bfound = 0; + foreach ($wkports as $wkport => $wkportdesc): ?> <option value="<?=$wkport;?>" <?php if ($wkport == $pconfig['dstendport']) { echo "selected"; $bfound = 1; } ?>><?=htmlspecialchars($wkportdesc);?></option> <?php endforeach; ?> </select> |
