From: Yu <jiung.yu@samsung.com>
Date: Fri, 7 May 2021 04:22:30 +0000 (+0900)
Subject: Fix issues reported by static analyzer
X-Git-Tag: accepted/tizen/6.5/unified/20211028.093125^0
X-Git-Url: http://review.tizen.org/git/?p=apps%2Fnative%2Fug-wifi-direct.git;a=commitdiff_plain;h=45d3102ac298453ce85d0192a5993895ec69a57c

Fix issues reported by static analyzer

Change-Id: Ib90660769c946a2e6b5e43611fa50be5226f4052
---

diff --git a/packaging/ug-setting-wifidirect-efl.spec b/packaging/ug-setting-wifidirect-efl.spec
index 32588df..4e1b48b 100644
--- a/packaging/ug-setting-wifidirect-efl.spec
+++ b/packaging/ug-setting-wifidirect-efl.spec
@@ -1,6 +1,6 @@
 Name:       ug-setting-wifidirect-efl
 Summary:    Wi-Fi Direct setting UI gadget
-Version:    1.12.1
+Version:    1.12.2
 Release:    1
 Group:      Applications/Network
 License:    Flora-1.1
diff --git a/ug-wifidirect/src/wfd_ug.c b/ug-wifidirect/src/wfd_ug.c
index cd21c29..cee643e 100644
--- a/ug-wifidirect/src/wfd_ug.c
+++ b/ug-wifidirect/src/wfd_ug.c
@@ -310,15 +310,15 @@ static void *on_create(ui_gadget_h ug, enum ug_mode mode, app_control_h control,
 		if (ret == APP_CONTROL_ERROR_NONE && wfds) {
 			DBG_SECURE(LOG_INFO, "Wfds name: %s", wfds);
 			ugd->wfds = strdup(wfds);
-			WFD_IF_FREE_MEM(wfds);
 		}
+		WFD_IF_FREE_MEM(wfds);
 
 		ret = app_control_get_extra_data(control, "viewtype", &viewtype);
 		if (ret == APP_CONTROL_ERROR_NONE && viewtype) {
 			DBG(LOG_INFO, "viewtype: %s\n", viewtype);
 			ugd->view_type = strdup(viewtype);
-			WFD_IF_FREE_MEM(viewtype);
 		}
+		WFD_IF_FREE_MEM(viewtype);
 
 		/*
 		* get the device filter
@@ -351,9 +351,8 @@ static void *on_create(ui_gadget_h ug, enum ug_mode mode, app_control_h control,
 				ugd->device_filter = WFD_DEVICE_TYPE_AUDIO;
 			else
 				ugd->device_filter = WFD_DEVICE_TYPE_OTHER;
-
-			WFD_IF_FREE_MEM(device_filter);
 		}
+		WFD_IF_FREE_MEM(device_filter);
 
 		/*
 		* get whether support auto exit after connection
@@ -366,9 +365,8 @@ static void *on_create(ui_gadget_h ug, enum ug_mode mode, app_control_h control,
 				ugd->is_auto_exit = true;
 			else
 				ugd->is_auto_exit = false;
-
-			WFD_IF_FREE_MEM(auto_exit);
 		}
+		WFD_IF_FREE_MEM(auto_exit);
 
 		/*
 		* get whether support multi connection,
@@ -381,9 +379,8 @@ static void *on_create(ui_gadget_h ug, enum ug_mode mode, app_control_h control,
 				ugd->is_multi_connect = false;
 			else
 				ugd->is_multi_connect = true;
-
-			WFD_IF_FREE_MEM(multi_connect);
 		}
+		WFD_IF_FREE_MEM(multi_connect);
 
 		/*
 		* get the title of UG
@@ -394,8 +391,8 @@ static void *on_create(ui_gadget_h ug, enum ug_mode mode, app_control_h control,
 			DBG(LOG_INFO, "Title of UG: %s", title);
 			WFD_IF_FREE_MEM(ugd->title);
 			ugd->title = strdup(title);
-			WFD_IF_FREE_MEM(title);
 		}
+		WFD_IF_FREE_MEM(title);
 	}
 
 	if (mode == UG_MODE_FULLVIEW)