[PATCH 11/17] tests: ap_tdls: wait before connectivity checks
Johannes Berg
johannes at sipsolutions.net
Mon Sep 25 00:20:51 PDT 2023
From: Johannes Berg <johannes.berg at intel.com>
All processes need to have a bit of time to mark the
kernel STAs authorized, otherwise traffic may fail.
Give them some time, and also use check_connectivity()
in connectivity() since it's the same check, just
different arguments.
Signed-off-by: Johannes Berg <johannes.berg at intel.com>
---
tests/hwsim/test_ap_tdls.py | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/tests/hwsim/test_ap_tdls.py b/tests/hwsim/test_ap_tdls.py
index 637dd3e531cc..2b6a440b0cf7 100644
--- a/tests/hwsim/test_ap_tdls.py
+++ b/tests/hwsim/test_ap_tdls.py
@@ -22,9 +22,7 @@ def start_ap_wpa2_psk(ap):
return hostapd.add_ap(ap, params)
def connectivity(dev, hapd):
- hwsim_utils.test_connectivity_sta(dev[0], dev[1])
- hwsim_utils.test_connectivity(dev[0], hapd)
- hwsim_utils.test_connectivity(dev[1], hapd)
+ check_connectivity(dev[0], dev[1], hapd)
def connect_2sta(dev, ssid, hapd, sae=False):
key_mgmt = "SAE" if sae else "WPA-PSK"
@@ -112,6 +110,8 @@ def tdls_check_ap(sta0, sta1, bssid, addr0, addr1):
raise Exception("Invalid frames through AP path")
def check_connectivity(sta0, sta1, hapd):
+ # give a bit of time for all sides to mark kernel STAs authorized
+ time.sleep(0.1)
hwsim_utils.test_connectivity_sta(sta0, sta1)
hwsim_utils.test_connectivity(sta0, hapd)
hwsim_utils.test_connectivity(sta1, hapd)
@@ -358,6 +358,8 @@ def test_ap_wpa2_tdls_bssid_mismatch(dev, apdev):
bssid=apdev[0]['bssid'])
dev[1].connect(ssid, psk=passphrase, scan_freq="2412",
bssid=apdev[1]['bssid'])
+ # give a bit of time for all sides to mark kernel STAs authorized
+ time.sleep(0.1)
hwsim_utils.test_connectivity_sta(dev[0], dev[1])
hwsim_utils.test_connectivity_iface(dev[0], hapd, "ap-br0")
hwsim_utils.test_connectivity_iface(dev[1], hapd, "ap-br0")
--
2.41.0
More information about the Hostap
mailing list