From bd7f8bf61325d4fcd069cec9a235f87b993b3642 Mon Sep 17 00:00:00 2001 From: Denis Kenzior Date: Thu, 7 Feb 2019 09:54:51 -0600 Subject: [PATCH] eapol: Relax message 2 / 4 key_length check Seems certain Apple implementations set the key length to 16, even though it should be 0. --- src/eapol.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/src/eapol.c b/src/eapol.c index 659cc825..7922ad0a 100644 --- a/src/eapol.c +++ b/src/eapol.c @@ -430,8 +430,7 @@ bool eapol_verify_ptk_2_of_4(const struct eapol_key *ek) return false; key_len = L_BE16_TO_CPU(ek->key_length); - if (key_len != 0) - return false; + L_WARN_ON(key_len != 0); VERIFY_IS_ZERO(ek->eapol_key_iv); VERIFY_IS_ZERO(ek->key_rsc); @@ -510,8 +509,7 @@ bool eapol_verify_ptk_4_of_4(const struct eapol_key *ek, bool is_wpa) return false; key_len = L_BE16_TO_CPU(ek->key_length); - if (key_len != 0) - return false; + L_WARN_ON(key_len != 0); VERIFY_IS_ZERO(ek->key_nonce); VERIFY_IS_ZERO(ek->eapol_key_iv);