fix coverity issue Unchecked return value 42/225042/2
authorwansu.yoo <wansu.yoo@samsung.com>
Mon, 17 Feb 2020 04:21:27 +0000 (13:21 +0900)
committerwansu.yoo <wansu.yoo@samsung.com>
Mon, 17 Feb 2020 04:28:38 +0000 (13:28 +0900)
Change-Id: I6fe43e94d3a5d87fe3aebd4708744fe4193e349f
Signed-off-by: wansu.yoo <wansu.yoo@samsung.com>
CMain/src/orchestration_dbus_server.c

index add463588f347d371ad7e6f4d717085a573e1a5f..ec11fe0b4d84d500eb0e7a1afefde4de17761606 100644 (file)
@@ -320,10 +320,13 @@ static int _request_service(GVariant *parameters, int origin_client_pid)
     DEBUG("\t client_pid = %d\n", client_pid);
 
     for (int i = 0; i < count; i++) {
-        g_variant_iter_loop (iter, "(ss)", &service_info[i].ExecutionType, &service_info[i].ExeCmd);
-
-        DEBUG("\t service_info[%d].ExecutionType = %s\n", i, service_info[i].ExecutionType);
-        DEBUG("\t service_info[%d].ExeCmd = %s\n", i, service_info[i].ExeCmd);
+        if (g_variant_iter_loop (iter, "(ss)", &service_info[i].ExecutionType, &service_info[i].ExeCmd)) {
+            DEBUG("\t service_info[%d].ExecutionType = %s\n", i, service_info[i].ExecutionType);
+            DEBUG("\t service_info[%d].ExeCmd = %s\n", i, service_info[i].ExeCmd);
+        } else {
+            DEBUG("\t no more items to iterate!!!\n");
+            break;
+        }
     }
     g_variant_iter_free (iter);