diff --git a/examples/c/asyncdemo.c b/examples/c/asyncdemo.c
index c86cd44354d5b8a4e59e8c12fde32b4c06dc376a..91ec6f24b190fdc1c4a4c7f9eba479dc2fc1125d 100644
--- a/examples/c/asyncdemo.c
+++ b/examples/c/asyncdemo.c
@@ -92,7 +92,7 @@ int main(int argc, char *argv[])
   }
 
   // a simple way to parse input parameters
-  if (argc >= 3) strcpy(db, argv[2]);
+  if (argc >= 3) strncpy(db, argv[2], sizeof(db) - 1);
   if (argc >= 4) points = atoi(argv[3]);
   if (argc >= 5) numOfTables = atoi(argv[4]);
 
diff --git a/source/client/src/clientMsgHandler.c b/source/client/src/clientMsgHandler.c
index 2191c54315d6324e819337fa41db3875e1506594..07624efe0479224740ee728a734d6b8763122009 100644
--- a/source/client/src/clientMsgHandler.c
+++ b/source/client/src/clientMsgHandler.c
@@ -456,12 +456,13 @@ static int32_t buildShowVariablesRsp(SArray* pVars, SRetrieveTableRsp** pRsp) {
   (*pRsp)->numOfCols = htonl(SHOW_VARIABLES_RESULT_COLS);
 
   int32_t len = blockEncode(pBlock, (*pRsp)->data, SHOW_VARIABLES_RESULT_COLS);
+  blockDataDestroy(pBlock);
+
   if(len != rspSize - sizeof(SRetrieveTableRsp)){
     uError("buildShowVariablesRsp error, len:%d != rspSize - sizeof(SRetrieveTableRsp):%" PRIu64, len, (uint64_t) (rspSize - sizeof(SRetrieveTableRsp)));
     return TSDB_CODE_TSC_INVALID_INPUT;
   }
 
-  blockDataDestroy(pBlock);
   return TSDB_CODE_SUCCESS;
 }
 
diff --git a/source/libs/catalog/src/ctgCache.c b/source/libs/catalog/src/ctgCache.c
index 06db2c3268a298c71d33586eead38643cae40d58..7ff8afd6a510f16c6a47df25550c9d2dbd0e3543 100644
--- a/source/libs/catalog/src/ctgCache.c
+++ b/source/libs/catalog/src/ctgCache.c
@@ -493,11 +493,9 @@ int32_t ctgCopyTbMeta(SCatalog *pCtg, SCtgTbMetaCtx *ctx, SCtgDBCache **pDb, SCt
 
   //ctgReleaseTbMetaToCache(pCtg, dbCache, tbCache);
 
-  if (tbCache) {
-    CTG_UNLOCK(CTG_READ, &tbCache->metaLock);
-    taosHashRelease(dbCache->tbCache, tbCache);
-    *pTb = NULL;
-  }
+  CTG_UNLOCK(CTG_READ, &tbCache->metaLock);
+  taosHashRelease(dbCache->tbCache, tbCache);
+  *pTb = NULL;
   
   ctgDebug("Got ctb %s meta from cache, will continue to get its stb meta, type:%d, dbFName:%s", ctx->pName->tname,
            ctx->tbInfo.tbType, dbFName);
@@ -1554,8 +1552,8 @@ int32_t ctgWriteTbMetaToCache(SCatalog *pCtg, SCtgDBCache *dbCache, char *dbFNam
     SCtgTbCache cache = {0};
     cache.pMeta = meta;
     if (taosHashPut(dbCache->tbCache, tbName, strlen(tbName), &cache, sizeof(SCtgTbCache)) != 0) {
-      taosMemoryFree(meta);
       ctgError("taosHashPut new tbCache failed, dbFName:%s, tbName:%s, tbType:%d", dbFName, tbName, meta->tableType);
+      taosMemoryFree(meta);
       CTG_ERR_RET(TSDB_CODE_OUT_OF_MEMORY);
     }
 
diff --git a/source/libs/qworker/src/qworker.c b/source/libs/qworker/src/qworker.c
index 66acadc23bd088efd40a2120b177551de100aa31..b120fecd9d8ae04da3f2d6be323176e9c08609ee 100644
--- a/source/libs/qworker/src/qworker.c
+++ b/source/libs/qworker/src/qworker.c
@@ -847,6 +847,9 @@ _return:
       qwBuildAndSendFetchRsp(qwMsg->msgType + 1, &qwMsg->connInfo, rsp, dataLen, code);
       QW_TASK_DLOG("%s send, handle:%p, code:%x - %s, dataLen:%d", TMSG_INFO(qwMsg->msgType + 1),
                    qwMsg->connInfo.handle, code, tstrerror(code), dataLen);
+    } else {
+      qwFreeFetchRsp(rsp);
+      rsp = NULL;
     }
   }
 
@@ -1217,7 +1220,7 @@ void qWorkerStopAllTasks(void *qWorkerMgmt) {
       QW_UPDATE_RSP_CODE(ctx, TSDB_CODE_VND_STOPPED);
       QW_SET_EVENT_RECEIVED(ctx, QW_EVENT_DROP);    
     } else {
-      qwDropTask(QW_FPARAMS());
+      (void)qwDropTask(QW_FPARAMS());
     }
 
     QW_UNLOCK(QW_WRITE, &ctx->lock);