Fix static analysis issue
[platform/core/appfw/pkgmgr-info.git] / src / server / appinfo_internal.cc
index 7469b75..d6f2359 100644 (file)
@@ -85,11 +85,11 @@ void ParseAppControl(GList** appcontrol,
 
 int GetSplashScreens(const tizen_base::Database& db,
     const char* appid, GList** splashscreens) {
-  auto q = tizen_base::Database::Sql(
+  auto q = std::move(tizen_base::Database::Sql(
       "SELECT src, type, orientation, indicatordisplay, "
       "operation, color_depth "
       "FROM package_app_splash_screen WHERE app_id=?")
-      .Bind(appid);
+      .Bind(appid));
 
   auto r = db.Exec(q);
   if (!r) {
@@ -120,10 +120,10 @@ int GetSplashScreens(const tizen_base::Database& db,
 
 int GetMetadata(const tizen_base::Database& db,
     const char* appid, GList** metadata) {
-  auto q = tizen_base::Database::Sql(
+  auto q = std::move(tizen_base::Database::Sql(
       "SELECT md_key, md_value "
       "FROM package_app_app_metadata WHERE app_id=?")
-      .Bind(appid);
+      .Bind(appid));
 
   auto r = db.Exec(q);
   if (!r) {
@@ -150,10 +150,10 @@ int GetMetadata(const tizen_base::Database& db,
 
 int GetAppControl(const tizen_base::Database& db,
     const char* appid, GList** appcontrol) {
-  auto q = tizen_base::Database::Sql(
+  auto q = std::move(tizen_base::Database::Sql(
       "SELECT app_control, visibility, app_control_id "
       "FROM package_app_app_control WHERE app_id=?")
-      .Bind(appid);
+      .Bind(appid));
 
   auto r = db.Exec(q);
   if (!r) {
@@ -171,10 +171,10 @@ int GetAppControl(const tizen_base::Database& db,
 
 int GetCategory(const tizen_base::Database& db,
     const char* appid, GList** category) {
-  auto q = tizen_base::Database::Sql(
+  auto q = std::move(tizen_base::Database::Sql(
       "SELECT category "
       "FROM package_app_app_category WHERE app_id=?")
-      .Bind(appid);
+      .Bind(appid));
   auto r = db.Exec(q);
   if (!r) {
     _LOGE("db.Exec() failed: %s", static_cast<const char*>(r));
@@ -192,10 +192,10 @@ int GetCategory(const tizen_base::Database& db,
 
 int GetResControl(const tizen_base::Database& db, const char* appid,
     GList** res_control) {
-  auto q = tizen_base::Database::Sql(
+  auto q = std::move(tizen_base::Database::Sql(
       "SELECT res_type, min_res_version, max_res_version, auto_close "
       "FROM package_app_res_control WHERE app_id=?")
-      .Bind(appid);
+      .Bind(appid));
 
   auto r = db.Exec(q);
   if (!r) {
@@ -385,12 +385,89 @@ bool __check_app_storage_status(pkgmgrinfo_filter_x* tmp_filter) {
   return true;
 }
 
+enum class PackageDisableFilterStatus {
+  None,
+  True,
+  False
+};
+
+PackageDisableFilterStatus GetPackageDisableFilterStatus(
+    const pkgmgrinfo_filter_x* filter) {
+  GSList* tmp_list = nullptr;
+  pkgmgrinfo_node_x* tmp_node = nullptr;
+  int property = -1;
+
+  property = _pminfo_appinfo_convert_to_prop_bool(
+        PMINFO_APPINFO_PROP_PKG_DISABLE);
+  for (tmp_list = filter->list; tmp_list != nullptr;
+      tmp_list = g_slist_next(tmp_list)) {
+    tmp_node = reinterpret_cast<pkgmgrinfo_node_x*>(tmp_list->data);
+    if (property != tmp_node->prop)
+      continue;
+
+    return strcasecmp(tmp_node->value, "true") == 0 ?
+        PackageDisableFilterStatus::True : PackageDisableFilterStatus::False;
+  }
+
+  return PackageDisableFilterStatus::None;
+}
+
+bool CheckPkgDisableFilter(PackageDisableFilterStatus expected,
+    const char* package_disable, application_x* app_info) {
+  if (!package_disable)
+    return true;
+
+  PackageDisableFilterStatus result = strcasecmp(package_disable, "true") == 0 ?
+      PackageDisableFilterStatus::True : PackageDisableFilterStatus::False;
+
+  if (expected == result)
+    return true;
+
+  if (result == PackageDisableFilterStatus::True)
+    LOGW("The package [%s] to which the application [%s] belongs is disabled",
+        app_info->package, app_info->appid);
+
+  return false;
+}
+
+bool CheckAppDisableFilter(const pkgmgrinfo_filter_x* filter,
+    const application_x* app_info) {
+  GSList* tmp_list = nullptr;
+  pkgmgrinfo_node_x* tmp_node = nullptr;
+  int property = -1;
+
+  property = _pminfo_appinfo_convert_to_prop_bool(
+        PMINFO_APPINFO_PROP_APP_DISABLE);
+  for (tmp_list = filter->list; tmp_list != nullptr;
+      tmp_list = g_slist_next(tmp_list)) {
+    tmp_node = reinterpret_cast<pkgmgrinfo_node_x*>(tmp_list->data);
+    if (property != tmp_node->prop)
+      continue;
+
+    if (strcasecmp(tmp_node->value, app_info->is_disabled) == 0)
+      return true;
+
+    if (strcasecmp(app_info->is_disabled, "true") == 0)
+      LOGW("The application [%s] is disabled", app_info->appid);
+
+    return false;
+  }
+  return true;
+}
+
 int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
     const char* locale, pkgmgrinfo_filter_x* filter, int flag,
     std::vector<std::shared_ptr<application_x>>& applications) {
   static const char query_raw[] =
       "SELECT DISTINCT ai.app_id, ai.app_installed_storage, "
       "ai.app_external_path";
+  static const char query_app_disable[] = ", "
+      "CASE "
+        "WHEN ai.app_disable='true' COLLATE NOCASE OR "
+          "ui.is_disabled='true' COLLATE NOCASE "
+        "THEN 'true' "
+        "ELSE 'false' "
+      "END AS app_disable";
   static const char query_basic[] =
       ", ai.app_component, ai.app_exec, "
       "ai.app_nodisplay, ai.app_type, ai.app_onboot, "
@@ -404,7 +481,7 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
       "ai.app_support_disable, ai.app_process_pool, "
       "ai.app_background_category, ai.app_package_type, "
       "ai.app_root_path, ai.app_api_version, ai.app_effective_appid, "
-      "ai.app_disable, ai.app_splash_screen_display, ai.app_tep_name, "
+      "ai.app_splash_screen_display, ai.app_tep_name, "
       "ai.app_zip_mount_file, ai.component_type, ai.package, "
       "ai.app_package_system, ai.app_removable, "
       "ai.app_package_installed_time, ai.app_support_mode, "
@@ -423,6 +500,8 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
       "AND app_locale=?), "
       "(SELECT app_icon FROM package_app_localized_info WHERE ai.app_id=app_id "
       "AND app_locale='No Locale'))";
+  static const char query_package_disable[] =
+      ", (SELECT package_disable FROM package_info WHERE ai.package=package)";
   static const char query_from_clause[] = " FROM package_app_info as ai";
   static const char query_uid_info_clause[] =
       " LEFT OUTER JOIN package_app_info_for_uid AS ui "
@@ -433,6 +512,9 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
   bool is_check_storage = true;
   const uid_t global_user_uid = GLOBAL_USER;
   std::string query = query_raw;
+  auto pkg_disable_filter_status = GetPackageDisableFilterStatus(filter);
+
+  query += query_app_disable;
 
   if (flag & PMINFO_APPINFO_GET_BASICINFO) {
     query += query_basic;
@@ -449,6 +531,9 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
     bind_params.push_back(locale);
   }
 
+  if (pkg_disable_filter_status != PackageDisableFilterStatus::None)
+    query += query_package_disable;
+
   bind_params.push_back(std::to_string(uid));
   is_check_storage = __check_app_storage_status(filter);
 
@@ -495,6 +580,7 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
     info->appid = GetCString(idx++, rec);
     info->installed_storage = GetCString(idx++, rec);
     info->external_path = GetCString(idx++, rec);
+    info->is_disabled = GetCString(idx++, rec);
     if (flag & PMINFO_APPINFO_GET_BASICINFO) {
       info->component = GetCString(idx++, rec);
       info->exec = GetCString(idx++, rec);
@@ -526,7 +612,6 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
       info->root_path = GetCString(idx++, rec);
       info->api_version = GetCString(idx++, rec);
       info->effective_appid = GetCString(idx++, rec);
-      info->is_disabled = GetCString(idx++, rec);
       info->splash_screen_display = GetCString(idx++, rec);
       info->tep_name = GetCString(idx++, rec);
       info->zip_mount_file = GetCString(idx++, rec);
@@ -553,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;
@@ -563,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;
@@ -577,6 +664,9 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
       tmp_record = GetCString(idx++, rec);
       if (_add_label_info_into_list(locale, tmp_record,
           &info->label)) {
+        if (tmp_record)
+          free(tmp_record);
+
         return PMINFO_R_ERROR;
       }
     }
@@ -585,10 +675,25 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
       tmp_record = GetCString(idx++, rec);
       if (_add_icon_info_into_list(locale, tmp_record,
           &info->icon)) {
+        if (tmp_record)
+          free(tmp_record);
+
         return PMINFO_R_ERROR;
       }
     }
 
+    if (pkg_disable_filter_status != PackageDisableFilterStatus::None) {
+      tmp_record = GetCString(idx++, rec);
+      if (tmp_record) {
+        bool filter_passed = CheckPkgDisableFilter(pkg_disable_filter_status,
+            tmp_record, info);
+        free(tmp_record);
+
+        if (!filter_passed)
+          continue;
+      }
+    }
+
     if (flag & PMINFO_APPINFO_GET_CATEGORY) {
       if (GetCategory(db, info->appid, &info->category))
         return PMINFO_R_ERROR;
@@ -620,6 +725,14 @@ int DoGetAppInfo(const tizen_base::Database& db, uid_t db_uid, uid_t uid,
       continue;
     }
 
+    if (!CheckAppDisableFilter(filter, info))
+      continue;
+
+    if (!(flag & PMINFO_APPINFO_GET_BASICINFO) && info->is_disabled) {
+      free(info->is_disabled);
+      info->is_disabled = nullptr;
+    }
+
     applications.push_back(std::move(info_auto));
   }