From 8319ee6335028d9caa444816498a3dfe4587f430 Mon Sep 17 00:00:00 2001 From: jim-p Date: Mon, 17 May 2010 11:54:43 -0400 Subject: Fix OpenVPN client disable variable. It is being set/tested improperly (or at least inconsistently). Partial fix for #449. --- usr/local/www/vpn_openvpn_client.php | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) (limited to 'usr') diff --git a/usr/local/www/vpn_openvpn_client.php b/usr/local/www/vpn_openvpn_client.php index 2d7197d..cbb6925 100644 --- a/usr/local/www/vpn_openvpn_client.php +++ b/usr/local/www/vpn_openvpn_client.php @@ -77,7 +77,7 @@ if($_GET['act']=="edit"){ if (isset($id) && $a_client[$id]) { - $pconfig['disable'] = $a_client[$id]['disable']; + $pconfig['disable'] = isset($a_client[$id]['disable']); $pconfig['mode'] = $a_client[$id]['mode']; $pconfig['protocol'] = $a_client[$id]['protocol']; $pconfig['interface'] = $a_client[$id]['interface']; @@ -207,7 +207,8 @@ if ($_POST) { else $client['vpnid'] = openvpn_vpnid_next(); - $client['disable'] = $pconfig['disable']; + if ($_POST['disable'] == "yes") + $server['disable'] = true; $client['protocol'] = $pconfig['protocol']; $client['dev_mode'] = $pconfig['dev_mode']; list($client['interface'], $client['ipaddr']) = explode ("|",$pconfig['interface']); @@ -808,7 +809,7 @@ function autotls_change() { $i = 0; foreach($a_client as $client): $disabled = "NO"; - if ($client['disable']) + if (isset($client['disable'])) $disabled = "YES"; $server = "{$client['server_addr']}:{$client['server_port']}"; ?> -- cgit v1.1