summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJunghyun Yeon <jungh.yeon@samsung.com>2021-04-01 13:56:29 +0900
committerJunghyun Yeon <jungh.yeon@samsung.com>2021-04-01 13:56:29 +0900
commit436a6cc2006417ba304115bc970c2c229ac278f1 (patch)
treee74012756ed5d292126dd0a7c39235fd8e11dbd3
parent7711957deb3cc931b737e5bda3ce3b0c0eb3095a (diff)
downloadpkgmgr-server-436a6cc2006417ba304115bc970c2c229ac278f1.tar.gz
pkgmgr-server-436a6cc2006417ba304115bc970c2c229ac278f1.tar.bz2
pkgmgr-server-436a6cc2006417ba304115bc970c2c229ac278f1.zip
Change order of functions
Unset exclusive should be done before free job. Change-Id: I975aa5917112edf73212b5988f25f13025e70c45 Signed-off-by: Junghyun Yeon <jungh.yeon@samsung.com>
-rw-r--r--src/pkgmgr-server.c34
1 files changed, 17 insertions, 17 deletions
diff --git a/src/pkgmgr-server.c b/src/pkgmgr-server.c
index f622958..5ba813a 100644
--- a/src/pkgmgr-server.c
+++ b/src/pkgmgr-server.c
@@ -1916,28 +1916,28 @@ gboolean queue_job(void *data)
case REQUEST_TYPE_REGISTER_PKG_UPDATE_INFO:
ret = __process_register_pkg_update_info(job);
__free_extra_info(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_UNREGISTER_PKG_UPDATE_INFO:
ret = __process_unregister_pkg_update_info(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_UNREGISTER_ALL_PKG_UPDATE_INFO:
ret = __process_unregister_all_pkg_update_info(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_ENABLE_APP:
ret = __process_enable_app(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_DISABLE_APP:
ret = __process_disable_app(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_GETSIZE:
__set_backend_busy(x);
@@ -1965,13 +1965,13 @@ gboolean queue_job(void *data)
break;
case REQUEST_TYPE_ENABLE_GLOBAL_APP_FOR_UID:
ret = __process_enable_global_app_for_uid(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_DISABLE_GLOBAL_APP_FOR_UID:
ret = __process_disable_global_app_for_uid(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_KILL:
ret = __process_kill(job);
@@ -1983,53 +1983,53 @@ gboolean queue_job(void *data)
break;
case REQUEST_TYPE_GENERATE_LICENSE_REQUEST:
ret = __process_generate_license_request(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_REGISTER_LICENSE:
ret = __process_register_license(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_DECRYPT_PACKAGE:
ret = __process_decrypt_package(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_ENABLE_APP_SPLASH_SCREEN:
ret = __process_update_app_splash_screen(job, 1);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_DISABLE_APP_SPLASH_SCREEN:
ret = __process_update_app_splash_screen(job, 0);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_SET_RESTRICTION_MODE:
ret = __process_set_restriction_mode(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_UNSET_RESTRICTION_MODE:
ret = __process_unset_restriction_mode(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_GET_RESTRICTION_MODE:
ret = __process_get_restriction_mode(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_SET_APP_LABEL:
ret = __process_set_app_label(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_SET_APP_ICON:
ret = __process_set_app_icon(job);
- _free_backend_job(job);
__unset_exclusive(job->is_exclusive);
+ _free_backend_job(job);
break;
case REQUEST_TYPE_MIGRATE_EXTERNAL_IMAGE:
__set_backend_busy(x);