diff --git a/frameworks/native/backup_ext/src/ext_backup_js.cpp b/frameworks/native/backup_ext/src/ext_backup_js.cpp index ad919e7a95bd52eb04035836524f0533d6b7e0bb..5f5982a0d901acda31bc9562f3fa6eac5933e5e8 100644 --- a/frameworks/native/backup_ext/src/ext_backup_js.cpp +++ b/frameworks/native/backup_ext/src/ext_backup_js.cpp @@ -248,7 +248,12 @@ ErrCode ExtBackupJs::OnBackup(function callback) return true; } HILOGI("CheckPromise(JS) OnBackup ok."); - return CallPromise(*jsRuntime, result, callbackInfo.get()); + auto ret = CallPromise(*jsRuntime, result, callbackInfo.get()); + if (!ret) { + HILOGE("CallPromise(JS) OnBackup fail."); + callbackInfo->callback(); + } + return ret; }; auto errCode = CallJsMethod("onBackup", jsRuntime_, jsObj_.get(), {}, retParser); @@ -281,7 +286,12 @@ ErrCode ExtBackupJs::OnRestore(function callback) return true; } HILOGI("CheckPromise(JS) OnRestore ok."); - return CallPromise(*jsRuntime, result, callbackInfo.get()); + auto ret = CallPromise(*jsRuntime, result, callbackInfo.get()); + if (!ret) { + HILOGE("CallPromise(JS) OnRestore fail."); + callbackInfo->callback(); + } + return ret; }; auto errCode = CallJsMethod("onRestore", jsRuntime_, jsObj_.get(), argParser, retParser); diff --git a/frameworks/native/backup_ext/src/ext_extension.cpp b/frameworks/native/backup_ext/src/ext_extension.cpp index 10feae97b3a229f20b97beec8954ae84e9888be2..338963a1531af677f62bf8580d7ecd04516b543c 100644 --- a/frameworks/native/backup_ext/src/ext_extension.cpp +++ b/frameworks/native/backup_ext/src/ext_extension.cpp @@ -289,6 +289,7 @@ int BackupExtExtension::DoBackup(const BJsonEntityExtensionConfig &usrConfig) auto proxy = ServiceProxy::GetInstance(); if (proxy == nullptr) { + HILOGE("Failed to obtain the ServiceProxy handle"); throw BError(BError::Codes::EXT_BROKEN_BACKUP_SA, std::generic_category().message(errno)); } diff --git a/utils/src/b_tarball/b_tarball_cmdline.cpp b/utils/src/b_tarball/b_tarball_cmdline.cpp index bb286faf11bf4ddbbb093cf33524d8dda448f48d..ffbfcff9afa90458f0c263297964222f3d525a05 100644 --- a/utils/src/b_tarball/b_tarball_cmdline.cpp +++ b/utils/src/b_tarball/b_tarball_cmdline.cpp @@ -47,8 +47,15 @@ static void VerifyArgv(const vector &argv) static bool IsTarFatalErrorOccur(string_view output) { - vector fatalError {"EOF", "bad xform", "bad header", "sparse overflow", - "short header", "empty archive", "Not tar"}; + vector fatalError {"EOF", + "bad xform", + "bad header", + "sparse overflow", + "short header", + "empty archive", + "Not tar", + "Operation not permitted", + "Permission denied"}; for (auto &item : fatalError) { if (output.find(item) != string_view::npos) { return true;