diff --git a/frameworks/native/backup_ext/src/ext_backup.cpp b/frameworks/native/backup_ext/src/ext_backup.cpp index 2bef6b2d358630a253c3e71b06d3fca1c9074649..53da3a7ea9b9c2cf6393b9cd3208f7897e3627ef 100644 --- a/frameworks/native/backup_ext/src/ext_backup.cpp +++ b/frameworks/native/backup_ext/src/ext_backup.cpp @@ -105,6 +105,11 @@ string ExtBackup::GetUsrConfig() const if (!client.GetProfileFromAbility(info, "ohos.extension.backup", config)) { throw BError(BError::Codes::EXT_INVAL_ARG, "Failed to invoke the GetProfileFromAbility method."); } + if (config.empty()) { + HILOGE("GetUsrConfig empty."); + } else { + HILOGI("GetUsrConfig:%{public}s", config[0].c_str()); + } return config.empty() ? "" : config[0]; } diff --git a/services/backup_sa/src/module_ipc/service.cpp b/services/backup_sa/src/module_ipc/service.cpp index 8952ef796bfb3400953e8a6ce503bff33fffbed4..d004008604c88ea1dcec9a29b331f43a24a62954 100644 --- a/services/backup_sa/src/module_ipc/service.cpp +++ b/services/backup_sa/src/module_ipc/service.cpp @@ -1145,6 +1145,11 @@ void Service::ClearSessionAndSchedInfo(const string &bundleName) sched_->RemoveExtConn(bundleName); HandleRestoreDepsBundle(bundleName); DelClearBundleRecord({bundleName}); + if (isOccupyingSession_.load() && session_->IsOnAllBundlesFinished()) { + SetOccupySession(false); + StopAll(nullptr, true); + return; + } sched_->Sched(); } catch (const BError &e) { return; @@ -1450,11 +1455,6 @@ void Service::ClearResidualBundleData(const std::string &bundleName) backUpConnection->DisconnectBackupExtAbility(); } ClearSessionAndSchedInfo(bundleName); - if (isOccupyingSession_.load() && session_->IsOnAllBundlesFinished()) { - SetOccupySession(false); - StopAll(nullptr, true); - return ; - } // 非清理任务,需要上报 if (session_->GetScenario() != IServiceReverse::Scenario::CLEAN) { OnAllBundlesFinished(BError(BError::Codes::OK)); @@ -1585,7 +1585,7 @@ ErrCode Service::AppendBundlesClearSession(const std::vector &bundle try { if (bundleNames.empty() || session_ == nullptr) { HILOGE("Init Incremental backup session error, session is empty"); - return BError(BError::Codes::SA_INVAL_ARG); + return EPERM; } session_->IncreaseSessionCnt(__PRETTY_FUNCTION__); // BundleMgrAdapter::GetBundleInfos可能耗时 auto backupInfos = BundleMgrAdapter::GetBundleInfos(bundleNames, session_->GetSessionUserId()); diff --git a/utils/src/b_json/b_json_entity_extension_config.cpp b/utils/src/b_json/b_json_entity_extension_config.cpp index 0d93a72fe89af3ab4d50bb3e2d4cbc5835bdd6ff..915ece4cabfc699b5d3f145ded71782f07d4fdb6 100644 --- a/utils/src/b_json/b_json_entity_extension_config.cpp +++ b/utils/src/b_json/b_json_entity_extension_config.cpp @@ -37,7 +37,7 @@ vector BJsonEntityExtensionConfig::GetIncludes() const return {BConstants::PATHES_TO_BACKUP.begin(), BConstants::PATHES_TO_BACKUP.end()}; } if (!obj_.isMember("includes")) { - HILOGD("'includes' field not found"); + HILOGE("'includes' field not found"); return {BConstants::PATHES_TO_BACKUP.begin(), BConstants::PATHES_TO_BACKUP.end()}; } if (!obj_["includes"].isArray()) {