tests: Enforce proper OCV behavior for SA Query Response from STA
Now that there is a pending mac80211 patch ("mac80211: allow SA-QUERY processing in userspace") to allow wpa_supplicant to take care of SA Query Request processing, start enforcing correct behavior for this in ocv_sa_query and wpa2_ocv_sta_override_sa_query_resp. Signed-off-by: Jouni Malinen <jouni@codeaurora.org>
This commit is contained in:
parent
88436baaac
commit
9818fbe9bc
2 changed files with 4 additions and 8 deletions
|
@ -90,7 +90,9 @@ def test_ocv_sa_query(dev, apdev):
|
||||||
# Test that client can handle SA Query with OCI element
|
# Test that client can handle SA Query with OCI element
|
||||||
if "OK" not in hapd.request("SA_QUERY " + dev[0].own_addr()):
|
if "OK" not in hapd.request("SA_QUERY " + dev[0].own_addr()):
|
||||||
raise Exception("SA_QUERY failed")
|
raise Exception("SA_QUERY failed")
|
||||||
time.sleep(0.1)
|
ev = hapd.wait_event(["OCV-FAILURE"], timeout=0.1)
|
||||||
|
if ev:
|
||||||
|
raise Exception("Unexpected OCV failure reported")
|
||||||
if wt.get_sta_counter("valid_saqueryresp_tx", apdev[0]['bssid'],
|
if wt.get_sta_counter("valid_saqueryresp_tx", apdev[0]['bssid'],
|
||||||
dev[0].own_addr()) < 1:
|
dev[0].own_addr()) < 1:
|
||||||
raise Exception("STA did not reply to SA Query")
|
raise Exception("STA did not reply to SA Query")
|
||||||
|
|
|
@ -985,10 +985,4 @@ def test_wpa2_ocv_sta_override_sa_query_resp(dev, apdev):
|
||||||
if "frame=saqueryresp" not in ev:
|
if "frame=saqueryresp" not in ev:
|
||||||
raise Exception("Unexpected OCV failure frame: " + ev)
|
raise Exception("Unexpected OCV failure frame: " + ev)
|
||||||
if "error=primary channel mismatch" not in ev:
|
if "error=primary channel mismatch" not in ev:
|
||||||
if "error=did not receive mandatory OCI" in ev:
|
|
||||||
# This is not correct, but do not report this as test failure for
|
|
||||||
# now since the issue seems to be in mac80211 not allowing
|
|
||||||
# wpa_supplicant to process the SA Query Request frame.
|
|
||||||
logger.info("Unexpected OCV failure error: " + ev)
|
|
||||||
else:
|
|
||||||
raise Exception("Unexpected OCV failure error: " + ev)
|
raise Exception("Unexpected OCV failure error: " + ev)
|
||||||
|
|
Loading…
Reference in a new issue