From 5ca762f62349c1f1d4f023a45d024c0ee1a7bcf0 Mon Sep 17 00:00:00 2001 From: Abhishek Vijay Date: Thu, 16 Aug 2018 17:54:31 +0530 Subject: [PATCH] handling return value properly Change-Id: Id8d0252dbf6b53bf4236ea6dac5f1de73aa65884 Signed-off-by: Abhishek Vijay --- src/account.c | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) diff --git a/src/account.c b/src/account.c index 223f504..aaa6400 100644 --- a/src/account.c +++ b/src/account.c @@ -522,9 +522,11 @@ int _unregister_account_provider(pkgmgrinfo_appinfo_h package_info_handle, void* _D("Unregistering the Account Provider."); char* app_id = NULL; - int get_appid_ret = pkgmgrinfo_appinfo_get_appid(package_info_handle, &app_id); + int pkgmgr_ret = -1; + + pkgmgr_ret = pkgmgrinfo_appinfo_get_appid(package_info_handle, &app_id); + retvm_if((pkgmgr_ret != PMINFO_R_OK || !app_id), pkgmgr_ret, "Error fetching app id"); - retvm_if(get_appid_ret == PMINFO_R_ERROR, PMINFO_R_ERROR, "Error fetching app id"); _D("appid : %s", app_id); int ret = ACCOUNT_ERROR_NONE; @@ -569,7 +571,11 @@ int _on_package_app_list_received_cb(pkgmgrinfo_appinfo_h handle, void *user_dat int ret = ACCOUNT_ERROR_NONE; char* app_id = NULL; uid_t uid = -1; - pkgmgrinfo_appinfo_get_appid(handle, &app_id); + int pkgmgr_ret = -1; + + pkgmgr_ret = pkgmgrinfo_appinfo_get_appid(handle, &app_id); + retvm_if((pkgmgr_ret != PMINFO_R_OK || !app_id), pkgmgr_ret, "Error fetching app id"); + _D("appid : %s", app_id); if (pkgmgr_installer_info_get_target_uid(&uid) < 0) { -- 2.34.1