Browse Source

EAP-TTLS: Fix a memory leak on error paths

The allocated challenge needs to be freed on these error paths as well.

Signed-off-by: Jouni Malinen <j@w1.fi>
Jouni Malinen 7 years ago
parent
commit
1f2ae8cff5
1 changed files with 2 additions and 0 deletions
  1. 2 0
      src/eap_peer/eap_ttls.c

+ 2 - 0
src/eap_peer/eap_ttls.c

@@ -629,6 +629,7 @@ static int eap_ttls_phase2_request_mschap(struct eap_sm *sm,
 			wpa_printf(MSG_ERROR,
 				   "EAP-TTLS/MSCHAP: Failed derive password hash");
 			wpabuf_free(msg);
+			os_free(challenge);
 			return -1;
 		}
 
@@ -641,6 +642,7 @@ static int eap_ttls_phase2_request_mschap(struct eap_sm *sm,
 			wpa_printf(MSG_ERROR,
 				   "EAP-TTLS/MSCHAP: Failed derive password");
 			wpabuf_free(msg);
+			os_free(challenge);
 			return -1;
 		}