diff options
author | Ermal LUÇI <eri@pfsense.org> | 2015-01-12 09:27:09 +0100 |
---|---|---|
committer | Ermal LUÇI <eri@pfsense.org> | 2015-01-12 09:27:09 +0100 |
commit | 64ed3e60f518949adc1cc54b4ea77c618afc130c (patch) | |
tree | b25ef3c270425e0d432944f3a374f89409e755fb /usr | |
parent | ab54ec9f8d362162ada3055e99143be6ea83b5ce (diff) | |
download | pfsense-64ed3e60f518949adc1cc54b4ea77c618afc130c.zip pfsense-64ed3e60f518949adc1cc54b4ea77c618afc130c.tar.gz |
Fix inherent issues with isset and empty values set as true by our parser. This made the piep configuration to be wrong at least for passthrough entries. Ticket #3932
Diffstat (limited to 'usr')
-rw-r--r-- | usr/local/www/services_captiveportal.php | 10 |
1 files changed, 8 insertions, 2 deletions
diff --git a/usr/local/www/services_captiveportal.php b/usr/local/www/services_captiveportal.php index c88f6af..8c1487b 100644 --- a/usr/local/www/services_captiveportal.php +++ b/usr/local/www/services_captiveportal.php @@ -339,8 +339,14 @@ if ($_POST) { $newcp['preauthurl'] = $_POST['preauthurl']; $newcp['blockedmacsurl'] = $_POST['blockedmacsurl']; $newcp['peruserbw'] = $_POST['peruserbw'] ? true : false; - $newcp['bwdefaultdn'] = $_POST['bwdefaultdn']; - $newcp['bwdefaultup'] = $_POST['bwdefaultup']; + if (isset($_POST['bwdefaultdn'])) + $newcp['bwdefaultdn'] = $_POST['bwdefaultdn']; + else + unset($newcp['bwdefaultdn']); + if (isset($_POST['bwdefaultup'])) + $newcp['bwdefaultup'] = $_POST['bwdefaultup']; + else + unset($newcp['bwdefaultup']); $newcp['certref'] = $_POST['certref']; $newcp['nohttpsforwards'] = $_POST['nohttpsforwards'] ? true : false; $newcp['logoutwin_enable'] = $_POST['logoutwin_enable'] ? true : false; |