diff --git a/services/distributeddataservice/service/data_share/gaussdb_rd/src/interface/src/document_store.cpp b/services/distributeddataservice/service/data_share/gaussdb_rd/src/interface/src/document_store.cpp
index 5d9f8a14b53af7255993e836d44449f04be463ca..2842cc5dcae35cd7d276a25ea5510f3596b08e36 100644
--- a/services/distributeddataservice/service/data_share/gaussdb_rd/src/interface/src/document_store.cpp
+++ b/services/distributeddataservice/service/data_share/gaussdb_rd/src/interface/src/document_store.cpp
@@ -404,7 +404,7 @@ int DocumentStore::UpsertDataIntoDB(std::shared_ptr<QueryContext> &context, Json
     }
     errCode = InsertIdToDocument(resultSet, filterObj, documentObj, docId);
     if (errCode != E_OK) {
-        return errCode;
+        goto END;
     }
     errCode = CheckUpsertConflict(resultSet, filterObj, docId, coll, isDataExist);
     // There are only three return values, the two other situation can continue to move forward.
diff --git a/services/distributeddataservice/service/data_share/gaussdb_rd/test/unittest/api/documentdb_data_test.cpp b/services/distributeddataservice/service/data_share/gaussdb_rd/test/unittest/api/documentdb_data_test.cpp
index 1ea370b55ac52d7a3030dae6a46ad713cfc50cca..565dd8e291d81be47670e840ea9a38253e6c4738 100644
--- a/services/distributeddataservice/service/data_share/gaussdb_rd/test/unittest/api/documentdb_data_test.cpp
+++ b/services/distributeddataservice/service/data_share/gaussdb_rd/test/unittest/api/documentdb_data_test.cpp
@@ -134,7 +134,7 @@ HWTEST_F(DocumentDBDataTest, UpsertDataTest006, TestSize.Level0)
     std::string filter = R""({"_id":"1234"})"";
     std::string document = R""({"name":"Tmono","age":18,"addr":{"city":"shanghai","postal":200001}})"";
 
-    for (auto flags : std::vector<unsigned int> { 2, 4, 8, 64, 1024, UINT32_MAX }) {
+    for (auto flags : std::vector<unsigned int>{ 2, 4, 8, 64, 1024, UINT32_MAX }) {
         EXPECT_EQ(GRD_UpsertDoc(g_db, g_coll, filter.c_str(), document.c_str(), flags), GRD_INVALID_ARGS);
     }
 }
@@ -191,6 +191,32 @@ HWTEST_F(DocumentDBDataTest, UpsertDataTest011, TestSize.Level0)
     ASSERT_EQ(result, GRD_INVALID_FORMAT);
 }
 
+/**
+ * @tc.name: UpdateDataTest012
+ * @tc.desc: Input parameter collectionName is null, invoke the GRD_UpsertDoc interface to update data.
+ * @tc.type: FUNC
+ * @tc.require:
+ * @tc.author: mazhao
+ */
+HWTEST_F(DocumentDBDataTest, UpsertDataTest012, TestSize.Level0)
+{
+    /**
+     * @tc.steps: step1. Insert a document.
+     * @tc.expected: step1. return GRD_OK.
+     */
+    int result = GRD_InsertDoc(g_db, g_coll, "{}", 0);
+    ASSERT_EQ(result, GRD_OK);
+    /**
+     * @tc.steps: step2. Parameter collectionName is Invalid format
+     * @tc.expected: step2. return Update faild.
+     */
+    result = GRD_UpsertDoc(g_db, "null", "{}", "{}", 1);
+    ASSERT_EQ(result, GRD_INVALID_ARGS);
+
+    result = GRD_UpsertDoc(g_db, "!！ &%$^%$&*%^。m中文、、请问E：112423123", "{}", "{}", 1);
+    ASSERT_EQ(result, GRD_INVALID_ARGS);
+}
+
 /**
  * @tc.name: UpdateDataTest001
  * @tc.desc: