From 3cefdf0d4a6bd3bb988912ef70669bbad33b570d Mon Sep 17 00:00:00 2001 From: Jouni Malinen Date: Thu, 6 May 2021 23:49:12 +0300 Subject: [PATCH] tests: Fix eap_proto_sake_server with Session ID = 255 Need to wrap back to 0 when changing value 255 to avoid generating a too large value to fit an octet field. This was resulting in errors due to a python exception (likely for about every 256th run). Signed-off-by: Jouni Malinen --- tests/hwsim/test_eap_proto.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tests/hwsim/test_eap_proto.py b/tests/hwsim/test_eap_proto.py index afdc45d70..a8f4aeb36 100644 --- a/tests/hwsim/test_eap_proto.py +++ b/tests/hwsim/test_eap_proto.py @@ -983,7 +983,7 @@ def test_eap_proto_sake_server(dev, apdev): # Unknown session # --> EAP-SAKE: Session ID mismatch sess, = struct.unpack('B', binascii.unhexlify(resp[20:22])) - sess = binascii.hexlify(struct.pack('B', sess + 1)).decode() + sess = binascii.hexlify(struct.pack('B', (sess + 1) % 256)).decode() msg = resp[0:4] + "0008" + resp[8:12] + "0008" + "3002" + sess + "00" tx_msg(dev[0], hapd, msg) # Unknown subtype