From b59977262bbc7c13991b859a4efcfb45e9e74e11 Mon Sep 17 00:00:00 2001 From: Denis Kenzior Date: Mon, 15 May 2017 15:23:03 -0500 Subject: [PATCH] handshake: Fix valgrind warning ==27901== Conditional jump or move depends on uninitialised value(s) ==27901== at 0x41157A: handshake_util_find_pmkid_kde (handshake.c:537) ==27901== by 0x40E03A: eapol_handle_ptk_1_of_4 (eapol.c:852) ==27901== by 0x40F3CD: eapol_key_handle (eapol.c:1417) ==27901== by 0x40F955: eapol_rx_packet (eapol.c:1607) ==27901== by 0x410321: __eapol_rx_packet (eapol.c:1915) --- src/handshake.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/handshake.c b/src/handshake.c index 2c705915..2079be50 100644 --- a/src/handshake.c +++ b/src/handshake.c @@ -534,7 +534,7 @@ const uint8_t *handshake_util_find_pmkid_kde(const uint8_t *data, pmkid = find_kde(data, data_len, &pmkid_len, pmkid_oui); - if (pmkid_len != 16) + if (pmkid && pmkid_len != 16) return NULL; return pmkid;