Remove warnings
[platform/core/appfw/pkgmgr-info.git] / parser / pkgmgr_parser_db.c
index d1f2d7c..1a4a0bd 100644 (file)
 #define BUFSIZE 4096
 #define OWNER_ROOT 0
 
-#define SET_SMACK_LABEL(x,uid) \
-       if(smack_setlabel((x), (((uid) == GLOBAL_USER)?"*":"User"), SMACK_LABEL_ACCESS)) _LOGE("failed chsmack -a \"User/*\" %s", x); \
-       else _LOGD("chsmack -a \"User/*\" %s", x);
+#define SET_SMACK_LABEL(x) \
+do { \
+       if (smack_setlabel((x), "System::Shared", SMACK_LABEL_ACCESS)) \
+               _LOGE("failed chsmack -a \"System::Shared\" %s", x); \
+       else \
+               _LOGD("chsmack -a \"System::Shared\" %s", x); \
+} while (0)
 
 sqlite3 *pkgmgr_parser_db;
 sqlite3 *pkgmgr_cert_db;
@@ -64,6 +68,7 @@ sqlite3 *pkgmgr_cert_db;
                                                "(package text primary key not null, " \
                                                "package_type text DEFAULT 'rpm', " \
                                                "package_version text, " \
+                                               "package_api_version text, " \
                                                "install_location text, " \
                                                "package_size text, " \
                                                "package_removable text DEFAULT 'true', " \
@@ -301,6 +306,7 @@ static GList *__create_locale_list(GList *locale, label_x *lbl, license_x *lcn,
 static void __preserve_guestmode_visibility_value(manifest_x *mfx);
 static int __guestmode_visibility_cb(void *data, int ncols, char **coltxt, char **colname);
 static int __pkgmgr_parser_create_db(sqlite3 **db_handle, const char *db_path);
+static int __parserdb_change_perm(const char *db_file, uid_t uid);
 
 static int __delete_subpkg_list_cb(void *data, int ncols, char **coltxt, char **colname)
 {
@@ -310,7 +316,7 @@ static int __delete_subpkg_list_cb(void *data, int ncols, char **coltxt, char **
        return 0;
 }
 
-static char *__get_str(const char *str)
+static const char *__get_str(const char *str)
 {
        if (str == NULL)
        {
@@ -492,11 +498,6 @@ static GList *__create_image_list(GList *locale, image_x *image)
        return locale;
 }
 
-static void __printfunc(gpointer data, gpointer userdata)
-{
-       _LOGD("%s  ", (char*)data);
-}
-
 static void __trimfunc(GList* trim_list)
 {
        char *trim_data = NULL;
@@ -1524,14 +1525,10 @@ static int __insert_manifest_info_in_db(manifest_x *mfx, uid_t uid)
        privileges_x *pvs = NULL;
        privilege_x *pv = NULL;
        char query[MAX_QUERY_LEN] = { '\0' };
-       char root[MAX_QUERY_LEN] = { '\0' };
        int ret = -1;
-       char *type = NULL;
-       char *path = NULL;
        const char *auth_name = NULL;
        const char *auth_email = NULL;
        const char *auth_href = NULL;
-       const char *apps_path = NULL;
 
        GList *pkglocale = NULL;
        GList *applocale = NULL;
@@ -1562,31 +1559,15 @@ static int __insert_manifest_info_in_db(manifest_x *mfx, uid_t uid)
        }
 
        /*Insert in the package_info DB*/
-       if (mfx->type)
-               type = strdup(mfx->type);
-       else
-               type = strdup("rpm");
-       /*Insert in the package_info DB*/
-       if (mfx->root_path)
-               path = strdup(mfx->root_path);
-       else{
-               if (strcmp(type,"rpm")==0) {
-                       apps_path = tzplatform_getenv(TZ_SYS_RO_APP);
-                       snprintf(root, MAX_QUERY_LEN - 1, "%s/%s", apps_path, mfx->package);
-               } else {
-                       apps_path = tzplatform_getenv(TZ_USER_APP);
-                       snprintf(root, MAX_QUERY_LEN - 1, "%s/%s", apps_path, mfx->package);
-               }
-               path = strdup(root);
-       }
        snprintf(query, MAX_QUERY_LEN,
-                "insert into package_info(package, package_type, package_version, install_location, package_size, " \
+                "insert into package_info(package, package_type, package_version, package_api_version, install_location, package_size, " \
                "package_removable, package_preload, package_readonly, package_update, package_appsetting, package_nodisplay, package_system," \
                "author_name, author_email, author_href, installed_time, installed_storage, storeclient_id, mainapp_id, package_url, root_path, csc_path) " \
-               "values('%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s')",\
+               "values('%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s', '%s')",\
                 mfx->package,
-                type,
+                mfx->type,
                 mfx->version,
+                __get_str(mfx->api_version),
                 __get_str(mfx->installlocation),
                 __get_str(mfx->package_size),
                 mfx->removable,
@@ -1604,43 +1585,13 @@ static int __insert_manifest_info_in_db(manifest_x *mfx, uid_t uid)
                 __get_str(mfx->storeclient_id),
                 mfx->mainapp_id,
                 __get_str(mfx->package_url),
-                path,
+                mfx->root_path,
                 __get_str(mfx->csc_path));
-       /*If package dont have main_package tag, this package is main package.*/
-       if (mfx->main_package == NULL) {
-               ret = __exec_query(query);
-               if (ret == -1) {
-                       _LOGD("Package Info DB Insert Failed\n");
-                       if (type) {
-                               free(type);
-                               type = NULL;
-                       }
-                       if (path) {
-                               free(path);
-                               path = NULL;
-                       }
-                       return -1;
-               }
-       } else {
-               /*If package has main_package tag, this package is sub package(ug, efl).
-               skip __exec_query for package_info and change pkgid with main_package*/
-               memset(root, '\0', MAX_QUERY_LEN);
-               snprintf(root, MAX_QUERY_LEN - 1, "/usr/apps/%s", mfx->main_package);
-               if (access(root, F_OK) == 0) {
-                       free((void *)mfx->package);
-                       mfx->package = strdup(mfx->main_package);
-               } else {
-                       _LOGE("main package[%s] is not installed\n", root);
-                       return -1;
-               }
-       }
-       if (type) {
-               free(type);
-               type = NULL;
-       }
-       if (path) {
-               free(path);
-               path = NULL;
+
+       ret = __exec_query(query);
+       if (ret == -1) {
+               _LOGD("Package Info DB Insert Failed\n");
+               return -1;
        }
 
        /*Insert in the package_privilege_info DB*/
@@ -1706,13 +1657,7 @@ static int __insert_manifest_info_in_db(manifest_x *mfx, uid_t uid)
        /*remove duplicated data in appimage*/
        __trimfunc(appimage);
 
-       /*g_list_foreach(pkglocale, __printfunc, NULL);*/
-       /*_LOGD("\n");*/
-       /*g_list_foreach(applocale, __printfunc, NULL);*/
-
-       /*package locale info, it is only for main package.*/
-       if (mfx->main_package == NULL)
-               g_list_foreach(pkglocale, __insert_pkglocale_info, (gpointer)mfx);
+       g_list_foreach(pkglocale, __insert_pkglocale_info, (gpointer)mfx);
 
        /*native app locale info*/
        up = mfx->uiapplication;
@@ -2050,7 +1995,7 @@ static int __update_preload_condition_in_db()
        return ret;
 }
 
-API int pkgmgr_parser_initialize_db(void)
+API int pkgmgr_parser_initialize_db(uid_t uid)
 {
        int ret = -1;
        /*Manifest DB*/
@@ -2152,10 +2097,17 @@ API int pkgmgr_parser_initialize_db(void)
                return ret;
        }
 
+       if( 0 != __parserdb_change_perm(getUserPkgCertDBPathUID(uid), uid)) {
+               _LOGD("Failed to change cert db permission\n");
+       }
+       if( 0 != __parserdb_change_perm(getUserPkgParserDBPathUID(uid), uid)) {
+               _LOGD("Failed to change parser db permission\n");
+       }
+
        return 0;
 }
 
-static int parserdb_change_perm(const char *db_file, uid_t uid)
+static int __parserdb_change_perm(const char *db_file, uid_t uid)
 {
        char buf[BUFSIZE];
        char journal_file[BUFSIZE];
@@ -2166,14 +2118,16 @@ static int parserdb_change_perm(const char *db_file, uid_t uid)
        files[1] = journal_file;
        files[2] = NULL;
 
-       if(db_file == NULL)
+       if (db_file == NULL)
                return -1;
 
-       if(getuid() != OWNER_ROOT) //At this time we should be root to apply this
-                       return 0;
+       if (getuid() != OWNER_ROOT) //At this time we should be root to apply this
+               return 0;
        snprintf(journal_file, sizeof(journal_file), "%s%s", db_file, "-journal");
-    userinfo = getpwuid(uid);
-    if (!userinfo) {
+       if (uid == OWNER_ROOT)
+               uid = GLOBAL_USER;
+       userinfo = getpwuid(uid);
+       if (!userinfo) {
                _LOGE("FAIL: user %d doesn't exist", uid);
                return -1;
        }
@@ -2181,23 +2135,58 @@ static int parserdb_change_perm(const char *db_file, uid_t uid)
 
        for (i = 0; files[i]; i++) {
                ret = chown(files[i], uid, userinfo->pw_gid);
-               SET_SMACK_LABEL(files[i],uid)
                if (ret == -1) {
-                       strerror_r(errno, buf, sizeof(buf));
-                       _LOGD("FAIL : chown %s %d.%d, because %s", db_file, uid, userinfo->pw_gid, buf);
+                       if (strerror_r(errno, buf, sizeof(buf)))
+                               strcpy(buf, "");
+                       _LOGD("FAIL : chown %s %d.%d : %s", files[i], uid,
+                                       userinfo->pw_gid, buf);
                        return -1;
                }
 
                ret = chmod(files[i], S_IRUSR | S_IWUSR | S_IRGRP | S_IWGRP | S_IROTH);
                if (ret == -1) {
-                       strerror_r(errno, buf, sizeof(buf));
-                       _LOGD("FAIL : chmod %s 0664, because %s", db_file, buf);
+                       if (strerror_r(errno, buf, sizeof(buf)))
+                               strcpy(buf, "");
+                       _LOGD("FAIL : chmod %s 0664 : %s", files[i], buf);
                        return -1;
                }
+               SET_SMACK_LABEL(files[i]);
        }
        return 0;
 }
 
+API int pkgmgr_parser_create_and_initialize_db(uid_t uid)
+{
+       int ret;
+
+       if (getuid() != OWNER_ROOT) {
+               _LOGE("Only root user is allowed");
+               return -1;
+       }
+
+       if (access(getUserPkgParserDBPathUID(uid), F_OK) != -1) {
+               _LOGE("Manifest db for user %d is already exists", uid);
+               return -1;
+       }
+
+       if (access(getUserPkgCertDBPathUID(uid), F_OK) != -1) {
+               _LOGE("Cert db for user %d is already exists", uid);
+               return -1;
+       }
+
+       ret = pkgmgr_parser_check_and_create_db(uid);
+       if (ret < 0)
+               return -1;
+       ret = pkgmgr_parser_initialize_db(uid);
+       if (ret < 0) {
+               pkgmgr_parser_close_db();
+               return -1;
+       }
+       pkgmgr_parser_close_db();
+
+       return 0;
+}
+
 API int pkgmgr_parser_check_and_create_db(uid_t uid)
 {
        int ret = -1;
@@ -2214,16 +2203,10 @@ API int pkgmgr_parser_check_and_create_db(uid_t uid)
                _LOGD("Cert DB creation Failed\n");
                return -1;
        }
-       if( 0 != parserdb_change_perm(getUserPkgCertDBPathUID(uid), uid)) {
-               _LOGD("Failed to change cert db permission\n");
-       }
-       if( 0 != parserdb_change_perm(getUserPkgParserDBPathUID(uid), uid)) {
-               _LOGD("Failed to change parser db permission\n");
-       }
        return 0;
 }
 
-void pkgmgr_parser_close_db()
+void pkgmgr_parser_close_db(void)
 {
        sqlite3_close(pkgmgr_parser_db);
        sqlite3_close(pkgmgr_cert_db);
@@ -2243,7 +2226,7 @@ API int pkgmgr_parser_insert_manifest_info_in_db(manifest_x *mfx)
                _LOGD("Failed to open DB\n");
                return ret;
        }
-       ret = pkgmgr_parser_initialize_db();
+       ret = pkgmgr_parser_initialize_db(GLOBAL_USER);
        if (ret == -1)
                goto err;
        /*Begin transaction*/
@@ -2287,7 +2270,7 @@ API int pkgmgr_parser_insert_manifest_info_in_usr_db(manifest_x *mfx, uid_t uid)
                _LOGD("Failed to open DB\n");
                return ret;
        }
-       ret = pkgmgr_parser_initialize_db();
+       ret = pkgmgr_parser_initialize_db(uid);
        if (ret == -1)
                goto err;
        /*Begin transaction*/
@@ -2330,7 +2313,7 @@ API int pkgmgr_parser_update_manifest_info_in_usr_db(manifest_x *mfx, uid_t uid)
                _LOGD("Failed to open DB\n");
                return ret;
        }
-       ret = pkgmgr_parser_initialize_db();
+       ret = pkgmgr_parser_initialize_db(uid);
        if (ret == -1)
                goto err;
        /*Preserve guest mode visibility*/