[PATCH] hostapd_acl_recv_radius: tagged is a pointer, not an int

Derrick Pallas pallas at meraki.com
Fri Nov 4 17:06:40 PDT 2016


Discovered & fixed by Louisa Chong, `tagged` is an `int*`, not an `int`, so
we actually want to take the address of `cache->vlan_id.tagged` here.

Signed-off-by: Louisa Chong <clouisa at meraki.com>
Signed-off-by: Derrick Pallas <pallas at meraki.com>
---
 src/ap/ieee802_11_auth.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/ap/ieee802_11_auth.c b/src/ap/ieee802_11_auth.c
index b890537..86230c9 100644
--- a/src/ap/ieee802_11_auth.c
+++ b/src/ap/ieee802_11_auth.c
@@ -569,7 +569,7 @@ hostapd_acl_recv_radius(struct radius_msg *msg, struct radius_msg *req,
 
 		notempty = &cache->vlan_id.notempty;
 		untagged = &cache->vlan_id.untagged;
-		tagged = cache->vlan_id.tagged;
+		tagged = &cache->vlan_id.tagged;
 		*notempty = !!radius_msg_get_vlanid(msg, untagged,
 						    MAX_NUM_TAGGED_VLAN,
 						    tagged);
-- 
2.10.1




More information about the Hostap mailing list