From: taesub kim <taesub.kim@samsung.com>
Date: Wed, 8 Nov 2017 06:42:43 +0000 (+0900)
Subject: Fix memory leak
X-Git-Tag: submit/tizen/20171108.071810~2^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F08%2F159308%2F1;p=platform%2Fcore%2Fconnectivity%2Fnet-config.git

Fix memory leak

Change-Id: I06bf6a5fdb1b5a63c252fc6769dd1d2a6c1f71a6
Signed-off-by: Taesub Kim <taesub.kim@samsung.com>
---

diff --git a/src/wifi-indicator.c b/src/wifi-indicator.c
index 1c99a18..86fd4d8 100755
--- a/src/wifi-indicator.c
+++ b/src/wifi-indicator.c
@@ -82,22 +82,16 @@ static int __netconfig_wifi_update_and_get_rssi(void)
 	}
 
 	message = netconfig_invoke_dbus_method(SUPPLICANT_SERVICE, if_path,
-				SUPPLICANT_INTERFACE ".Interface", "SignalPoll", NULL);
-
-	g_free(if_path);
+			SUPPLICANT_INTERFACE ".Interface", "SignalPoll", NULL);
 	if (message == NULL) {
 		ERR("Fail to get SignalPoll from wpa_supplicant");
+		g_free(if_path);
 		return 0;
 	}
 
 	g_variant_get(message, "(v)", &value);
 
 	g_variant_get(value, "a{sv}", &iter);
-	if (iter == NULL) {
-		ERR("Fail to get list from SignalPoll");
-		return 0;
-	}
-
 	while (g_variant_iter_loop(iter, "{sv}", &key, &variant)) {
 		key_value = g_variant_get_int32(variant);
 
@@ -115,6 +109,10 @@ static int __netconfig_wifi_update_and_get_rssi(void)
 	if (value)
 		g_variant_unref(value);
 
+	g_variant_unref(message);
+
+	g_free(if_path);
+
 	netconfig_wifi_rssi = rssi_dbm;
 
 	return rssi_dbm;