From: Hyunsoo Park <hance.park@samsung.com>
Date: Tue, 18 May 2021 04:45:03 +0000 (+0900)
Subject: [testsuite] Fix memory leak
X-Git-Tag: submit/tizen/20210604.030216~1
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=bb5a2ca5f03364b70e1505fff427e05a1fe57b71;p=platform%2Fcore%2Fapi%2Fscreen-mirroring.git

[testsuite] Fix memory leak

Change-Id: I045be0892d310990494cca568c659684872439c6
Signed-off-by: Hyunsoo Park <hance.park@samsung.com>
---

diff --git a/packaging/capi-media-screen-mirroring.spec b/packaging/capi-media-screen-mirroring.spec
index 0aa2784..582f538 100644
--- a/packaging/capi-media-screen-mirroring.spec
+++ b/packaging/capi-media-screen-mirroring.spec
@@ -1,6 +1,6 @@
 Name:       capi-media-screen-mirroring
 Summary:    A screen mirroring library in Tizen C API
-Version:    0.2.21
+Version:    0.2.22
 Release:    1
 Group:      Multimedia/API
 License:    Apache-2.0
diff --git a/test_sink/scmirroring_sink_test.c b/test_sink/scmirroring_sink_test.c
index 14d219d..f1d3f22 100644
--- a/test_sink/scmirroring_sink_test.c
+++ b/test_sink/scmirroring_sink_test.c
@@ -754,7 +754,6 @@ void _connection_cb(int error_code, wifi_direct_connection_state_e connection_st
 	return;
 }
 
-
 static int __wifi_direct_device_connect()
 {
 	if (strlen(g_src_mac_addr) > 17 || strlen(g_src_mac_addr) <= 0) {
@@ -775,8 +774,6 @@ static int __wifi_direct_device_connect()
 static void __interpret(char *cmd)
 {
 	int ret = SCMIRRORING_ERROR_NONE;
-	gchar **value;
-	value = g_strsplit(cmd, " ", 0);
 
 	if (strncmp(cmd, "D", 1) == 0) {
 		g_print("Disconnect\n");
@@ -814,8 +811,16 @@ static void __interpret(char *cmd)
 	else if (strncmp(cmd, "a", 1) == 0) {
 		ret = __scmirroring_sink_create(NULL);
 		if (ret == SCMIRRORING_ERROR_NONE) {
-			ret = scmirroring_sink_set_ip_and_port(g_scmirroring, value[1], value[2]);
-			g_print("Input server IP and port number IP[%s] Port[%s]\n", value[1], value[2]);
+			gchar **value = g_strsplit(cmd, " ", 0);
+			if (value) {
+				if (value[1] && value[2]) {
+					ret = scmirroring_sink_set_ip_and_port(g_scmirroring, value[1], value[2]);
+					g_print("Input server IP and port number IP[%s] Port[%s]\n", value[1], value[2]);
+				} else {
+					g_print("Check command again. i.g 'a 192.168.0.1 2022'\n");
+				}
+				g_strfreev(value);
+			}
 		}
 	} else if (strncmp(cmd, "s", 1) == 0) {
 		g_print("Start\n");
@@ -823,10 +828,18 @@ static void __interpret(char *cmd)
 	}
 #else
 	else if (strncmp(cmd, "b", 1) == 0) {
-		strncpy(g_src_mac_addr, value[1], sizeof(g_src_mac_addr));
-		g_src_mac_addr[17] = '\0';
-		g_print("Src mac address : %s\n", g_src_mac_addr);
-		ret = __wifi_direct_device_connect();
+		gchar **value = g_strsplit(cmd, " ", 0);
+		if (value) {
+			if (value[1]) {
+				strncpy(g_src_mac_addr, value[1], sizeof(g_src_mac_addr));
+				g_src_mac_addr[17] = '\0';
+				g_print("Src mac address : %s\n", g_src_mac_addr);
+				ret = __wifi_direct_device_connect();
+			} else {
+				g_print("Check command again. i.g 'b [mac_address]'\n");
+			}
+			g_strfreev(value);
+		}
 	}
 #endif
 	else