diff mbox

Avoid Potential memory leak.

Message ID 833617682.1134241452857463524.JavaMail.weblogic@ep2mlwas08a
State Accepted
Headers show

Commit Message

Purushottam Kushwaha Jan. 15, 2016, 11:31 a.m. UTC 
Called function [nl80211_ht_vht_overrides()] is not freeing "msg" resource in fail cases.

Signed-off-by: Purushottam Kushwaha <p.kushwaha@samsung.com>
---
 src/drivers/driver_nl80211.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

Comments

Jouni Malinen Jan. 15, 2016, 6:53 p.m. UTC | #1 
On Fri, Jan 15, 2016 at 11:31:03AM +0000, Purushottam Kushwaha wrote:
> Called function [nl80211_ht_vht_overrides()] is not freeing "msg" resource in fail cases.

Thanks, applied.
diff mbox

Patch

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