From 4ecb5e0f7fd29d2e9e70efe6b623bc1ecf75de40 Mon Sep 17 00:00:00 2001
From: Jeff Tao <jianhui.tao@qq.com>
Date: Fri, 10 Apr 2020 11:51:00 +0800
Subject: [PATCH] refCount may be negative is vnodeRelease is called
 appropriately add assert there in case it happens

---
 src/vnode/main/src/vnodeMain.c | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/src/vnode/main/src/vnodeMain.c b/src/vnode/main/src/vnodeMain.c
index 182b4b6257..8d36f59da0 100644
--- a/src/vnode/main/src/vnodeMain.c
+++ b/src/vnode/main/src/vnodeMain.c
@@ -202,11 +202,13 @@ int32_t vnodeClose(int32_t vgId) {
 
 void vnodeRelease(void *pVnodeRaw) {
   SVnodeObj *pVnode = pVnodeRaw;
+  int32_t    vgId = pVnode->vgId;
 
   int32_t refCount = atomic_sub_fetch_32(&pVnode->refCount, 1);
+  assert(refCount >= 0);
 
   if (refCount > 0) {
-    dTrace("pVnode:%p vgId:%d, release vnode, refCount:%d", pVnode, pVnode->vgId, refCount);
+    dTrace("pVnode:%p vgId:%d, release vnode, refCount:%d", pVnode, vgId, refCount);
     return;
   }
 
-- 
GitLab