From: Jouni Malinen <j@w1.fi>
Date: Wed, 30 Dec 2009 20:37:23 +0000 (+0200)
Subject: Fix couple of memory leaks in wpa_config_get_all()
X-Git-Tag: hostap_0_7_1~146
X-Git-Url: http://www.project-moonshot.org/gitweb/?a=commitdiff_plain;h=0474686514ac2ba80b352792b034eec63ca6886e;p=libeap.git

Fix couple of memory leaks in wpa_config_get_all()
---

diff --git a/wpa_supplicant/config.c b/wpa_supplicant/config.c
index 692b52f..da56dc8 100644
--- a/wpa_supplicant/config.c
+++ b/wpa_supplicant/config.c
@@ -1860,12 +1860,18 @@ char ** wpa_config_get_all(struct wpa_ssid *ssid, int get_keys)
 		if (field->key_data && !get_keys)
 			continue;
 		value = field->writer(field, ssid);
-		if (value == NULL || os_strlen(value) == 0)
+		if (value == NULL)
 			continue;
+		if (os_strlen(value) == 0) {
+			os_free(value);
+			continue;
+		}
 
 		key = os_strdup(field->name);
-		if (key == NULL)
+		if (key == NULL) {
+			os_free(value);
 			goto err;
+		}
 
 		props[fields_num * 2] = key;
 		props[fields_num * 2 + 1] = value;