From 565363b2816e01c8f992dced6ee214f9fe20a78f Mon Sep 17 00:00:00 2001
From: liubb <1653617791@qq.com>
Date: Mon, 14 Sep 2020 17:15:14 +0800
Subject: [PATCH] =?UTF-8?q?update=20kv=5Fstore/src/kvstore=5Fcommon/kvstor?=
 =?UTF-8?q?e=5Fcommon.c.=20=E5=85=AC=E5=85=B1=E5=9F=BA=E7=A1=80=E5=BA=93kv?=
 =?UTF-8?q?store=E5=8F=82=E6=95=B0=E5=88=A4=E6=96=AD=E6=9D=A1=E4=BB=B6?=
 =?UTF-8?q?=E4=BC=98=E5=8C=96?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

---
 kv_store/src/kvstore_common/kvstore_common.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/kv_store/src/kvstore_common/kvstore_common.c b/kv_store/src/kvstore_common/kvstore_common.c
index 754e3ec..2b713df 100755
--- a/kv_store/src/kvstore_common/kvstore_common.c
+++ b/kv_store/src/kvstore_common/kvstore_common.c
@@ -105,6 +105,11 @@ void DeleteKVCache(const char* key)
 
 void AddKVCache(const char* key, const char* value, boolean isNew)
 {
+    size_t keyLen = strnlen(key, MAX_KEY_LEN);
+    size_t valueLen = strnlen(value, MAX_VALUE_LEN);
+    if ((keyLen >= MAX_KEY_LEN) || (valueLen >= MAX_VALUE_LEN)) {
+        return;
+    }
     if (isNew) {
         DeleteKVCache(key);
     }
@@ -113,12 +118,6 @@ void AddKVCache(const char* key, const char* value, boolean isNew)
         return;
     }
     (void)memset_s(item, sizeof(KvItem), 0, sizeof(KvItem));
-    size_t keyLen = strnlen(key, MAX_KEY_LEN);
-    size_t valueLen = strnlen(value, MAX_VALUE_LEN);
-    if ((keyLen >= MAX_KEY_LEN) || (valueLen >= MAX_VALUE_LEN)) {
-        FreeItem(item);
-        return;
-    }
     item->key = (char *)malloc(keyLen + 1);
     item->value = (char *)malloc(valueLen + 1);
     if ((item->key == NULL) || (item->value == NULL)) {
-- 
GitLab