Commit 28ff9d83 authored by Ad Schellevis's avatar Ad Schellevis Committed by Franco Fichtner

(legacy) regression services_upnp.php

(cherry picked from commit 1ffc2f54)
parent 372dd303
...@@ -83,11 +83,13 @@ if ($_SERVER['REQUEST_METHOD'] === 'GET') { ...@@ -83,11 +83,13 @@ if ($_SERVER['REQUEST_METHOD'] === 'GET') {
if (!empty($pconfig['enable']) && (empty($pconfig['enable_upnp']) && empty($pconfig['enable_natpmp']))) { if (!empty($pconfig['enable']) && (empty($pconfig['enable_upnp']) && empty($pconfig['enable_natpmp']))) {
$input_errors[] = gettext('At least one of \'UPnP\' or \'NAT-PMP\' must be allowed'); $input_errors[] = gettext('At least one of \'UPnP\' or \'NAT-PMP\' must be allowed');
} }
foreach($pconfig['iface_array'] as $iface) { if (!empty($pconfig['iface_array'])) {
if ($iface == 'wan') { foreach($pconfig['iface_array'] as $iface) {
$input_errors[] = gettext('It is a security risk to specify WAN in the \'Interface\' field'); if ($iface == 'wan') {
} elseif ($iface == $pconfig['ext_iface']) { $input_errors[] = gettext('It is a security risk to specify WAN in the \'Interface\' field');
$input_errors[] = gettext('You cannot select the external interface as an internal interface.'); } elseif ($iface == $pconfig['ext_iface']) {
$input_errors[] = gettext('You cannot select the external interface as an internal interface.');
}
} }
} }
if (!empty($pconfig['overridewanip']) && !is_ipaddr($pconfig['overridewanip'])) { if (!empty($pconfig['overridewanip']) && !is_ipaddr($pconfig['overridewanip'])) {
...@@ -225,12 +227,12 @@ include("head.inc"); ...@@ -225,12 +227,12 @@ include("head.inc");
<td><a id="help_for_iface_array" href="#" class="showhelp"><i class="fa fa-info-circle"></i></a> <?=gettext("Interfaces (generally LAN)");?></td> <td><a id="help_for_iface_array" href="#" class="showhelp"><i class="fa fa-info-circle"></i></a> <?=gettext("Interfaces (generally LAN)");?></td>
<td> <td>
<select class="selectpicker" name="iface_array[]" multiple="multiple"> <select class="selectpicker" name="iface_array[]" multiple="multiple">
<option value="lo0" <?=in_array('lo0', $pconfig['iface_array']) ? "selected=\"selected\"" : "";?>> <option value="lo0" <?=!empty($pconfig['iface_array']) && in_array('lo0', $pconfig['iface_array']) ? "selected=\"selected\"" : "";?>>
<?=gettext("Localhost");?> <?=gettext("Localhost");?>
</option> </option>
<?php <?php
foreach (get_configured_interface_with_descr() as $iface => $ifacename):?> foreach (get_configured_interface_with_descr() as $iface => $ifacename):?>
<option value="<?=$iface;?>" <?=in_array($iface, $pconfig['iface_array']) ? "selected=\"selected\"" : "";?>> <option value="<?=$iface;?>" <?=!empty($pconfig['iface_array']) && in_array($iface, $pconfig['iface_array']) ? "selected=\"selected\"" : "";?>>
<?=htmlspecialchars($ifacename);?> <?=htmlspecialchars($ifacename);?>
</option> </option>
<?php <?php
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment