Fix static analysis issue 81/296881/1
authorilho kim <ilho159.kim@samsung.com>
Mon, 7 Aug 2023 05:37:41 +0000 (14:37 +0900)
committerilho kim <ilho159.kim@samsung.com>
Mon, 7 Aug 2023 05:37:41 +0000 (14:37 +0900)
Avoid null pointer dereference

Change-Id: Id47e7cf188e299142a090e08f696810ac8e66f45
Signed-off-by: ilho kim <ilho159.kim@samsung.com>
src/server/appinfo_internal.cc

index f56b175..1e05dce 100644 (file)
@@ -638,7 +638,8 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
     } else {
       tmp_record = GetCString(idx++, rec);
       if (tmp_record != nullptr) {
-        if (strcasecmp(info->is_disabled, "false") == 0 &&
+        if (info->is_disabled &&
+            strcasecmp(info->is_disabled, "false") == 0 &&
             strcasecmp(tmp_record, "false") == 0) {
           free(info->is_disabled);
           info->is_disabled = tmp_record;
@@ -648,7 +649,8 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
       }
       tmp_record = GetCString(idx++, rec);
       if (tmp_record != nullptr) {
-        if (strcasecmp(info->splash_screen_display, "false") == 0 &&
+        if (info->splash_screen_display &&
+            strcasecmp(info->splash_screen_display, "false") == 0 &&
             strcasecmp(tmp_record, "false") == 0) {
           free(info->splash_screen_display);
           info->splash_screen_display = tmp_record;