Amazing-Mirror/fragattacks
1
0
Fork 0
mirror of https://github.com/vanhoefm/fragattacks.git synced 2025-01-31 01:04:03 -05:00
Code Issues Projects Releases Wiki Activity

EAPOL: Fix static analyzer warnings for pac_opaque_encr_key

Browse Source 
The allocation was not verified to complete successfully and the
allocated memory was not freed on error paths.

Signed-hostap: Adriana Reus <adriana.reus@intel.com>
This commit is contained in:
Adriana Reus 2013-11-03 15:21:30 +02:00 committed by Jouni Malinen
parent 3139270903
commit 72c12c1d30
1 changed files with 6 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/eapol_auth/eapol_auth_sm.c
View File

@ -1062,6 +1062,10 @@ static int eapol_auth_conf_clone(struct eapol_auth_config *dst,
} }
if (src->pac_opaque_encr_key) { if (src->pac_opaque_encr_key) {
dst->pac_opaque_encr_key = os_malloc(16); dst->pac_opaque_encr_key = os_malloc(16);
if (dst->pac_opaque_encr_key == NULL) {
os_free(dst->eap_req_id_text);
return -1;
}
os_memcpy(dst->pac_opaque_encr_key, src->pac_opaque_encr_key, os_memcpy(dst->pac_opaque_encr_key, src->pac_opaque_encr_key,
16); 16);
} else } else
@ -1070,6 +1074,7 @@ static int eapol_auth_conf_clone(struct eapol_auth_config *dst,
dst->eap_fast_a_id = os_malloc(src->eap_fast_a_id_len); dst->eap_fast_a_id = os_malloc(src->eap_fast_a_id_len);
if (dst->eap_fast_a_id == NULL) { if (dst->eap_fast_a_id == NULL) {
os_free(dst->eap_req_id_text); os_free(dst->eap_req_id_text);
os_free(dst->pac_opaque_encr_key);
return -1; return -1;
} }
os_memcpy(dst->eap_fast_a_id, src->eap_fast_a_id, os_memcpy(dst->eap_fast_a_id, src->eap_fast_a_id,
@ -1081,6 +1086,7 @@ static int eapol_auth_conf_clone(struct eapol_auth_config *dst,
dst->eap_fast_a_id_info = os_strdup(src->eap_fast_a_id_info); dst->eap_fast_a_id_info = os_strdup(src->eap_fast_a_id_info);
if (dst->eap_fast_a_id_info == NULL) { if (dst->eap_fast_a_id_info == NULL) {
os_free(dst->eap_req_id_text); os_free(dst->eap_req_id_text);
os_free(dst->pac_opaque_encr_key);
os_free(dst->eap_fast_a_id); os_free(dst->eap_fast_a_id);
return -1; return -1;
} }