Merge pull request #894 from ctripcorp/bugfix/memory_leak_for_zset2

repair direct memory leak
ctripcorp · Oct 11, 2024 · d0bb509 · d0bb509
2 parents af4d3ee + ed978bd
commit d0bb509
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/.../redis-core/src/main/java/com/ctrip/xpipe/redis/core/redis/rdb/parser/RdbZSet2Parser.java b/.../redis-core/src/main/java/com/ctrip/xpipe/redis/core/redis/rdb/parser/RdbZSet2Parser.java
@@ -86,6 +86,7 @@ public Integer read(ByteBuf byteBuf) {
                         propagateCmdIfNeed(member, score.readDoubleLE());
 
                         member = null;
+                        score.release();
                         score = null;
                         readCnt++;
                         if (readCnt >= len.getLenValue()) {
@@ -134,7 +135,10 @@ public void reset() {
         }
         this.state = STATE.READ_INIT;
         this.member = null;
-        this.score = null;
+        if (score != null) {
+            score.release();
+            score = null;
+        }
     }
 
     @Override