diff --git a/services/bundlemgr/include/bundle_data_mgr.h b/services/bundlemgr/include/bundle_data_mgr.h
index f473cfd14d4c086c6f3d6e62bf14e3d394faf207..5916b13a172894f90aee6dfc6df123acc97533c3 100644
--- a/services/bundlemgr/include/bundle_data_mgr.h
+++ b/services/bundlemgr/include/bundle_data_mgr.h
@@ -789,7 +789,7 @@ public:
     bool ImplicitQueryInfos(const Want &want, int32_t flags, int32_t userId, bool withDefault,
         std::vector<AbilityInfo> &abilityInfos, std::vector<ExtensionAbilityInfo> &extensionInfos,
         bool &findDefaultApp);
-    bool UpateExtResources(const std::string &bundleName,
+    bool UpdateExtResources(const std::string &bundleName,
         const std::vector<ExtendResourceInfo> &extendResourceInfos);
     bool RemoveExtResources(const std::string &bundleName,
         const std::vector<std::string> &moduleNames);
diff --git a/services/bundlemgr/include/extend_resource/extend_resource_manager_host_impl.h b/services/bundlemgr/include/extend_resource/extend_resource_manager_host_impl.h
index ca5748b6f5521bbf53866c052f77782c30c5dba8..5667329813e9b5fca5acfb7d5a2d7804bb9eb056 100644
--- a/services/bundlemgr/include/extend_resource/extend_resource_manager_host_impl.h
+++ b/services/bundlemgr/include/extend_resource/extend_resource_manager_host_impl.h
@@ -67,11 +67,11 @@ private:
         const std::string &bundleName,
         const std::vector<std::string> &filePaths,
         std::vector<ExtendResourceInfo> &extendResourceInfos);
-    bool UpateExtResourcesDb(const std::string &bundleName,
+    bool UpdateExtResourcesDb(const std::string &bundleName,
         const std::vector<ExtendResourceInfo> &extendResourceInfos);
     bool RemoveExtResourcesDb(
         const std::string &bundleName, const std::vector<std::string> &moduleNames);
-    void InnerSaveExtendResourceInfo(
+    bool InnerSaveExtendResourceInfo(
         const std::string &bundleName, const std::vector<std::string> &filePaths,
         const std::vector<ExtendResourceInfo> &extendResourceInfos);
     ErrCode CheckModuleExist(
diff --git a/services/bundlemgr/src/bundle_data_mgr.cpp b/services/bundlemgr/src/bundle_data_mgr.cpp
index ba58194c3cfc49339e023f16838bc726f002739f..63deb31ae4db6398d387b898b4b9c35031945671 100644
--- a/services/bundlemgr/src/bundle_data_mgr.cpp
+++ b/services/bundlemgr/src/bundle_data_mgr.cpp
@@ -2677,7 +2677,7 @@ bool BundleDataMgr::GetApplicationInfos(
     return !appInfos.empty();
 }
 
-bool BundleDataMgr::UpateExtResources(const std::string &bundleName,
+bool BundleDataMgr::UpdateExtResources(const std::string &bundleName,
     const std::vector<ExtendResourceInfo> &extendResourceInfos)
 {
     if (bundleName.empty()) {
diff --git a/services/bundlemgr/src/extend_resource/extend_resource_manager_host_impl.cpp b/services/bundlemgr/src/extend_resource/extend_resource_manager_host_impl.cpp
index e2afc24c5249bb8bf1fda5d2d2119a31d71dc11c..2a00a17b9c895d5b363acde179500dfa46c42016 100644
--- a/services/bundlemgr/src/extend_resource/extend_resource_manager_host_impl.cpp
+++ b/services/bundlemgr/src/extend_resource/extend_resource_manager_host_impl.cpp
@@ -181,11 +181,15 @@ ErrCode ExtendResourceManagerHostImpl::ProcessAddExtResource(
         return ERR_EXT_RESOURCE_MANAGER_PARSE_FILE_FAILED;
     }
 
-    InnerSaveExtendResourceInfo(bundleName, newFilePaths, extendResourceInfos);
+    if (!InnerSaveExtendResourceInfo(bundleName, newFilePaths, extendResourceInfos)) {
+        APP_LOGE("save %{public}s extendResource failed", bundleName.c_str());
+        RollBack(newFilePaths);
+        return ERR_EXT_RESOURCE_MANAGER_PARSE_FILE_FAILED;
+    }
     return ERR_OK;
 }
 
-void ExtendResourceManagerHostImpl::InnerSaveExtendResourceInfo(
+bool ExtendResourceManagerHostImpl::InnerSaveExtendResourceInfo(
     const std::string &bundleName,
     const std::vector<std::string> &filePaths,
     const std::vector<ExtendResourceInfo> &extendResourceInfos)
@@ -203,7 +207,7 @@ void ExtendResourceManagerHostImpl::InnerSaveExtendResourceInfo(
 
         newExtendResourceInfos.emplace_back(extendResourceInfos[i]);
     }
-    UpateExtResourcesDb(bundleName, newExtendResourceInfos);
+    return UpdateExtResourcesDb(bundleName, newExtendResourceInfos);
 }
 
 ErrCode ExtendResourceManagerHostImpl::ParseExtendResourceFile(
@@ -286,7 +290,7 @@ bool ExtendResourceManagerHostImpl::GetInnerBundleInfo(
     return dataMgr->FetchInnerBundleInfo(bundleName, info);
 }
 
-bool ExtendResourceManagerHostImpl::UpateExtResourcesDb(const std::string &bundleName,
+bool ExtendResourceManagerHostImpl::UpdateExtResourcesDb(const std::string &bundleName,
     const std::vector<ExtendResourceInfo> &extendResourceInfos)
 {
     auto dataMgr = DelayedSingleton<BundleMgrService>::GetInstance()->GetDataMgr();
@@ -294,7 +298,7 @@ bool ExtendResourceManagerHostImpl::UpateExtResourcesDb(const std::string &bundl
         APP_LOGE("Get dataMgr shared_ptr nullptr");
         return false;
     }
-    return dataMgr->UpateExtResources(bundleName, extendResourceInfos);
+    return dataMgr->UpdateExtResources(bundleName, extendResourceInfos);
 }
 
 bool ExtendResourceManagerHostImpl::RemoveExtResourcesDb(const std::string &bundleName,
diff --git a/services/bundlemgr/test/unittest/bms_data_mgr_test/bms_data_mgr_test.cpp b/services/bundlemgr/test/unittest/bms_data_mgr_test/bms_data_mgr_test.cpp
index a4f58c4796fae64e4863a80b92578937af6711bb..5e6dc307b3051deae10123e75b011ff9709ed8e8 100644
--- a/services/bundlemgr/test/unittest/bms_data_mgr_test/bms_data_mgr_test.cpp
+++ b/services/bundlemgr/test/unittest/bms_data_mgr_test/bms_data_mgr_test.cpp
@@ -2249,24 +2249,24 @@ HWTEST_F(BmsDataMgrTest, ModifyApplicationInfoByCloneInfo_0100, Function | Small
 }
 
 /**
- * @tc.number: UpateExtResources_0100
- * @tc.name: test UpateExtResources
+ * @tc.number: UpdateExtResources_0100
+ * @tc.name: test UpdateExtResources
  * @tc.desc: 1.test update external resources
  */
-HWTEST_F(BmsDataMgrTest, UpateExtResources_0100, Function | SmallTest | Level1)
+HWTEST_F(BmsDataMgrTest, UpdateExtResources_0100, Function | SmallTest | Level1)
 {
     auto dataMgr = GetDataMgr();
     ASSERT_NE(dataMgr, nullptr);
     std::string bundleName = "";
     std::vector<ExtendResourceInfo> extendResourceInfos;
-    bool ret = dataMgr->UpateExtResources(bundleName, extendResourceInfos);
+    bool ret = dataMgr->UpdateExtResources(bundleName, extendResourceInfos);
     EXPECT_EQ(ret, false);
 
     std::map<std::string, InnerBundleInfo> infos;
     InnerBundleInfo innerBundleInfo;
     infos.emplace(BUNDLE_NAME, innerBundleInfo);
     dataMgr->bundleInfos_.swap(infos);
-    ret = dataMgr->UpateExtResources(BUNDLE_NAME, extendResourceInfos);
+    ret = dataMgr->UpdateExtResources(BUNDLE_NAME, extendResourceInfos);
     EXPECT_EQ(ret, true);
 }
 
diff --git a/services/bundlemgr/test/unittest/bms_extend_resource_manager_test/bms_extend_resource_manager_test.cpp b/services/bundlemgr/test/unittest/bms_extend_resource_manager_test/bms_extend_resource_manager_test.cpp
index 7bf8faa17edf4498f278fada6b9a46a5bbf418f3..5d942472da97b239684026530fdb67543563513d 100644
--- a/services/bundlemgr/test/unittest/bms_extend_resource_manager_test/bms_extend_resource_manager_test.cpp
+++ b/services/bundlemgr/test/unittest/bms_extend_resource_manager_test/bms_extend_resource_manager_test.cpp
@@ -374,7 +374,7 @@ HWTEST_F(BmsExtendResourceManagerTest, ExtResourceTest_1000, Function | SmallTes
     EXPECT_EQ(ret, ERR_APPEXECFWK_INSTALLD_GET_PROXY_ERROR);
 
     std::vector<ExtendResourceInfo> extendResourceInfos;
-    ret = impl.UpateExtResourcesDb(BUNDLE_NAME, extendResourceInfos);
+    ret = impl.UpdateExtResourcesDb(BUNDLE_NAME, extendResourceInfos);
     EXPECT_EQ(ret, false);
 }
 
@@ -1273,7 +1273,7 @@ HWTEST_F(BmsExtendResourceManagerTest, ExtResourceTest_1001, Function | SmallTes
     };
     auto ret = impl.CopyToTempDir(BUNDLE_NAME, oldFilePaths, newFilePaths);
     std::vector<ExtendResourceInfo> extendResourceInfos;
-    ret = impl.UpateExtResourcesDb(BUNDLE_NAME, extendResourceInfos);
+    ret = impl.UpdateExtResourcesDb(BUNDLE_NAME, extendResourceInfos);
     EXPECT_TRUE(ret);
 }
 
diff --git a/test/fuzztest/fuzztest_others/processaddextresource_fuzzer/processaddextresource_fuzzer.cpp b/test/fuzztest/fuzztest_others/processaddextresource_fuzzer/processaddextresource_fuzzer.cpp
index aad3a39ce6e937f7a7bb698a94e5d9c1ed692eab..8f03f73751c2bc1175276ca925751bb98bdfb876 100644
--- a/test/fuzztest/fuzztest_others/processaddextresource_fuzzer/processaddextresource_fuzzer.cpp
+++ b/test/fuzztest/fuzztest_others/processaddextresource_fuzzer/processaddextresource_fuzzer.cpp
@@ -29,7 +29,7 @@ namespace {
         auto ret = impl.ProcessAddExtResource(TEST_BUNDLE, filePaths);
         std::vector<ExtendResourceInfo> extendResourceInfos;
         impl.InnerSaveExtendResourceInfo(TEST_BUNDLE, filePaths, extendResourceInfos);
-        impl.UpateExtResourcesDb(TEST_BUNDLE, extendResourceInfos);
+        impl.UpdateExtResourcesDb(TEST_BUNDLE, extendResourceInfos);
         std::vector<std::string> moduleNames;
         impl.RemoveExtResourcesDb(TEST_BUNDLE, moduleNames);
         impl.InnerRemoveExtendResources(TEST_BUNDLE, moduleNames, extendResourceInfos);
diff --git a/test/fuzztest/fuzztest_others/updateextresourcesdb_fuzzer/updateextresourcesdb_fuzzer.cpp b/test/fuzztest/fuzztest_others/updateextresourcesdb_fuzzer/updateextresourcesdb_fuzzer.cpp
index 872ee732816076845e83be32de2c9f1be9e795b9..853a6e13d6c8d058852e2c628e17796aa5357442 100644
--- a/test/fuzztest/fuzztest_others/updateextresourcesdb_fuzzer/updateextresourcesdb_fuzzer.cpp
+++ b/test/fuzztest/fuzztest_others/updateextresourcesdb_fuzzer/updateextresourcesdb_fuzzer.cpp
@@ -36,7 +36,7 @@ namespace {
             return false;
         }
         std::vector<ExtendResourceInfo> extendResourceInfos;
-        ret = impl.UpateExtResourcesDb(BUNDLE_NAME, extendResourceInfos);
+        ret = impl.UpdateExtResourcesDb(BUNDLE_NAME, extendResourceInfos);
         return ret;
     }
 }