jeltz/hostap
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

EAP-PAX: Do not debug print result if eap_pax_mac() fails

Browse source 
This gets rid of a valgrind warning on uninitialized memory read in the
eap_proto_pax_errors test case where the result was used after the
failed eap_pax_mac() call.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
This commit is contained in:
Jouni Malinen 2016-05-16 22:25:25 +03:00 committed by Jouni Malinen
parent 92abe3782f
commit 0884633577
1 changed files with 9 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
src/eap_peer/eap_pax.c
View file

@ -278,8 +278,15 @@ static struct wpabuf * eap_pax_process_std_3(struct eap_pax_data *data,
pos, EAP_PAX_MAC_LEN);
if (eap_pax_mac(data->mac_id, data->ck, EAP_PAX_CK_LEN,
data->rand.r.y, EAP_PAX_RAND_LEN,
(u8 *) data->cid, data->cid_len, NULL, 0, mac) < 0 ||
os_memcmp_const(pos, mac, EAP_PAX_MAC_LEN) != 0) {
(u8 *) data->cid, data->cid_len, NULL, 0, mac) < 0) {
wpa_printf(MSG_INFO,
"EAP-PAX: Could not derive MAC_CK(B, CID)");
ret->methodState = METHOD_DONE;
ret->decision = DECISION_FAIL;
return NULL;
}
if (os_memcmp_const(pos, mac, EAP_PAX_MAC_LEN) != 0) {
wpa_printf(MSG_INFO, "EAP-PAX: Invalid MAC_CK(B, CID) "
"received");
wpa_hexdump(MSG_MSGDUMP, "EAP-PAX: expected MAC_CK(B, CID)",