curvefs/client: memcache client set nullptr after operation fail

Signed-off-by: ilixiaocui <ilixiaocui@163.com>
opencurve · Mar 24, 2023 · c9a8f47 · c9a8f47
1 parent d42e78f
commit c9a8f47
Showing 1 changed file with 7 additions and 3 deletions.
diff --git a/curvefs/src/client/kvclient/memcache_client.h b/curvefs/src/client/kvclient/memcache_client.h
@@ -118,6 +118,8 @@ class MemCachedClient : public KVClient {
             return true;
         }
         *errorlog = ResError(res);
+        memcached_free(tcli);
+        tcli = nullptr;
         LOG(ERROR) << "Set key = " << key << " error = " << *errorlog;
         return false;
     }
@@ -144,9 +146,11 @@ class MemCachedClient : public KVClient {
 
         *errorlog = ResError(ue);
         if (ue != MEMCACHED_NOTFOUND) {
-            LOG(ERROR) << "Get key = " << key << " error = " << *errorlog
-                       << ", get_value_len = " << value_length
-                       << ", expect_value_len = " << length;
+          LOG(ERROR) << "Get key = " << key << " error = " << *errorlog
+                     << ", get_value_len = " << value_length
+                     << ", expect_value_len = " << length;
+          memcached_free(tcli);
+          tcli = nullptr;
         }
 
         return false;