From 918df2227efd6c629799819ba4b61802cf68bf9c Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Fri, 6 Dec 2019 17:26:57 +0200 Subject: [PATCH] SAE: Reject unexpected Status Code in SAE commit explicitly (AP) Previously, this case was ignored silently in AP mode. While that could be a reasonable approach for an unexpected condition, it would be fine to reject this case explicitly as well. This makes it somewhat easier to test unexpected SAE H2E vs. looping behavior. Signed-off-by: Jouni Malinen --- src/ap/ieee802_11.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/ap/ieee802_11.c b/src/ap/ieee802_11.c index 07723d0cf..6a638f090 100644 --- a/src/ap/ieee802_11.c +++ b/src/ap/ieee802_11.c @@ -1122,8 +1122,10 @@ static void handle_auth_sae(struct hostapd_data *hapd, struct sta_info *sta, if (!sta->sae) { if (auth_transaction != 1 || !sae_status_success(hapd, status_code)) { - resp = -1; - goto remove_sta; + wpa_printf(MSG_DEBUG, "SAE: Unexpected Status Code %u", + status_code); + resp = WLAN_STATUS_UNSPECIFIED_FAILURE; + goto reply; } sta->sae = os_zalloc(sizeof(*sta->sae)); if (!sta->sae) {