diff mbox

eap: Fix possible memory leak in eap_leap_process_request()

Message ID 1496909880-3511-4-git-send-email-andrei.otcheretianski@intel.com
State Accepted
Headers show

Commit Message

Andrei Otcheretianski June 8, 2017, 8:17 a.m. UTC 
From: Ilan Peer <ilan.peer@intel.com>

Free 'resp' object in case of failure to derive response.

Signed-off-by: Ilan Peer <ilan.peer@intel.com>
---
 src/eap_peer/eap_leap.c | 1 +
 1 file changed, 1 insertion(+)

Comments

Jouni Malinen July 8, 2017, 7:33 p.m. UTC | #1 
On Thu, Jun 08, 2017 at 11:17:57AM +0300, Andrei Otcheretianski wrote:
> Free 'resp' object in case of failure to derive response.

Thanks, applied.
diff mbox

Patch

diff --git a/src/eap_peer/eap_leap.c b/src/eap_peer/eap_leap.c
index 7bb62e7..233b9ee 100644
--- a/src/eap_peer/eap_leap.c
+++ b/src/eap_peer/eap_leap.c
@@ -120,6 +120,7 @@  static struct wpabuf * eap_leap_process_request(struct eap_sm *sm, void *priv,
 	     nt_challenge_response(challenge, password, password_len, rpos))) {
 		wpa_printf(MSG_DEBUG, "EAP-LEAP: Failed to derive response");
 		ret->ignore = TRUE;
+		wpabuf_free(resp);
 		return NULL;
 	}
 	os_memcpy(data->peer_response, rpos, LEAP_RESPONSE_LEN);