[PATCH] AP: Remove unneeded check for 'added_unassociated'
Andrei Otcheretianski
andrei.otcheretianski at intel.com
Mon Aug 21 09:36:21 PDT 2017
From: Ilan Peer <ilan.peer at intel.com>
In fils_hlp_finish_assoc() the station is already added to the
driver so it is not needed to check the 'added_unassociated'
flag.
Signed-off-by: Ilan Peer <ilan.peer at intel.com>
---
src/ap/ieee802_11.c | 9 +++------
1 file changed, 3 insertions(+), 6 deletions(-)
diff --git a/src/ap/ieee802_11.c b/src/ap/ieee802_11.c
index 6f51214..f061839 100644
--- a/src/ap/ieee802_11.c
+++ b/src/ap/ieee802_11.c
@@ -2837,14 +2837,11 @@ void fils_hlp_finish_assoc(struct hostapd_data *hapd, struct sta_info *sta)
sta->hlp_dhcp_discover = NULL;
/*
- * Remove the station in case tranmission of a success response fails
- * (the STA was added associated to the driver) or if the station was
- * previously added unassociated.
+ * Remove the station in case transmission of a success response fails.
+ * At this point the station was already added associated to the driver.
*/
- if (reply_res != WLAN_STATUS_SUCCESS || sta->added_unassoc) {
+ if (reply_res != WLAN_STATUS_SUCCESS)
hostapd_drv_sta_remove(hapd, sta->addr);
- sta->added_unassoc = 0;
- }
}
--
2.7.4
More information about the Hostap
mailing list