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

tests: Do not check TX status in dpp_qr_code_auth_neg_chan

Browse source 
This avoids race conditions with UML time-travel where some cases ended
up not reporting the DPP-TX-STATUS event and that resulted in discarding
events that are more important to verify for correct behavior.

Signed-off-by: Jouni Malinen <quic_jouni@quicinc.com>
This commit is contained in:
Jouni Malinen 2023-12-20 12:46:05 +02:00 committed by Jouni Malinen
parent ecb1e04d85
commit 54ab50d658
1 changed files with 0 additions and 12 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
tests/hwsim/test_dpp.py
View file

@ -879,12 +879,6 @@ def test_dpp_qr_code_auth_neg_chan(dev, apdev):
if "freq=2462 type=1" not in ev: if "freq=2462 type=1" not in ev:
raise Exception("Unexpected RX data for Authentication Response: " + ev) raise Exception("Unexpected RX data for Authentication Response: " + ev)
ev = dev[0].wait_event(["DPP-TX-STATUS"], timeout=5)
if ev is None:
raise Exception("TX status for DPP Authentication Response not reported")
if "freq=2462 result=SUCCESS" not in ev:
raise Exception("Unexpected TX status for Authentication Response: " + ev)
ev = dev[1].wait_event(["DPP-TX "], timeout=5) ev = dev[1].wait_event(["DPP-TX "], timeout=5)
if ev is None: if ev is None:
raise Exception("DPP Authentication Confirm not sent") raise Exception("DPP Authentication Confirm not sent")
@ -897,12 +891,6 @@ def test_dpp_qr_code_auth_neg_chan(dev, apdev):
if "freq=2462 type=2" not in ev: if "freq=2462 type=2" not in ev:
raise Exception("Unexpected RX data for Authentication Confirm: " + ev) raise Exception("Unexpected RX data for Authentication Confirm: " + ev)
ev = dev[1].wait_event(["DPP-TX-STATUS"], timeout=5)
if ev is None:
raise Exception("TX status for DPP Authentication Confirm not reported")
if "freq=2462 result=SUCCESS" not in ev:
raise Exception("Unexpected TX status for Authentication Confirm: " + ev)
wait_auth_success(dev[0], dev[1], configurator=dev[1], enrollee=dev[0], wait_auth_success(dev[0], dev[1], configurator=dev[1], enrollee=dev[0],
stop_responder=True) stop_responder=True)