From: Wonnam Jang <wn.jang@samsung.com>
Date: Wed, 22 Jun 2016 01:06:39 +0000 (+0900)
Subject: Fix deference before checking null
X-Git-Tag: accepted/tizen/common/20160622.150112~1^2
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=refs%2Fchanges%2F50%2F75850%2F2;p=platform%2Fcore%2Fuifw%2Fvoice-control.git

Fix deference before checking null

Change-Id: I3cd94fc13dadcb932911f97060cd6b5049115c1c
Signed-off-by: Wonnam Jang <wn.jang@samsung.com>
---

diff --git a/server/vcd_client_data.c b/server/vcd_client_data.c
index c922c4d..e6d66b6 100644
--- a/server/vcd_client_data.c
+++ b/server/vcd_client_data.c
@@ -815,6 +815,9 @@ int __show_client_list()
 	} else {
 		iter = g_slist_nth(g_client_list, 0);
 		for (i = 0; i < count; i++) {
+			if (NULL == iter)
+				break;
+
 			data = iter->data;
 
 			SLOG(LOG_DEBUG, TAG_VCD, "[%dth] pid(%d)", i, data->pid);
@@ -835,6 +838,9 @@ int __show_client_list()
 	} else {
 		iter = g_slist_nth(g_widget_list, 0);
 		for (i = 0; i < count; i++) {
+			if (NULL == iter)
+				break;
+
 			widget_data = iter->data;
 
 			SLOG(LOG_DEBUG, TAG_VCD, "[%dth] pid(%d)", i, widget_data->pid);
@@ -862,6 +868,9 @@ int __show_command_list(GSList* cmd_group)
 	} else {
 		iter = g_slist_nth(cmd_group, 0);
 		for (i = 0; i < count; i++) {
+			if (NULL == iter)
+				break;
+
 			data = iter->data;
 
 			if (NULL != data->parameter) {
@@ -889,6 +898,9 @@ GSList* __client_get_item(const int pid)
 	if (0 < count) {
 		iter = g_slist_nth(g_client_list, 0);
 		for (i = 0; i < count; i++) {
+			if (NULL == iter)
+				break;
+
 			data = iter->data;
 
 			if (pid == data->pid)
@@ -912,6 +924,9 @@ vc_client_info_s* __client_get_element(int pid)
 	if (0 < count) {
 		iter = g_slist_nth(g_client_list, 0);
 		for (i = 0; i < count; i++) {
+			if (NULL == iter)
+				break;
+
 			data = iter->data;
 
 			if (pid == data->pid)
@@ -1163,6 +1178,9 @@ GSList* __widget_get_item(int pid)
 	if (0 < count) {
 		iter = g_slist_nth(g_widget_list, 0);
 		for (i = 0; i < count; i++) {
+			if (NULL == iter)
+				break;
+
 			data = iter->data;
 
 			if (pid == data->pid)
@@ -1186,6 +1204,9 @@ widget_info_s* __widget_get_element(int pid)
 	if (0 < count) {
 		iter = g_slist_nth(g_widget_list, 0);
 		for (i = 0; i < count; i++) {
+			if (NULL == iter)
+				break;
+
 			data = iter->data;
 
 			if (NULL != data) {