diff --git a/frameworks/js/napi/vpn/src/context/get_app_info_context.cpp b/frameworks/js/napi/vpn/src/context/get_app_info_context.cpp index 9a1e30f606933062f8d4ed3404e5294ce1ebcc59..4e6216cd4f53edb184049c09dfd1b451399dc727 100644 --- a/frameworks/js/napi/vpn/src/context/get_app_info_context.cpp +++ b/frameworks/js/napi/vpn/src/context/get_app_info_context.cpp @@ -34,7 +34,7 @@ void GetAppInfoContext::ParseParams(napi_value *params, size_t paramsCount) SetParseOK(SetCallback(params[0]) == napi_ok); break; default: - SetErrorCode(NETMANAGER_EXT_ERR_PARAMETER_ERROR); + SetErrorCode(NETMANAGER_ERR_PARAMETER_INVALID); SetNeedThrowException(true); break; } diff --git a/frameworks/js/napi/vpn/src/networkvpn_service.cpp b/frameworks/js/napi/vpn/src/networkvpn_service.cpp index 578e6a5992d788d1cf9c6796a6c74208c6c87529..d93b755b86272936efab471008d66ea36fb3d8d5 100644 --- a/frameworks/js/napi/vpn/src/networkvpn_service.cpp +++ b/frameworks/js/napi/vpn/src/networkvpn_service.cpp @@ -861,7 +861,7 @@ int32_t NetworkVpnService::DestroyVpn(const std::string &vpnId) { if (vpnId.empty()) { NETMGR_EXT_LOG_E("vpnId is empty"); - return NETMANAGER_EXT_ERR_PARAMETER_ERROR; + return NETMANAGER_ERR_PARAMETER_INVALID; } std::unique_lock locker(netVpnMutex_); std::string vpnBundleName = GetBundleName(); @@ -1186,7 +1186,7 @@ int32_t NetworkVpnService::GetConnectedVpnAppInfo(std::vector &bund if (vpnObj == nullptr || vpnObj->multiVpnInfo_ == nullptr) { NETMGR_EXT_LOG_E("GetConnectedVpnAppInfo failed, vpnObj invalid"); it = vpnObjMap_.erase(it); - return NETMANAGER_EXT_ERR_INTERNAL; + return NETMANAGER_ERR_SYSTEM_INTERNAL; } if (userId == vpnObj->multiVpnInfo_->userId) { std::string name = vpnObj->multiVpnInfo_->bundleName; @@ -1331,7 +1331,7 @@ int32_t NetworkVpnService::RegisterMultiVpnEvent(const sptr & if (!CheckVpnPermission(vpnBundleName)) { return NETMANAGER_EXT_ERR_PERMISSION_DENIED; } - int32_t ret = NETMANAGER_EXT_ERR_OPERATION_FAILED; + int32_t ret = NETMANAGER_ERR_SYSTEM_INTERNAL; if (!networkVpnServiceFfrtQueue_) { NETMGR_EXT_LOG_E("FFRT Create Fail"); return ret; @@ -1350,7 +1350,7 @@ int32_t NetworkVpnService::UnregisterMultiVpnEvent(const sptr if (!CheckVpnPermission(vpnBundleName)) { return NETMANAGER_EXT_ERR_PERMISSION_DENIED; } - int32_t ret = NETMANAGER_EXT_ERR_OPERATION_FAILED; + int32_t ret = NETMANAGER_ERR_SYSTEM_INTERNAL; if (!networkVpnServiceFfrtQueue_) { NETMGR_EXT_LOG_E("FFRT Create Fail"); return ret; @@ -1469,21 +1469,21 @@ int32_t NetworkVpnService::SyncRegisterMultiVpnEvent(const sptr autoLock(remoteMutex_); for (auto iterCb = multiVpnEventCallbacks_.begin(); iterCb != multiVpnEventCallbacks_.end(); iterCb++) { if (((*iterCb)->callback)->AsObject().GetRefPtr() == callback->AsObject().GetRefPtr()) { NETMGR_EXT_LOG_E("Register multi vpn event callback failed, callback already exists"); - return NETMANAGER_EXT_ERR_OPERATION_FAILED; + return NETMANAGER_ERR_SYSTEM_INTERNAL; } } if (multiVpnEventCallbacks_.size() >= MAX_CALLBACK_COUNT) { NETMGR_EXT_LOG_E("callback above max count, return error."); - return NETMANAGER_EXT_ERR_PARAMETER_ERROR; + return NETMANAGER_ERR_PARAMETER_INVALID; } if (!AddClientDeathRecipient(callback)) { - return NETMANAGER_EXT_ERR_OPERATION_FAILED; + return NETMANAGER_ERR_SYSTEM_INTERNAL; } int32_t userId = AppExecFwk::Constants::UNSPECIFIED_USERID; if (AccountSA::OsAccountManager::GetForegroundOsAccountLocalId(userId) != ERR_OK) { @@ -1515,7 +1515,7 @@ int32_t NetworkVpnService::SyncUnregisterMultiVpnEvent(const sptr proxy = GetProxy(); if (proxy == nullptr) { NETMGR_EXT_LOG_E("DestroyVpn proxy is nullptr"); - return NETMANAGER_EXT_ERR_GET_PROXY_FAIL; + return NETMANAGER_ERR_SYSTEM_INTERNAL; } return proxy->DestroyVpn(vpnId); } @@ -349,7 +349,7 @@ int32_t NetworkVpnClient::GetConnectedVpnAppInfo(std::vector &bundl sptr proxy = GetProxy(); if (proxy == nullptr) { NETMGR_EXT_LOG_E("GetConnectedVpnAppInfo proxy is nullptr"); - return NETMANAGER_EXT_ERR_GET_PROXY_FAIL; + return NETMANAGER_ERR_SYSTEM_INTERNAL; } return proxy->GetConnectedVpnAppInfo(bundleNameList); }