diff mbox series

[2/2] AP: Always include WPA_STA_AUTHORIZED in station flags mask

Message ID 20231128101410.1931280-2-andrei.otcheretianski@intel.com
State Accepted
Headers show
Series [1/2] AP: MLD: Do not modify flags for link stations | expand

Commit Message

Andrei Otcheretianski Nov. 28, 2023, 10:14 a.m. UTC 
From: Ilan Peer <ilan.peer@intel.com>

As otherwise, the underlying driver would not handle a change
in the 'authorized' state.

Signed-off-by: Ilan Peer <ilan.peer@intel.com>
---
 src/ap/ap_drv_ops.c | 7 ++-----
 1 file changed, 2 insertions(+), 5 deletions(-)
diff mbox series

Patch

diff --git a/src/ap/ap_drv_ops.c b/src/ap/ap_drv_ops.c
index d303f932f7..f4e1dd9269 100644
--- a/src/ap/ap_drv_ops.c
+++ b/src/ap/ap_drv_ops.c
@@ -319,7 +319,8 @@  int hostapd_set_sta_flags(struct hostapd_data *hapd, struct sta_info *sta)
 	bool is_link_sta = hostapd_sta_is_link_sta(hapd, sta);
 
 	total_flags = hostapd_sta_flags_to_drv(sta->flags);
-	set_flags = WPA_STA_SHORT_PREAMBLE | WPA_STA_WMM | WPA_STA_MFP;
+	set_flags = WPA_STA_SHORT_PREAMBLE | WPA_STA_WMM | WPA_STA_MFP |
+		WPA_STA_AUTHORIZED;
 
 	/*
 	 * All the station flags other than WPA_STA_SHORT_PREAMBLE are relevant
@@ -336,10 +337,6 @@  int hostapd_set_sta_flags(struct hostapd_data *hapd, struct sta_info *sta)
 		return 0;
 	}
 
-	if (((!hapd->conf->ieee802_1x && !hapd->conf->wpa) ||
-	     sta->auth_alg == WLAN_AUTH_FT) &&
-	    sta->flags & WLAN_STA_AUTHORIZED)
-		set_flags |= WPA_STA_AUTHORIZED;
 	flags_or = total_flags & set_flags;
 	flags_and = total_flags | ~set_flags;
 	return hostapd_sta_set_flags(hapd, sta->addr, total_flags,