From 3918c4123f06adbe92cb249db21fa0426b814bc7 Mon Sep 17 00:00:00 2001 From: Johannes Berg Date: Mon, 25 Sep 2023 09:20:55 +0200 Subject: [PATCH] tests: ap_open_poll_sta_no_ack: Fix mgmt-rx race 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 --- 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 a3bea763a..beb0e1818 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")