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 <jouni@codeaurora.org>
This commit is contained in:
Jouni Malinen 2019-12-06 17:26:57 +02:00 committed by Jouni Malinen
parent 405946d76a
commit 918df2227e

View file

@ -1122,8 +1122,10 @@ static void handle_auth_sae(struct hostapd_data *hapd, struct sta_info *sta,
if (!sta->sae) { if (!sta->sae) {
if (auth_transaction != 1 || if (auth_transaction != 1 ||
!sae_status_success(hapd, status_code)) { !sae_status_success(hapd, status_code)) {
resp = -1; wpa_printf(MSG_DEBUG, "SAE: Unexpected Status Code %u",
goto remove_sta; status_code);
resp = WLAN_STATUS_UNSPECIFIED_FAILURE;
goto reply;
} }
sta->sae = os_zalloc(sizeof(*sta->sae)); sta->sae = os_zalloc(sizeof(*sta->sae));
if (!sta->sae) { if (!sta->sae) {