[PATCH 15/17] tests: test_ap_open_poll_sta_no_ack: fix mgmt-rx race

Johannes Berg johannes at sipsolutions.net
Mon Sep 25 00:20:55 PDT 2023


From: Johannes Berg <johannes.berg at intel.com>

We need to wait for the MGMT-RX event before disabling
ext_mgmt_frame_handling again, otherwise we might be
disabling it and hostapd only receives the deauth frame
after we already disable it, defeating the purpose of
the test.

Signed-off-by: Johannes Berg <johannes.berg at intel.com>
---
 tests/hwsim/test_ap_open.py | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tests/hwsim/test_ap_open.py b/tests/hwsim/test_ap_open.py
index a3bea763a1c4..beb0e18187b5 100644
--- a/tests/hwsim/test_ap_open.py
+++ b/tests/hwsim/test_ap_open.py
@@ -725,6 +725,9 @@ def test_ap_open_poll_sta_no_ack(dev, apdev):
     hapd.set("ext_mgmt_frame_handling", "1")
     dev[0].request("DISCONNECT")
     dev[0].wait_disconnected()
+    # eat up the deauth frame, so it cannot be processed
+    # after we disable ext_mgmt_frame_handling again
+    hapd.wait_event(["MGMT-RX"], timeout=1)
     hapd.set("ext_mgmt_frame_handling", "0")
     if "OK" not in hapd.request("POLL_STA " + addr):
         raise Exception("POLL_STA failed")
-- 
2.41.0




More information about the Hostap mailing list