From fd35ed5bba97565d35567d0723f841be9936280e Mon Sep 17 00:00:00 2001 From: Ilan Peer Date: Mon, 21 Aug 2017 19:36:21 +0300 Subject: [PATCH] AP: Remove unneeded check for 'added_unassociated' 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 --- 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 4252fccaa..5163139f5 100644 --- a/src/ap/ieee802_11.c +++ b/src/ap/ieee802_11.c @@ -2860,14 +2860,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; - } }