diff options
| author | Phil Davis <phil.davis@inf.org> | 2015-03-13 07:30:52 +0545 |
|---|---|---|
| committer | Chris Buechler <cmb@pfsense.org> | 2015-03-12 22:03:09 -0500 |
| commit | 580f4f3f835487644786251579b5c58557448ac7 (patch) | |
| tree | 7fb77e3e256bedf1a5f80363a64c26c85a9736b1 | |
| parent | ec253cd500088b4257ade0b63b31e5ff9ef3d29a (diff) | |
| download | pfsense-580f4f3f835487644786251579b5c58557448ac7.zip pfsense-580f4f3f835487644786251579b5c58557448ac7.tar.gz | |
Missin double equals in captiveportal.inc
Looking at where this is nested inside various if statements, I do not think this error did too much harm - only to the $mac['descr'] - in this particular code flow $username is not used for important stuff after this point.
Conflicts:
etc/inc/captiveportal.inc
| -rw-r--r-- | etc/inc/captiveportal.inc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/etc/inc/captiveportal.inc b/etc/inc/captiveportal.inc index f6c0930..98dbfc9 100644 --- a/etc/inc/captiveportal.inc +++ b/etc/inc/captiveportal.inc @@ -1962,7 +1962,7 @@ function portal_allow($clientip,$clientmac,$username,$password = null, $attribut if ($attributes['voucher']) $mac['logintype'] = "voucher"; } - if ($username = "unauthenticated") + if ($username == "unauthenticated") $mac['descr'] = "Auto-added"; else $mac['descr'] = "Auto-added for user {$username}"; |
