fluencelabs/redis
1
0
Fork 0
mirror of https://github.com/fluencelabs/redis synced 2025-04-10 19:26:04 +00:00
Code Issues Projects Releases Wiki Activity

ZREMRANGEBYLEX memory leak removed calling zslFreeLexRange().

Browse Source
This commit is contained in:
antirez 2014-04-18 13:01:04 +02:00
parent 85a2f2354e
commit ab3afe2f4d
1 changed files with 5 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/t_zset.c
View File

@ -1406,7 +1406,7 @@ void zremrangeGenericCommand(redisClient *c, int rangetype) {
/* Step 2: Lookup & range sanity checks if needed. */ /* Step 2: Lookup & range sanity checks if needed. */
if ((zobj = lookupKeyWriteOrReply(c,key,shared.czero)) == NULL || if ((zobj = lookupKeyWriteOrReply(c,key,shared.czero)) == NULL ||
checkType(c,zobj,REDIS_ZSET)) return; checkType(c,zobj,REDIS_ZSET)) goto cleanup;
if (rangetype == ZRANGE_RANK) { if (rangetype == ZRANGE_RANK) {
/* Sanitize indexes. */ /* Sanitize indexes. */
@ -1419,7 +1419,7 @@ void zremrangeGenericCommand(redisClient *c, int rangetype) {
* The range is empty when start > end or start >= length. */ * The range is empty when start > end or start >= length. */
if (start > end || start >= llen) { if (start > end || start >= llen) {
addReply(c,shared.czero); addReply(c,shared.czero);
return; goto cleanup;
} }
if (end >= llen) end = llen-1; if (end >= llen) end = llen-1;
} }
@ -1473,6 +1473,9 @@ void zremrangeGenericCommand(redisClient *c, int rangetype) {
} }
server.dirty += deleted; server.dirty += deleted;
addReplyLongLong(c,deleted); addReplyLongLong(c,deleted);
cleanup:
if (rangetype == ZRANGE_LEX) zslFreeLexRange(&lexrange);
} }
void zremrangebyrankCommand(redisClient *c) { void zremrangebyrankCommand(redisClient *c) {