diff options
| author | Jouni Malinen <jouni@codeaurora.org> | 2019-12-06 15:26:57 (GMT) |
|---|---|---|
| committer | Jouni Malinen <j@w1.fi> | 2019-12-06 15:26:57 (GMT) |
| commit | 918df2227efd6c629799819ba4b61802cf68bf9c (patch) | |
| tree | ead335bf9de843b5307be5d5c8278f958ee48d73 | |
| parent | 405946d76a6e438514f59dd17ad44b6fb698af15 (diff) | |
| download | hostap-918df2227efd6c629799819ba4b61802cf68bf9c.zip hostap-918df2227efd6c629799819ba4b61802cf68bf9c.tar.gz hostap-918df2227efd6c629799819ba4b61802cf68bf9c.tar.bz2 | |
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>
| -rw-r--r-- | src/ap/ieee802_11.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/src/ap/ieee802_11.c b/src/ap/ieee802_11.c index 07723d0..6a638f0 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) { |