From: Jouni Malinen <jouni@qca.qualcomm.com>
Date: Wed, 29 Nov 2017 11:03:48 +0000 (+0200)
Subject: DPP: Fix a memory leak on an error path
X-Git-Tag: hostap_2_7~761
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=622934128b9938c060aef0a4c5b96aad4436fd59;p=thirdparty%2Fhostap.git

DPP: Fix a memory leak on an error path

Need to free temporary allocations if dpp_build_conf_start() fails to
allocate memory.

Signed-off-by: Jouni Malinen <jouni@qca.qualcomm.com>
---

diff --git a/src/common/dpp.c b/src/common/dpp.c
index 74cdddac7..c70f73b0c 100644
--- a/src/common/dpp.c
+++ b/src/common/dpp.c
@@ -4149,7 +4149,7 @@ skip_groups:
 	tailroom += signed1_len + signed2_len + signed3_len;
 	buf = dpp_build_conf_start(auth, conf, tailroom);
 	if (!buf)
-		return NULL;
+		goto fail;
 
 	wpabuf_put_str(buf, "\"cred\":{\"akm\":\"dpp\",\"signedConnector\":\"");
 	wpabuf_put_str(buf, signed1);