From cc229ee94a897423d6eac771eefe8684c2fa1c2c Mon Sep 17 00:00:00 2001
From: Phil Davis <phil.davis@inf.org>
Date: Fri, 13 Mar 2015 07:30:52 +0545
Subject: [PATCH] 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.
---
 etc/inc/captiveportal.inc | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/etc/inc/captiveportal.inc b/etc/inc/captiveportal.inc
index 002195f102..cbc197e16e 100644
--- a/etc/inc/captiveportal.inc
+++ b/etc/inc/captiveportal.inc
@@ -2068,7 +2068,7 @@ function portal_allow($clientip,$clientmac,$username,$password = null, $attribut
 					$mac['logintype'] = "voucher";
 				}
 			}
-			if ($username = "unauthenticated") {
+			if ($username == "unauthenticated") {
 				$mac['descr'] =  "Auto-added";
 			} else {
 				$mac['descr'] =  "Auto-added for user {$username}";