diff --git a/src/client/src/tscSystem.c b/src/client/src/tscSystem.c
index 2765907ab0479188bda2fbfe736e854e3be0a9a3..633d590e3d697ae2f1cc01967320132c6e3f0078 100644
--- a/src/client/src/tscSystem.c
+++ b/src/client/src/tscSystem.c
@@ -478,11 +478,11 @@ static setConfRet taos_set_config_imp(const char *config){
     if(!taosReadConfigOption(item->string, item->valuestring, NULL, NULL, TAOS_CFG_CSTATUS_OPTION, TSDB_CFG_CTYPE_B_CLIENT)){
       ret.retCode = -1;
       if (strlen(ret.retMsg) == 0){
-        snprintf(ret.retMsg, 1000, "part error|%s", item->string);
+        snprintf(ret.retMsg, 1024, "part error|%s", item->string);
       }else{
-        char tmp[1024] = {0};
-        strcpy(tmp, ret.retMsg);
-        snprintf(ret.retMsg, 1000, "%s|%s", tmp, item->string);
+        char tmp[1000] = {0};
+        strncpy(tmp, ret.retMsg, 1000);
+        snprintf(ret.retMsg, 1024, "%s|%s", tmp, item->string);
       }
     }
   }
diff --git a/src/client/tests/setConfigTest.cpp b/src/client/tests/setConfigTest.cpp
index 9b982741b78e05a01c7f634c80f2001ccdef0418..1b3f74c1898d50b1912b05d35a94283ffdb1bafa 100644
--- a/src/client/tests/setConfigTest.cpp
+++ b/src/client/tests/setConfigTest.cpp
@@ -14,7 +14,7 @@ TEST(testCase, set_config_test1) {
 
   const char *config2 = "{\"debugFlag\":\"199\"}";
   ret = taos_set_config(config2);  // not take effect
-  ASSERT_EQ(ret.retCode, -5)
+  ASSERT_EQ(ret.retCode, -5);
   printf("msg:%d->%s", ret.retCode, ret.retMsg);
 
   bool readResult = taosReadGlobalCfg();  // load file config, debugFlag not take effect
@@ -57,7 +57,7 @@ TEST(testCase, set_config_test4) {
 }
 
 TEST(testCase, set_config_test5) {
-  const char *config = "ddd";
+  const char *config = "\"ddd\"";
   setConfRet ret = taos_set_config(config);
   ASSERT_EQ(ret.retCode, -3);
   printf("msg:%d->%s", ret.retCode, ret.retMsg);