From: Purushottam Kushwaha <p.kushwaha@samsung.com>
Date: Fri, 15 Jan 2016 11:31:03 +0000 (+0000)
Subject: nl8021: Avoid potential memory leak on error path
X-Git-Tag: hostap_2_6~973
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=e15dcf6d1bc2725388555523effca75b1ffab735;p=thirdparty%2Fhostap.git

nl8021: Avoid potential memory leak on error path

The called function nl80211_ht_vht_overrides() was not freeing "msg"
resource in error cases.

Signed-off-by: Purushottam Kushwaha <p.kushwaha@samsung.com>
---

diff --git a/src/drivers/driver_nl80211.c b/src/drivers/driver_nl80211.c
index cdd150434..4cb92d0a4 100644
--- a/src/drivers/driver_nl80211.c
+++ b/src/drivers/driver_nl80211.c
@@ -4549,8 +4549,9 @@ retry:
 			goto fail;
 	}
 
-	if (nl80211_ht_vht_overrides(msg, params) < 0)
-		return -1;
+	ret = nl80211_ht_vht_overrides(msg, params);
+	if (ret < 0)
+		goto fail;
 
 	ret = send_and_recv_msgs(drv, msg, NULL, NULL);
 	msg = NULL;