diff options
author | Renato Botelho <garga@FreeBSD.org> | 2013-11-21 09:36:50 -0200 |
---|---|---|
committer | Renato Botelho <garga@FreeBSD.org> | 2013-11-21 09:36:50 -0200 |
commit | 1ad6ad279f44676f149f3f2a35ed641b739f4dfd (patch) | |
tree | b15903ad24b9535cb3fdcc2e4046c6658e66ac77 /usr | |
parent | 186ab4ea1ae72495d52038b08ba118014b3abfab (diff) | |
download | pfsense-1ad6ad279f44676f149f3f2a35ed641b739f4dfd.zip pfsense-1ad6ad279f44676f149f3f2a35ed641b739f4dfd.tar.gz |
Fix an issue that changes wrong gateway entry when items are hidden
Diffstat (limited to 'usr')
-rwxr-xr-x | usr/local/www/system_gateways_edit.php | 27 |
1 files changed, 15 insertions, 12 deletions
diff --git a/usr/local/www/system_gateways_edit.php b/usr/local/www/system_gateways_edit.php index 91601c2..365ff11 100755 --- a/usr/local/www/system_gateways_edit.php +++ b/usr/local/www/system_gateways_edit.php @@ -75,11 +75,11 @@ if (isset($id) && $a_gateways[$id]) { $pconfig['dynamic'] = true; $pconfig['gateway'] = $a_gateways[$id]['gateway']; $pconfig['defaultgw'] = isset($a_gateways[$id]['defaultgw']); - $pconfig['latencylow'] = $a_gateway_item[$id]['latencylow']; - $pconfig['latencyhigh'] = $a_gateway_item[$id]['latencyhigh']; - $pconfig['losslow'] = $a_gateway_item[$id]['losslow']; - $pconfig['losshigh'] = $a_gateway_item[$id]['losshigh']; - $pconfig['down'] = $a_gateway_item[$id]['down']; + $pconfig['latencylow'] = $a_gateways[$id]['latencylow']; + $pconfig['latencyhigh'] = $a_gateways[$id]['latencyhigh']; + $pconfig['losslow'] = $a_gateways[$id]['losslow']; + $pconfig['losshigh'] = $a_gateways[$id]['losshigh']; + $pconfig['down'] = $a_gateways[$id]['down']; $pconfig['monitor'] = $a_gateways[$id]['monitor']; $pconfig['monitor_disable'] = isset($a_gateways[$id]['monitor_disable']); $pconfig['descr'] = $a_gateways[$id]['descr']; @@ -91,6 +91,9 @@ if (isset($_GET['dup'])) { unset($pconfig['attribute']); } +if (isset($id) && $a_gateways[$id]) + $realid = $a_gateways[$id]['attribute']; + if ($_POST) { unset($input_errors); @@ -386,12 +389,12 @@ if ($_POST) { $gateway['monitor_disable'] = true; else if (is_ipaddr($_POST['monitor'])) { /* NOTE: If monitor ip is changed need to cleanup the old static route */ - if ($_POST['monitor'] != "dynamic" && !empty($a_gateway_item[$id]) && is_ipaddr($a_gateway_item[$id]['monitor']) && - $_POST['monitor'] != $a_gateway_item[$id]['monitor'] && $gateway['gateway'] != $a_gateway_item[$id]['monitor']) { - if (is_ipaddrv4($a_gateway_item[$id]['monitor'])) - mwexec("/sbin/route delete " . escapeshellarg($a_gateway_item[$id]['monitor'])); + if ($_POST['monitor'] != "dynamic" && !empty($a_gateway_item[$realid]) && is_ipaddr($a_gateway_item[$realid]['monitor']) && + $_POST['monitor'] != $a_gateway_item[$realid]['monitor'] && $gateway['gateway'] != $a_gateway_item[$realid]['monitor']) { + if (is_ipaddrv4($a_gateway_item[$realid]['monitor'])) + mwexec("/sbin/route delete " . escapeshellarg($a_gateway_item[$realid]['monitor'])); else - mwexec("/sbin/route delete -inet6 " . escapeshellarg($a_gateway_item[$id]['monitor'])); + mwexec("/sbin/route delete -inet6 " . escapeshellarg($a_gateway_item[$realid]['monitor'])); } $gateway['monitor'] = $_POST['monitor']; } @@ -422,8 +425,8 @@ if ($_POST) { $gateway['down'] = $_POST['down']; /* when saving the manual gateway we use the attribute which has the corresponding id */ - if (isset($id) && $a_gateway_item[$id]) - $a_gateway_item[$id] = $gateway; + if (isset($realid) && $a_gateway_item[$realid]) + $a_gateway_item[$realid] = $gateway; else $a_gateway_item[] = $gateway; |