DGNum/hostapd
6
0
Fork 0
Code Issues Pull requests 1 Projects Releases Packages Wiki Activity Actions

tests: Make pmksa_cache_{,opportunistic_}multiple_sta more robust

Browse source 
Clear wpa_supplicant scan cache before starting these test cases since
the ROAM command depends on the correct BSS entry being found.
scan_for_bss() does not enforce that correct entry to be present if
there was an earlier BSS entry with the same BSSID.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
This commit is contained in:
Jouni Malinen 2016-09-23 16:02:46 +03:00 • committed by Jouni Malinen
parent 78a37200f8
commit 1c82fbf271
1 changed files with 10 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
tests/hwsim/test_pmksa_cache.py
View file

@ -599,6 +599,8 @@ def test_pmksa_cache_multiple_sta(dev, apdev):
params = hostapd.wpa2_eap_params(ssid="test-pmksa-cache") params = hostapd.wpa2_eap_params(ssid="test-pmksa-cache")
hostapd.add_ap(apdev[0], params) hostapd.add_ap(apdev[0], params)
bssid = apdev[0]['bssid'] bssid = apdev[0]['bssid']
for d in dev:
d.flush_scan_cache()
dev[0].connect("test-pmksa-cache", proto="RSN", key_mgmt="WPA-EAP", dev[0].connect("test-pmksa-cache", proto="RSN", key_mgmt="WPA-EAP",
eap="GPSK", identity="gpsk-user-session-timeout", eap="GPSK", identity="gpsk-user-session-timeout",
password="abcdefghijklmnop0123456789abcdef", password="abcdefghijklmnop0123456789abcdef",
@ -614,6 +616,7 @@ def test_pmksa_cache_multiple_sta(dev, apdev):
wpas = WpaSupplicant(global_iface='/tmp/wpas-wlan5') wpas = WpaSupplicant(global_iface='/tmp/wpas-wlan5')
wpas.interface_add("wlan5") wpas.interface_add("wlan5")
wpas.flush_scan_cache()
wpas.connect("test-pmksa-cache", proto="RSN", key_mgmt="WPA-EAP", wpas.connect("test-pmksa-cache", proto="RSN", key_mgmt="WPA-EAP",
eap="GPSK", identity="gpsk user", eap="GPSK", identity="gpsk user",
password="abcdefghijklmnop0123456789abcdef", password="abcdefghijklmnop0123456789abcdef",
@ -626,11 +629,13 @@ def test_pmksa_cache_multiple_sta(dev, apdev):
for sta in [ dev[1], dev[0], dev[2], wpas ]: for sta in [ dev[1], dev[0], dev[2], wpas ]:
sta.dump_monitor() sta.dump_monitor()
sta.scan_for_bss(bssid2, freq="2412") sta.scan_for_bss(bssid2, freq="2412")
sta.request("ROAM " + bssid2) if "OK" not in sta.request("ROAM " + bssid2):
raise Exception("ROAM command failed (" + sta.ifname + ")")
ev = sta.wait_event(["CTRL-EVENT-EAP-SUCCESS"], timeout=10) ev = sta.wait_event(["CTRL-EVENT-EAP-SUCCESS"], timeout=10)
if ev is None: if ev is None:
raise Exception("EAP success timed out") raise Exception("EAP success timed out")
sta.wait_connected(timeout=10, error="Roaming timed out") sta.wait_connected(timeout=10, error="Roaming timed out")
sta.dump_monitor()
logger.info("Roam back to AP1") logger.info("Roam back to AP1")
for sta in [ dev[1], wpas, dev[0], dev[2] ]: for sta in [ dev[1], wpas, dev[0], dev[2] ]:
@ -658,8 +663,11 @@ def test_pmksa_cache_opportunistic_multiple_sta(dev, apdev):
params['okc'] = "1" params['okc'] = "1"
hostapd.add_ap(apdev[0], params) hostapd.add_ap(apdev[0], params)
bssid = apdev[0]['bssid'] bssid = apdev[0]['bssid']
for d in dev:
d.flush_scan_cache()
wpas = WpaSupplicant(global_iface='/tmp/wpas-wlan5') wpas = WpaSupplicant(global_iface='/tmp/wpas-wlan5')
wpas.interface_add("wlan5") wpas.interface_add("wlan5")
wpas.flush_scan_cache()
for sta in [ dev[0], dev[1], dev[2], wpas ]: for sta in [ dev[0], dev[1], dev[2], wpas ]:
sta.connect("test-pmksa-cache", proto="RSN", key_mgmt="WPA-EAP", sta.connect("test-pmksa-cache", proto="RSN", key_mgmt="WPA-EAP",
eap="GPSK", identity="gpsk user", eap="GPSK", identity="gpsk user",
@ -684,6 +692,7 @@ def test_pmksa_cache_opportunistic_multiple_sta(dev, apdev):
pmksa2 = sta.get_pmksa(bssid2) pmksa2 = sta.get_pmksa(bssid2)
if pmksa2 is None: if pmksa2 is None:
raise Exception("No PMKSA cache entry created") raise Exception("No PMKSA cache entry created")
sta.dump_monitor()
logger.info("Roam back to AP1") logger.info("Roam back to AP1")
for sta in [ dev[0], dev[1], dev[2], wpas ]: for sta in [ dev[0], dev[1], dev[2], wpas ]: