diff --git a/interfaces/inner_api/appexecfwk_base/include/appexecfwk_errors.h b/interfaces/inner_api/appexecfwk_base/include/appexecfwk_errors.h index ba718e67f9c473ec9849d49bee2973d0da3b82e7..dfac06139e72bc2a2afc89c77ffdec0411d2fdba 100644 --- a/interfaces/inner_api/appexecfwk_base/include/appexecfwk_errors.h +++ b/interfaces/inner_api/appexecfwk_base/include/appexecfwk_errors.h @@ -613,8 +613,6 @@ enum { // Only used for hisysevent report ERR_APPEXECFWK_UNINSTALL_AND_INSTALL = 8522540, - ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW = 8522541, - // only allow install in u1 when set u1enable ERR_APPEXECFWK_INSTALL_U1ENABLE_CAN_ONLY_INSTALL_IN_U1_WITH_NOT_SINGLETON = 8522550, ERR_APPEXECFWK_INSTALL_U1_ENABLE_NOT_SUPPORT_APP_SERVICE_AND_SHARED_BUNDLE = 8522551, diff --git a/interfaces/inner_api/appexecfwk_base/include/bundle_constants.h b/interfaces/inner_api/appexecfwk_base/include/bundle_constants.h index 32d0b3181cf9ffc9e8b6706a0813b9c79a132cd6..19d746653b510a06c2441814a4abb264196a49f3 100644 --- a/interfaces/inner_api/appexecfwk_base/include/bundle_constants.h +++ b/interfaces/inner_api/appexecfwk_base/include/bundle_constants.h @@ -71,7 +71,6 @@ const std::set EXPECTED_ERROR = { ERR_APPEXECFWK_CLONE_INSTALL_APP_INDEX_EXISTED, ERR_APPEXECFWK_CLONE_INSTALL_APP_NOT_EXISTED, ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL, - ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW, ERR_APPEXECFWK_UNINSTALL_CONTROLLED }; diff --git a/interfaces/inner_api/appexecfwk_core/src/bundlemgr/bundle_status_callback_proxy.cpp b/interfaces/inner_api/appexecfwk_core/src/bundlemgr/bundle_status_callback_proxy.cpp index d45a0f9e24c932d912d36de3f9b0e03c19626614..009a803c7adba278a8a3a2c5b224c1dab444c2a7 100644 --- a/interfaces/inner_api/appexecfwk_core/src/bundlemgr/bundle_status_callback_proxy.cpp +++ b/interfaces/inner_api/appexecfwk_core/src/bundlemgr/bundle_status_callback_proxy.cpp @@ -146,7 +146,6 @@ void BundleStatusCallbackProxy::InitResultMap() {ERR_APPEXECFWK_INSTALL_SINGLETON_INCOMPATIBLE, "ERR_APPEXECFWK_INSTALL_SINGLETON_INCOMPATIBLE"}, {ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_INSTALL, "ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_INSTALL"}, {ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL, "ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL"}, - {ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW, "ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL"}, {ERR_APPEXECFWK_INSTALL_DEVICE_TYPE_NOT_SUPPORTED, "ERR_APPEXECFWK_INSTALL_DEVICE_TYPE_NOT_SUPPORTED"}, {ERR_APPEXECFWK_INSTALL_SYSCAP_FAILED_AND_DEVICE_TYPE_ERROR, "ERR_APPEXECFWK_INSTALL_SYSCAP_FAILED_AND_DEVICE_TYPE_ERROR"}, diff --git a/services/bundlemgr/src/base_bundle_installer.cpp b/services/bundlemgr/src/base_bundle_installer.cpp index 5eebb1f1470a7834a5e0bbedcfa15905b6ae7381..50d6014469ca3e504db9cf744b87300721e19b77 100644 --- a/services/bundlemgr/src/base_bundle_installer.cpp +++ b/services/bundlemgr/src/base_bundle_installer.cpp @@ -345,7 +345,7 @@ ErrCode BaseBundleInstaller::UninstallBundle(const std::string &bundleName, cons int32_t uid = Constants::INVALID_UID; bool isUninstalledFromBmsExtension = false; ErrCode result = ProcessBundleUninstall(bundleName, installParam, uid); - if (result == ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW) { + if (result == ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL) { CheckBundleNameAndStratAbility(bundleName, appIdentifier_); } if (result == ERR_APPEXECFWK_UNINSTALL_MISSING_INSTALLED_BUNDLE) { @@ -413,7 +413,7 @@ ErrCode BaseBundleInstaller::CheckUninstallInnerBundleInfo(const InnerBundleInfo if (!info.GetUninstallState()) { LOG_E(BMS_TAG_INSTALLER, "bundle : %{public}s can not be uninstalled, uninstallState : %{public}d", bundleName.c_str(), info.GetUninstallState()); - return ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW; + return ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL; } if (info.GetApplicationBundleType() != BundleType::SHARED) { LOG_E(BMS_TAG_INSTALLER, "uninstall bundle is not shared library"); @@ -1750,7 +1750,7 @@ ErrCode BaseBundleInstaller::ProcessBundleUninstall( if (!isForcedUninstall) { LOG_E(BMS_TAG_INSTALLER, "bundle : %{public}s can not be uninstalled, uninstallState : %{public}d", bundleName.c_str(), oldInfo.GetUninstallState()); - return ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW; + return ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL; } } @@ -2028,7 +2028,7 @@ ErrCode BaseBundleInstaller::ProcessBundleUninstall( !oldInfo.GetUninstallState() && installParam.GetKillProcess() && !installParam.GetIsUninstallAndRecover()) { LOG_E(BMS_TAG_INSTALLER, "bundle : %{public}s can not be uninstalled, uninstallState : %{public}d", bundleName.c_str(), oldInfo.GetUninstallState()); - return ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW; + return ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL; } bool isModuleExist = oldInfo.FindModule(modulePackage); diff --git a/services/bundlemgr/src/event_report.cpp b/services/bundlemgr/src/event_report.cpp index 8e064721b0a11efbdd8ff0b105ab75b32da54edf..2e48383db45814c4c526c55262237125821aa8e0 100644 --- a/services/bundlemgr/src/event_report.cpp +++ b/services/bundlemgr/src/event_report.cpp @@ -51,7 +51,6 @@ const std::set INTERCEPTED_ERROR_CODE_SET = { ERR_APPEXECFWK_INSTALL_ENTERPRISE_BUNDLE_NOT_ALLOWED, ERR_BUNDLE_MANAGER_CODE_SIGNATURE_DELIVERY_FILE_FAILED, ERR_APPEXECFWK_INSTALL_PERMISSION_DENIED, - ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL, ERR_APPEXECFWK_INSTALL_EXISTED_ENTERPRISE_BUNDLE_NOT_ALLOWED }; constexpr const char* PARTITION_NAME = "/data"; diff --git a/services/bundlemgr/src/status_receiver_proxy.cpp b/services/bundlemgr/src/status_receiver_proxy.cpp index 6da781dcfe26fef31ac9bd47e3ee43288fd8013b..dd058edccef31a55d1b8033441c140bf446679d0 100644 --- a/services/bundlemgr/src/status_receiver_proxy.cpp +++ b/services/bundlemgr/src/status_receiver_proxy.cpp @@ -533,8 +533,6 @@ const std::unordered_map MAP_RECEIVED_RESULTS { {IStatusReceiver::ERR_INSTALL_DISALLOWED, MSG_ERR_INSTALL_DISALLOWED}}, {ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL, {IStatusReceiver::ERR_UNINSTALL_DISALLOWED, MSG_ERR_UNINSTALL_DISALLOWED}}, - {ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW, - {IStatusReceiver::ERR_UNINSTALL_DISALLOWED, MSG_ERR_UNINSTALL_DISALLOWED}}, {ERR_APPEXECFWK_UNINSTALL_SHARE_APP_LIBRARY_IS_NOT_EXIST, {IStatusReceiver::ERR_APPEXECFWK_UNINSTALL_SHARE_APP_LIBRARY_IS_NOT_EXIST, MSG_ERR_UNINSTALL_SHARE_APP_LIBRARY_NOT_EXIST}}, diff --git a/services/bundlemgr/test/unittest/bms_bundle_data_group_test/bms_bundle_data_group_test.cpp b/services/bundlemgr/test/unittest/bms_bundle_data_group_test/bms_bundle_data_group_test.cpp index c33d91db9976c266d1c2553f44e94d736b081281..206561122af9397893603f947d6bb666e5145b21 100644 --- a/services/bundlemgr/test/unittest/bms_bundle_data_group_test/bms_bundle_data_group_test.cpp +++ b/services/bundlemgr/test/unittest/bms_bundle_data_group_test/bms_bundle_data_group_test.cpp @@ -1547,7 +1547,7 @@ HWTEST_F(BmsBundleDataGroupTest, BaseBundleInstaller_0004, Function | SmallTest installParam.concentrateSendEvent = true; auto result = installer.UninstallBundle(bundleName, installParam); - EXPECT_EQ(result, ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW); + EXPECT_EQ(result, ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL); dataMgr->bundleInfos_.erase(bundleName); } @@ -1600,7 +1600,7 @@ HWTEST_F(BmsBundleDataGroupTest, BaseBundleInstaller_0007, Function | SmallTest info.uninstallState_ = false; std::string bundleName; auto ret = installer.CheckUninstallInnerBundleInfo(info, bundleName); - EXPECT_EQ(ret, ERR_APPEXECFWK_UNINSTALL_STATE_NOT_ALLOW); + EXPECT_EQ(ret, ERR_BUNDLE_MANAGER_APP_CONTROL_DISALLOWED_UNINSTALL); } /**