]> git.ipfire.org Git - thirdparty/hostap.git/commit - src/ap/drv_callbacks.c
PMF: Do not start SA Query procedure if there is no association
authorAshok Kumar <aponnaia@codeaurora.org>
Fri, 12 Oct 2018 04:06:05 +0000 (09:36 +0530)
committerJouni Malinen <j@w1.fi>
Mon, 15 Oct 2018 21:33:20 +0000 (00:33 +0300)
commitedb28006c4d199918d50a08f8c592bd184c22624
treebc653137300b1cfb45b35996fc45d98874645422
parent97e27300f49e44cd8fe5b3758376f66bd4d45cf0
PMF: Do not start SA Query procedure if there is no association

Previous implementation ended up triggering PMF check for previous
association and SA Query procedure incorrectly in cases where there is a
STA entry in hostapd, but that STA is not in associated state. This
resulted in undesired temporary rejection of the association with status
code 30.

This ended up breaking OWE group negotiation when PMF is in use since
the check for the OWE group would have happened only after this earlier
PMF check and rejection (i.e., the station got status code 30 instead of
the expected 77).

For example, when the AP is configured with OWE group 21 and a station
tries groups 19, 20, and 21 (in this sequence), the first two
Association Request frames should be rejected with status code 77.
However, only the first one got that status code while the second one
got status code 30 due to that issue with PMF existing association
check.

Furthermore, hostapd was continuing with SA Query procedure with
unencrypted Action frames in this type of case even though there was no
existing association (and obviously, not an encryption key either).

Fix this by checking that the STA entry is in associated state before
initiating SA Query procedure based on the PMF rules.

Signed-off-by: Ashok Kumar <aponnaia@codeaurora.org>
src/ap/drv_callbacks.c
src/ap/ieee802_11.c