Fix issue detected by static analysis tool

Calling app_get_id without checking return value (as is done elsewhere 7 out of 8 times).

Change-Id: I788f407ba70fef89c2c4951d588c0e8a73b1c5c5
Signed-off-by: Jihoon Kim <jihoon48.kim@samsung.com>

diff --git a/src/sclcoreimpl.cpp b/src/sclcoreimpl.cpp
index 63cecd4..a04e18d 100644 (file)
--- a/src/sclcoreimpl.cpp
+++ b/src/sclcoreimpl.cpp
@@ -71,14 +71,17 @@ sclboolean CSCLCoreImpl::prepare()
 
     if (!m_uuid) {
         char *appid = NULL;
-        app_get_id(&appid);
-
-        LOGD("appid : '%s'\n", appid);
-
-        if (appid) {
-            m_uuid = strdup(appid);
-            free(appid);
+        int app_ret = app_get_id(&appid);
+        if (app_ret == APP_ERROR_NONE) {
+            LOGD("appid : '%s'\n", appid);
+
+            if (appid) {
+                m_uuid = strdup(appid);
+                free(appid);
+            }
         }
+        else
+            LOGW("Failed to get app id(%d)", app_ret);
     }
 
     if (!m_window_creation_defer_flag) {
@@ -482,4 +485,4 @@ void CSCLCoreImpl::send_key_event_processing_result(scim::KeyEvent &key, sclu32
 void CSCLCoreImpl::set_dotnet_flag(sclboolean flag)
 {
     m_core_ui.set_dotnet_flag(flag);
-}
\ No newline at end of file
+}