diff --git a/frameworks/native/backup_ext/src/tar_file.cpp b/frameworks/native/backup_ext/src/tar_file.cpp
index 51dba1aac08baeda8dd1f25141cd5e0416b03997..6f456bf84184a35f4396e467a211c7cb39144f32 100644
--- a/frameworks/native/backup_ext/src/tar_file.cpp
+++ b/frameworks/native/backup_ext/src/tar_file.cpp
@@ -307,7 +307,7 @@ bool TarFile::WriteFileContent(const string &fileName, off_t size, int &err)
     int fd = open(fileName.c_str(), O_RDONLY | O_CLOEXEC);
     if (fd < 0) {
         err = errno;
-        HILOGE("Failed to open file %{public}s, err = %{public}d", fileName.data(), errno);
+        HILOGE("Failed to open file %{public}s, err = %{public}d", GetAnonyString(fileName).data(), errno);
         return false;
     }
 
diff --git a/frameworks/native/backup_ext/src/untar_file.cpp b/frameworks/native/backup_ext/src/untar_file.cpp
index 5e40a672583d0dbbfb4607be77b1d588084889d6..da37232158a225fd1588bb75581f58a9aac4e813 100644
--- a/frameworks/native/backup_ext/src/untar_file.cpp
+++ b/frameworks/native/backup_ext/src/untar_file.cpp
@@ -284,9 +284,9 @@ int UntarFile::DealIncreParseTarFileResult(const std::tuple<int, bool, ErrFileIn
     }
     if (!isFilter) {
         fileInfos[fileName] = fileSize;
-        if (!subErrInfo.empty()) {
-            errInfos.merge(subErrInfo);
-        }
+    }
+    if (!subErrInfo.empty()) {
+        errInfos.merge(subErrInfo);
     }
     return 0;
 }
@@ -478,7 +478,7 @@ bool UntarFile::IsValidTarBlock(TarHeader &header)
     if (strncmp(header.magic, TMAGIC.c_str(), TMAGIC_LEN - 1) == 0 && VerifyChecksum(header)) {
         return true;
     }
-    HILOGW("Invalid tar block");
+    HILOGE("Invalid tar block");
     return false;
 }
 
@@ -510,7 +510,7 @@ ErrFileInfo UntarFile::CreateDir(string &path, mode_t mode)
         path[len - 1] = '\0';
     }
     if (access(path.c_str(), F_OK) != 0) {
-        HILOGW("directory does not exist, path:%{public}s, err = %{public}d", path.c_str(), errno);
+        HILOGE("directory does not exist, path:%{public}s, err = %{public}d", path.c_str(), errno);
         if (!ForceCreateDirectoryWithMode(path, mode)) {
             HILOGE("Failed to force create directory %{public}s, err = %{public}d", path.c_str(), errno);
             errFileInfo[path].push_back(errno);
@@ -527,7 +527,7 @@ FILE *UntarFile::CreateFile(string &filePath, mode_t mode, char fileType)
     }
 
     uint32_t len = filePath.length();
-    HILOGW("Failed to open file %{public}d, %{public}s, err = %{public}d", len,
+    HILOGE("Failed to open file %{public}d, %{public}s, err = %{public}d", len,
         GetAnonyPath(filePath).c_str(), errno);
     size_t pos = filePath.rfind('/');
     if (pos == string::npos) {
@@ -538,7 +538,7 @@ FILE *UntarFile::CreateFile(string &filePath, mode_t mode, char fileType)
     if (ForceCreateDirectory(path)) {
         f = fopen(filePath.c_str(), "wb+");
         if (f == nullptr) {
-            HILOGE("Failed to create file %{public}s, err = %{public}d", GetAnonyPath(filePath).c_str(), errno);
+            HILOGE("Failed to open file %{public}s, err = %{public}d", GetAnonyPath(filePath).c_str(), errno);
         }
     }
 
diff --git a/services/backup_sa/include/module_ipc/svc_session_manager.h b/services/backup_sa/include/module_ipc/svc_session_manager.h
index e420dc5fccfbf0c7f1d6db24208e5a5fa4f298a7..2671c0fd65ca21327ff38d6bddddf5081365c02b 100644
--- a/services/backup_sa/include/module_ipc/svc_session_manager.h
+++ b/services/backup_sa/include/module_ipc/svc_session_manager.h
@@ -60,7 +60,7 @@ struct BackupExtInfo {
     /* Timer Status: true is start & false is stop */
     bool extTimerStatus {false};
     bool fwkTimerStatus {false};
-    uint32_t timeout;
+    uint32_t timeout = BConstants::TIMEOUT_INVALID;
     uint32_t startTime;
     int64_t dataSize;
     int64_t lastIncrementalTime;
diff --git a/services/backup_sa/src/module_external/bms_adapter.cpp b/services/backup_sa/src/module_external/bms_adapter.cpp
index e1f415e89dfa100823f36734360d015f3d1f25f8..072902b0d7e1504625aaebc8aacf6e67e0eecc3a 100644
--- a/services/backup_sa/src/module_external/bms_adapter.cpp
+++ b/services/backup_sa/src/module_external/bms_adapter.cpp
@@ -342,7 +342,7 @@ vector<BJsonEntityCaps::BundleInfo> BundleMgrAdapter::GetBundleInfosForIncrement
 
     vector<BIncrementalData> bundleNames;
     vector<BJsonEntityCaps::BundleInfo> bundleInfos;
-    HILOGI("End get installedBundles count is:%{public}zu", installedBundles.size());
+    HILOGI("Begin get bundle infos");
     for (auto const &installedBundle : installedBundles) {
         if (installedBundle.applicationInfo.codePath == HMOS_HAP_CODE_PATH ||
             installedBundle.applicationInfo.codePath == LINUX_HAP_CODE_PATH) {
@@ -353,8 +353,8 @@ vector<BJsonEntityCaps::BundleInfo> BundleMgrAdapter::GetBundleInfosForIncrement
             GetAllowAndExtName(installedBundle.extensionInfos);
         if (!allToBackup) {
             bundleInfos.emplace_back(BJsonEntityCaps::BundleInfo {installedBundle.name, installedBundle.appIndex,
-                installedBundle.versionCode, installedBundle.versionName, 0, 0, allToBackup, fullBackupOnly, extName,
-                restoreDeps, supportScene, extraInfo});
+                installedBundle.versionCode, installedBundle.versionName, 0, 0, allToBackup, fullBackupOnly,
+                extName, restoreDeps, supportScene, extraInfo});
             continue;
         }
         auto it = std::find_if(extraIncreData.begin(), extraIncreData.end(),
@@ -385,15 +385,14 @@ vector<BJsonEntityCaps::BundleInfo> BundleMgrAdapter::GetFullBundleInfos(int32_t
     }
     vector<string> bundleNames;
     vector<BJsonEntityCaps::BundleInfo> bundleInfos;
-    HILOGI("End get installedBundles count is:%{public}zu", installedBundles.size());
     for (auto const &installedBundle : installedBundles) {
+        HILOGI("Begin get bundle infos, bundleName = %{public}s", installedBundle.name.data());
         if (installedBundle.applicationInfo.codePath == HMOS_HAP_CODE_PATH ||
             installedBundle.applicationInfo.codePath == LINUX_HAP_CODE_PATH) {
             HILOGI("Unsupported applications, name : %{public}s", installedBundle.name.data());
             continue;
         }
         if (installedBundle.appIndex > 0) {
-            HILOGI("Current bundle %{public}s is a twin application", installedBundle.name.c_str());
             std::string bundleNameIndexInfo = BJsonUtil::BuildBundleNameIndexInfo(installedBundle.name,
                 installedBundle.appIndex);
             bundleNames.emplace_back(bundleNameIndexInfo);
diff --git a/services/backup_sa/src/module_ipc/svc_restore_deps_manager.cpp b/services/backup_sa/src/module_ipc/svc_restore_deps_manager.cpp
index 70879196a310d2eff12400e017c4bb4832d96ab4..682b0a2b4ecc6514ba7244c64fa014bf618c17cb 100644
--- a/services/backup_sa/src/module_ipc/svc_restore_deps_manager.cpp
+++ b/services/backup_sa/src/module_ipc/svc_restore_deps_manager.cpp
@@ -33,7 +33,7 @@ vector<string> SvcRestoreDepsManager::GetRestoreBundleNames(const vector<BJsonEn
         restoreBundleNames.emplace_back(bundleNameIndexInfo);
         string restoreDeps = bundleInfo.restoreDeps;
         if (!restoreDeps.empty()) {
-            HILOGI("RestoreDeps is not empty, bundleName=%{public}s", bundleNameIndexInfo.c_str());
+            HILOGI("RestoreDeps is not empty, bundleName=%{public}s", bundleInfo.name.c_str());
             if (IsAllDepsRestored(bundleNameIndexInfo)) {
                 HILOGI("RestoreDeps is all restored, bundleName=%{public}s", bundleInfo.name.c_str());
             }
diff --git a/services/backup_sa/src/module_ipc/svc_session_manager.cpp b/services/backup_sa/src/module_ipc/svc_session_manager.cpp
index ada69b6488f29bb0af5a556449c5195f142e9787..6e2b04b8813abaf4c981543fab9a1a426b5f84a3 100644
--- a/services/backup_sa/src/module_ipc/svc_session_manager.cpp
+++ b/services/backup_sa/src/module_ipc/svc_session_manager.cpp
@@ -163,7 +163,7 @@ bool SvcSessionManager::OnBundleFileReady(const string &bundleName, const string
     if (!impl_.clientToken) {
         throw BError(BError::Codes::SA_INVAL_ARG, "No caller token was specified");
     }
-    HILOGD("Begin, bundleName name is:%{public}s", bundleName.c_str());
+    HILOGD("Begin, bundleName name is:%{private}s", bundleName.c_str());
     auto it = impl_.backupExtNameMap.find(bundleName);
     if (it == impl_.backupExtNameMap.end()) {
         stringstream ss;
@@ -190,7 +190,7 @@ bool SvcSessionManager::OnBundleFileReady(const string &bundleName, const string
             return true;
         }
     }
-    HILOGD("End, bundleName name is:%{public}s", bundleName.c_str());
+    HILOGD("End, bundleName name is:%{private}s", bundleName.c_str());
     return false;
 }
 
@@ -574,7 +574,6 @@ bool SvcSessionManager::IsOnAllBundlesFinished()
 
 bool SvcSessionManager::IsOnOnStartSched()
 {
-    HILOGI("Begin");
     shared_lock<shared_mutex> lock(lock_);
     if (!impl_.clientToken) {
         throw BError(BError::Codes::SA_INVAL_ARG, "No caller token was specified");
@@ -582,7 +581,7 @@ bool SvcSessionManager::IsOnOnStartSched()
     if (impl_.isBackupStart && impl_.backupExtNameMap.size()) {
         return true;
     }
-    HILOGI("End");
+
     return false;
 }
 
diff --git a/tests/mock/napi/include/napi_mock.h b/tests/mock/napi/include/napi_mock.h
index bc699b791d8ba1548bcf6abc11f318245c9546d1..5f70f4f022c00af96c55e54410330bc2f22a0541 100644
--- a/tests/mock/napi/include/napi_mock.h
+++ b/tests/mock/napi/include/napi_mock.h
@@ -60,6 +60,7 @@ public:
     virtual napi_status napi_create_function(napi_env, const char*, size_t, napi_callback, void*, napi_value*) = 0;
     virtual napi_status napi_open_handle_scope(napi_env, napi_handle_scope*) = 0;
     virtual napi_status napi_close_handle_scope(napi_env, napi_handle_scope) = 0;
+    virtual napi_status napi_is_exception_pending(napi_env, bool*) = 0;
 public:
     static inline std::shared_ptr<Napi> napi = nullptr;
 };
@@ -96,6 +97,7 @@ public:
     MOCK_METHOD6(napi_create_function, napi_status(napi_env, const char*, size_t, napi_callback, void*, napi_value*));
     MOCK_METHOD2(napi_open_handle_scope, napi_status(napi_env, napi_handle_scope*));
     MOCK_METHOD2(napi_close_handle_scope, napi_status(napi_env, napi_handle_scope));
+    MOCK_METHOD2(napi_is_exception_pending, napi_status(napi_env, bool*));
 };
 } // namespace OHOS::FileManagement::Backup
 #endif // TEST_UNITTEST_MOCK_NAPI_H
\ No newline at end of file
diff --git a/tests/mock/napi/src/napi_mock.cpp b/tests/mock/napi/src/napi_mock.cpp
index 0310429d36660f0c13e192b167aa541ec70dd138..5b672ce665a256f232ce8cfa4b2bb3360c68aa84 100644
--- a/tests/mock/napi/src/napi_mock.cpp
+++ b/tests/mock/napi/src/napi_mock.cpp
@@ -18,12 +18,6 @@
 
 int uv_queue_work(uv_loop_t* loop, uv_work_t* req, uv_work_cb work_cb, uv_after_work_cb after_work_cb)
 {
-    if (work_cb) {
-        work_cb(req);
-    }
-    if (after_work_cb) {
-        after_work_cb(req, 0);
-    }
     return OHOS::FileManagement::Backup::Napi::napi->uv_queue_work(loop, req, work_cb, after_work_cb);
 }
 
@@ -163,4 +157,9 @@ NAPI_EXTERN napi_status napi_open_handle_scope(napi_env env, napi_handle_scope*
 NAPI_EXTERN napi_status napi_close_handle_scope(napi_env env, napi_handle_scope scope)
 {
     return OHOS::FileManagement::Backup::Napi::napi->napi_close_handle_scope(env, scope);
+}
+
+napi_status napi_is_exception_pending(napi_env env, bool* result)
+{
+    return OHOS::FileManagement::Backup::Napi::napi->napi_is_exception_pending(env, result);
 }
\ No newline at end of file
diff --git a/tests/unittests/backup_sa/module_ipc/service_test.cpp b/tests/unittests/backup_sa/module_ipc/service_test.cpp
index e0224ccd38263d306beedca7dfdac38f3d0189d1..21493107959d46e21434d476a5a34b574f424818 100644
--- a/tests/unittests/backup_sa/module_ipc/service_test.cpp
+++ b/tests/unittests/backup_sa/module_ipc/service_test.cpp
@@ -22,6 +22,7 @@
 #include "module_ipc/service.h"
 #include "service.cpp"
 #include "service_reverse_mock.h"
+#include "sub_service.cpp"
 #include "test_manager.h"
 
 namespace OHOS::FileManagement::Backup {
diff --git a/tests/unittests/backup_utils/b_json/b_json_entity_ext_manage_test.cpp b/tests/unittests/backup_utils/b_json/b_json_entity_ext_manage_test.cpp
index a09c55ebcb4078fde2d0f056290d2126acc82c6c..239a274bd5d832f0ff427e16b675148c0f5a7598 100644
--- a/tests/unittests/backup_utils/b_json/b_json_entity_ext_manage_test.cpp
+++ b/tests/unittests/backup_utils/b_json/b_json_entity_ext_manage_test.cpp
@@ -526,6 +526,31 @@ HWTEST_F(BJsonEntityExtManageTest, b_json_entity_ext_manage_0803, testing::ext::
     GTEST_LOG_(INFO) << "BJsonEntityExtManageTest-end b_json_entity_ext_manage_0803";
 }
 
+/**
+ * @tc.number: SUB_backup_b_json_entity_ext_manage_0804
+ * @tc.name: b_json_entity_ext_manage_0804
+ * @tc.desc: 测试GetExtManageInfo
+ * @tc.size: MEDIUM
+ * @tc.type: FUNC
+ * @tc.level Level 0
+ * @tc.require: I6F3GV
+ */
+HWTEST_F(BJsonEntityExtManageTest, b_json_entity_ext_manage_0804, testing::ext::TestSize.Level0)
+{
+    GTEST_LOG_(INFO) << "BJsonEntityExtManageTest-begin b_json_entity_ext_manage_0804";
+    try {
+        string_view sv = R"([{"isBigFile":false}, {"fileName":"test"}])";
+        BJsonCachedEntity<BJsonEntityExtManage> cachedEntity(sv);
+        auto cache = cachedEntity.Structuralize();
+        auto mp = cache.GetExtManageInfo();
+        EXPECT_TRUE(mp.empty());
+    } catch (...) {
+        EXPECT_TRUE(false);
+        GTEST_LOG_(INFO) << "BJsonEntityExtManageTest-an exception occurred.";
+    }
+    GTEST_LOG_(INFO) << "BJsonEntityExtManageTest-end b_json_entity_ext_manage_0804";
+}
+
 /**
  * @tc.number: SUB_backup_b_json_entity_ext_manage_0900
  * @tc.name: b_json_entity_ext_manage_0900
@@ -674,4 +699,32 @@ HWTEST_F(BJsonEntityExtManageTest, b_json_entity_ext_manage_0903, testing::ext::
     }
     GTEST_LOG_(INFO) << "BJsonEntityExtManageTest-end b_json_entity_ext_manage_0903";
 }
+
+/**
+ * @tc.number: SUB_backup_b_json_entity_ext_manage_0904
+ * @tc.name: b_json_entity_ext_manage_0904
+ * @tc.desc: 测试CheckOwnPackTar各种异常分支
+ * @tc.size: MEDIUM
+ * @tc.type: FUNC
+ * @tc.level Level 1
+ * @tc.require: issuesI9JXNH
+ */
+HWTEST_F(BJsonEntityExtManageTest, b_json_entity_ext_manage_0904, testing::ext::TestSize.Level1)
+{
+    GTEST_LOG_(INFO) << "BJsonEntityExtManageTest-begin b_json_entity_ext_manage_0904";
+    try {
+        string fileName = "/home/user/test.tar";
+        auto ret = CheckOwnPackTar(fileName);
+        EXPECT_FALSE(ret);
+
+        fileName = string(BConstants::PATH_BUNDLE_BACKUP_HOME)
+            .append(BConstants::SA_BUNDLE_BACKUP_BACKUP).append("/part1.tar");
+        ret = CheckOwnPackTar(fileName);
+        EXPECT_TRUE(ret);
+    } catch (...) {
+        EXPECT_TRUE(false);
+        GTEST_LOG_(INFO) << "BJsonEntityExtManageTest-an exception occurred.";
+    }
+    GTEST_LOG_(INFO) << "BJsonEntityExtManageTest-end b_json_entity_ext_manage_0904";
+}
 } // namespace OHOS::FileManagement::Backup
\ No newline at end of file
diff --git a/tests/unittests/backup_utils/b_json/b_report_entity_test.cpp b/tests/unittests/backup_utils/b_json/b_report_entity_test.cpp
index 3e51ac25818a0f48ff46284217bc782764ccb5bf..cbdba7332b544426885262d980bc04c16559f53b 100644
--- a/tests/unittests/backup_utils/b_json/b_report_entity_test.cpp
+++ b/tests/unittests/backup_utils/b_json/b_report_entity_test.cpp
@@ -227,4 +227,124 @@ HWTEST_F(BReportEntityTest, b_report_entity_DealLine_0100, testing::ext::TestSiz
     GTEST_LOG_(INFO) << "BReportEntityTest-end b_report_entity_DealLine_0100";
 }
 
+/**
+ * @tc.number: SUB_backup_b_report_entity_DealLine_0101
+ * @tc.name: b_report_entity_DealLine_0101
+ * @tc.desc: Test function of DealLine interface for SUCCESS.
+ * @tc.size: MEDIUM
+ * @tc.type: FUNC
+ * @tc.level Level 1
+ */
+HWTEST_F(BReportEntityTest, b_report_entity_DealLine_0101, testing::ext::TestSize.Level1)
+{
+    GTEST_LOG_(INFO) << "BReportEntityTest-begin b_report_entity_DealLine_0101";
+    try {
+        unordered_map<string, int> keys;
+        int num = 1;
+        string line = "";
+        unordered_map<string, struct ReportFileInfo> infos;
+        DealLine(keys, num, line, infos);
+        EXPECT_EQ(infos.size(), 0);
+    } catch (const exception &e) {
+        GTEST_LOG_(INFO) << "BReportEntityTest-an exception occurred by DealLine. " << e.what();
+        EXPECT_TRUE(false);
+    }
+    GTEST_LOG_(INFO) << "BReportEntityTest-end b_report_entity_DealLine_0101";
+}
+
+/**
+ * @tc.number: SUB_backup_b_report_entity_StorageDealLine_0100
+ * @tc.name: b_report_entity_StorageDealLine_0100
+ * @tc.desc: Test function of DealLine interface for SUCCESS.
+ * @tc.size: MEDIUM
+ * @tc.type: FUNC
+ * @tc.level Level 1
+ */
+HWTEST_F(BReportEntityTest, b_report_entity_StorageDealLine_0100, testing::ext::TestSize.Level1)
+{
+    GTEST_LOG_(INFO) << "BReportEntityTest-begin b_report_entity_StorageDealLine_0100";
+    try {
+        unordered_map<string, int> keys;
+        int num = 1;
+        string line = "";
+        StorageDealLine(keys, num, line);
+        EXPECT_TRUE(true);
+    } catch (const exception &e) {
+        GTEST_LOG_(INFO) << "BReportEntityTest-an exception occurred by DealLine. " << e.what();
+        EXPECT_TRUE(false);
+    }
+    GTEST_LOG_(INFO) << "BReportEntityTest-end b_report_entity_StorageDealLine_0100";
+}
+
+/**
+ * @tc.number: SUB_backup_b_report_entity_StorageDealLine_0101
+ * @tc.name: b_report_entity_StorageDealLine_0101
+ * @tc.desc: Test function of DealLine interface for SUCCESS.
+ * @tc.size: MEDIUM
+ * @tc.type: FUNC
+ * @tc.level Level 1
+ */
+HWTEST_F(BReportEntityTest, b_report_entity_StorageDealLine_0101, testing::ext::TestSize.Level1)
+{
+    GTEST_LOG_(INFO) << "BReportEntityTest-begin b_report_entity_StorageDealLine_0101";
+    try {
+        unordered_map<string, int> keys;
+        int num = 0;
+        string line = "test\r";
+        StorageDealLine(keys, num, line);
+        EXPECT_TRUE(true);
+    } catch (const exception &e) {
+        GTEST_LOG_(INFO) << "BReportEntityTest-an exception occurred by DealLine. " << e.what();
+        EXPECT_TRUE(false);
+    }
+    GTEST_LOG_(INFO) << "BReportEntityTest-end b_report_entity_StorageDealLine_0101";
+}
+
+/**
+ * @tc.number: SUB_backup_b_report_entity_StorageDealLine_0102
+ * @tc.name: b_report_entity_StorageDealLine_0102
+ * @tc.desc: Test function of DealLine interface for SUCCESS.
+ * @tc.size: MEDIUM
+ * @tc.type: FUNC
+ * @tc.level Level 1
+ */
+HWTEST_F(BReportEntityTest, b_report_entity_StorageDealLine_0102, testing::ext::TestSize.Level1)
+{
+    GTEST_LOG_(INFO) << "BReportEntityTest-begin b_report_entity_StorageDealLine_0102";
+    try {
+        unordered_map<string, int> keys;
+        int num = 1;
+        string line = "key1;key2;key3";
+        StorageDealLine(keys, num, line);
+        EXPECT_TRUE(true);
+    } catch (const exception &e) {
+        GTEST_LOG_(INFO) << "BReportEntityTest-an exception occurred by DealLine. " << e.what();
+        EXPECT_TRUE(false);
+    }
+    GTEST_LOG_(INFO) << "BReportEntityTest-end b_report_entity_StorageDealLine_0102";
+}
+
+/**
+ * @tc.number: SUB_backup_b_report_entity_StorageDealLine_0103
+ * @tc.name: b_report_entity_StorageDealLine_0103
+ * @tc.desc: Test function of DealLine interface for SUCCESS.
+ * @tc.size: MEDIUM
+ * @tc.type: FUNC
+ * @tc.level Level 1
+ */
+HWTEST_F(BReportEntityTest, b_report_entity_StorageDealLine_0103, testing::ext::TestSize.Level1)
+{
+    GTEST_LOG_(INFO) << "BReportEntityTest-begin b_report_entity_StorageDealLine_0103";
+    try {
+        unordered_map<string, int> keys;
+        int num = INFO_ALIGN_NUM;
+        string line = "key1;key2;key3";
+        StorageDealLine(keys, num, line);
+        EXPECT_TRUE(true);
+    } catch (const exception &e) {
+        GTEST_LOG_(INFO) << "BReportEntityTest-an exception occurred by DealLine. " << e.what();
+        EXPECT_TRUE(false);
+    }
+    GTEST_LOG_(INFO) << "BReportEntityTest-end b_report_entity_StorageDealLine_0103";
+}
 } // namespace OHOS::FileManagement::Backup
\ No newline at end of file
diff --git a/utils/include/b_jsonutil/b_jsonutil.h b/utils/include/b_jsonutil/b_jsonutil.h
index 0758165801532e06bcdb7359e3d69af6aa063335..37c050a31f1ff8ad7429d2a0c358d5420244d324 100644
--- a/utils/include/b_jsonutil/b_jsonutil.h
+++ b/utils/include/b_jsonutil/b_jsonutil.h
@@ -96,17 +96,7 @@ public:
      */
     static bool BuildExtensionErrInfo(std::string &jsonStr, int errCode, std::string errMsg);
 
-    /**
-     * @brief 拼接包名和分身对应的索引
-     *
-     * @param bundleName 包名
-     * @param bundleIndex 索引
-     *
-     * @return 拼接结果
-     */
-    static std::string BuildBundleNameIndexInfo(const std::string &bundleName, int bundleIndex);
-
-    /**
+    /* *
      * @brief 组建恢复文件错误信息的json
      *
      * @param jsonStr 组建结果
@@ -117,9 +107,8 @@ public:
      */
     static bool BuildExtensionErrInfo(std::string &jsonStr, std::map<std::string, std::vector<int>> errFileInfo);
 
-    /**
+    /* *
      * @brief 组建App进度返回的信息
-     *
      * @param jsonStr 组建结果
      * @param onProcessRet onProcess接口返回值
      *
@@ -128,6 +117,15 @@ public:
      */
     static bool BuildOnProcessRetInfo(std::string &jsonStr, std::string onProcessRet);
 
+    /* *
+     * @brief 拼接包名和分身对应的索引
+     *
+     * @param bundleName 包名
+     * @param bundleIndex 索引
+     *
+     * @return 拼接结果
+     */
+    static std::string BuildBundleNameIndexInfo(const std::string &bundleName, int bundleIndex);
     /**
      * @brief 组建App进度返回的信息
      *
diff --git a/utils/src/b_jsonutil/b_jsonutil.cpp b/utils/src/b_jsonutil/b_jsonutil.cpp
index f3e55f4f14a24599ecca13e2df766c27996ed290..751054b9a6c0a685d914be3da6b1d6f9a3beaa7c 100644
--- a/utils/src/b_jsonutil/b_jsonutil.cpp
+++ b/utils/src/b_jsonutil/b_jsonutil.cpp
@@ -167,13 +167,11 @@ bool BJsonUtil::HasUnicastInfo(std::string &bundleInfo)
         cJSON *infoItem = cJSON_GetArrayItem(infos, i);
         if (!cJSON_IsObject(infoItem)) {
             HILOGE("Parse json error, info item is not an object");
-            cJSON_Delete(root);
             continue;
         }
         cJSON *type = cJSON_GetObjectItem(infoItem, "type");
         if (type == nullptr || !cJSON_IsString(type) || (type->valuestring == nullptr)) {
             HILOGE("Parse json type element error");
-            cJSON_Delete(root);
             continue;
         }
         if (string(type->valuestring).compare(BConstants::UNICAST_TYPE) == 0) {
@@ -306,17 +304,6 @@ bool BJsonUtil::BuildExtensionErrInfo(std::string &jsonStr, int errCode, std::st
     return true;
 }
 
-std::string BJsonUtil::BuildBundleNameIndexInfo(const std::string &bundleName, int appIndex)
-{
-    std::string result = bundleName;
-    if (appIndex == BUNDLE_INDEX_DEFAULT_VAL) {
-        return result;
-    }
-    result += BUNDLE_INDEX_SPLICE;
-    result += std::to_string(appIndex);
-    return result;
-}
-
 bool BJsonUtil::BuildExtensionErrInfo(std::string &jsonStr, std::map<std::string, std::vector<int>> errFileInfo)
 {
     cJSON *errJson = cJSON_CreateObject();
@@ -353,7 +340,6 @@ bool BJsonUtil::BuildExtensionErrInfo(std::string &jsonStr, std::map<std::string
     cJSON_free(data);
     return true;
 }
-}
 
 bool OHOS::FileManagement::Backup::BJsonUtil::BuildOnProcessRetInfo(std::string &jsonStr, std::string onProcessRet)
 {
@@ -381,6 +367,17 @@ bool OHOS::FileManagement::Backup::BJsonUtil::BuildOnProcessRetInfo(std::string
     return true;
 }
 
+std::string BJsonUtil::BuildBundleNameIndexInfo(const std::string &bundleName, int appIndex)
+{
+    std::string result = bundleName;
+    if (appIndex == BUNDLE_INDEX_DEFAULT_VAL) {
+        return result;
+    }
+    result += BUNDLE_INDEX_SPLICE;
+    result += std::to_string(appIndex);
+    return result;
+}
+
 bool OHOS::FileManagement::Backup::BJsonUtil::BuildOnProcessErrInfo(std::string &reportInfo, std::string path, int err)
 {
     cJSON *info = cJSON_CreateObject();
@@ -460,4 +457,5 @@ bool OHOS::FileManagement::Backup::BJsonUtil::BuildBundleInfoJson(int32_t userId
     cJSON_Delete(root);
     free(jsonStr);
     return true;
-}
\ No newline at end of file
+}
+}