Skip to content

Commit d876346

Browse files
birdiecodegpotter2
andauthored
Fix: replace data variable with trigger variable in EAPOL_KEY.guess_key_number (#4703)
* Fix: replace data variable with trigger variable in [EAPOL_KEY.guess_key_number] * Update EAPOL-Key 1 test --------- Co-authored-by: gpotter2 <10530980+gpotter2@users.noreply.github.com>
1 parent d215192 commit d876346

2 files changed

Lines changed: 2 additions & 2 deletions

File tree

scapy/layers/eap.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -494,7 +494,7 @@ def guess_key_number(self):
494494
"""
495495
if self.key_type == 1:
496496
if self.key_ack == 1:
497-
if self.key_mic == 0:
497+
if self.has_key_mic == 0:
498498
return 1
499499
if self.install == 1:
500500
return 3

test/scapy/layers/eap.uts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,7 @@ assert(eapol_key.key_ack == 1)
8686
assert(eapol_key.key_mic == b"\x00" * 16)
8787
assert(eapol_key.secure == 0)
8888
assert(eapol_key.key_data_length == 22)
89-
assert(eapol_key.guess_key_number() == 0)
89+
assert(eapol_key.guess_key_number() == 1)
9090

9191
= EAPOL_KEY - Key 2 - Dissection (2)
9292
s = b'\x02\x03\x00\x75\x02\x01\x0a\x00\x10\x00\x00\x00\x00\x00\x00\x00\x00\x60\x5e\x85\xa7\x9c\xfa\xfd\xb0\xea\xa0\x50\x68\x3f\x97\xbe\x1b\x66\xde\xf7\xbc\x65\x20\x57\x31\x68\x71\xc2\x73\xc5\xae\x47\x7f\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x91\x89\xcd\xf1\x88\x54\x8e\x73\xcd\x37\xd5\x78\x52\x66\x05\x88\x00\x16\x30\x14\x01\x00\x00\x0f\xac\x04\x01\x00\x00\x0f\xac\x04\x01\x00\x00\x0f\xac\x02\x28\x00'

0 commit comments

Comments
 (0)