diff options
| author | Phil Davis <phil.davis@inf.org> | 2015-12-17 08:27:38 +0545 |
|---|---|---|
| committer | Stephen Beaver <sbeaver@netgate.com> | 2015-12-17 07:32:49 -0500 |
| commit | c1ac389132480d6913723bc25f0edc54f43a56c9 (patch) | |
| tree | 711bc2337dbbc88ef66296c92b3a13601bae365b /src | |
| parent | 0cf1b4f4f29f2ad63691d598e83a9327ae9867ff (diff) | |
| download | pfsense-c1ac389132480d6913723bc25f0edc54f43a56c9.zip pfsense-c1ac389132480d6913723bc25f0edc54f43a56c9.tar.gz | |
services_pppoe_edit handle both GET and POST
This wacky logic would only have got to the $_POST checks if $_GET['id'] was already set.
I haven't looked to analyse it - but how would anything using $_POST have worked?
Diffstat (limited to 'src')
| -rw-r--r-- | src/usr/local/www/services_pppoe_edit.php | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/src/usr/local/www/services_pppoe_edit.php b/src/usr/local/www/services_pppoe_edit.php index f358749..6da30c6 100644 --- a/src/usr/local/www/services_pppoe_edit.php +++ b/src/usr/local/www/services_pppoe_edit.php @@ -88,8 +88,8 @@ $a_pppoes = &$config['pppoes']['pppoe']; if (is_numericint($_GET['id'])) { $id = $_GET['id']; - -if (isset($_POST['id']) && is_numericint($_POST['id'])) +} +if (isset($_POST['id']) && is_numericint($_POST['id'])) { $id = $_POST['id']; } |
