From ec5b59c8751fe5365545459e3aed71d4bc9215a9 Mon Sep 17 00:00:00 2001
From: factosea <285808407@qq.com>
Date: Tue, 27 Feb 2024 21:11:31 +0800
Subject: [PATCH] fix: add config item

---
 source/util/src/tconfig.c | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/source/util/src/tconfig.c b/source/util/src/tconfig.c
index 0253f1a80c..e98c264df5 100644
--- a/source/util/src/tconfig.c
+++ b/source/util/src/tconfig.c
@@ -442,15 +442,6 @@ int32_t cfgCheckRangeForDynUpdate(SConfig *pCfg, const char *name, const char *p
 }
 
 static int32_t cfgAddItem(SConfig *pCfg, SConfigItem *pItem, const char *name) {
-  int size = pCfg->array->size;
-  for (int32_t i = 0; i < size; ++i) {
-    SConfigItem *existItem = taosArrayGet(pCfg->array, i);
-    char xx[45];
-    if (existItem != NULL && strcmp(existItem->name, pItem->name) == 0) {
-      return 0;
-    }
-  }
-
   pItem->stype = CFG_STYPE_DEFAULT;
   pItem->name = taosStrdup(name);
   if (pItem->name == NULL) {
@@ -458,6 +449,15 @@ static int32_t cfgAddItem(SConfig *pCfg, SConfigItem *pItem, const char *name) {
     return -1;
   }
 
+  int size = pCfg->array->size;
+  for (int32_t i = 0; i < size; ++i) {
+    SConfigItem *existItem = taosArrayGet(pCfg->array, i);
+    if (existItem != NULL && strcmp(existItem->name, pItem->name) == 0) {
+      taosMemoryFree(pItem->name);
+      return 0;
+    }
+  }
+
   int32_t len = strlen(name);
   char    lowcaseName[CFG_NAME_MAX_LEN + 1] = {0};
   strntolower(lowcaseName, name, TMIN(CFG_NAME_MAX_LEN, len));