Commit c74e7c77 authored by antirez's avatar antirez
Browse files

ZRANGEBYSCORE memory leak fixed, ZRANGEBYSCORE initial test added

parent 1c85b79f
...@@ -4587,6 +4587,7 @@ static void zrangebyscoreCommand(redisClient *c) { ...@@ -4587,6 +4587,7 @@ static void zrangebyscoreCommand(redisClient *c) {
* it later */ * it later */
lenobj = createObject(REDIS_STRING,NULL); lenobj = createObject(REDIS_STRING,NULL);
addReply(c,lenobj); addReply(c,lenobj);
decrRefCount(lenobj);
while(ln && ln->score <= max) { while(ln && ln->score <= max) {
ele = ln->obj; ele = ln->obj;
......
...@@ -1014,6 +1014,27 @@ proc main {server port} { ...@@ -1014,6 +1014,27 @@ proc main {server port} {
list $v1 $v2 [$r zscore zset foo] [$r zscore zset bar] list $v1 $v2 [$r zscore zset foo] [$r zscore zset bar]
} {{bar foo} {foo bar} -2 6} } {{bar foo} {foo bar} -2 6}
test {ZRANGEBYSCORE basics} {
$r del zset
$r zadd zset 1 a
$r zadd zset 2 b
$r zadd zset 3 c
$r zadd zset 4 d
$r zadd zset 5 e
$r zrangebyscore zset 2 4
} {b c d}
test {Sorted sets +inf and -inf handling} {
$r del zset
$r zadd zset -100 a
$r zadd zset 200 b
$r zadd zset -300 c
$r zadd zset 1000000 d
$r zadd zset +inf max
$r zadd zset -inf min
$r zrange zset 0 -1
} {min c a b d max}
test {EXPIRE - don't set timeouts multiple times} { test {EXPIRE - don't set timeouts multiple times} {
$r set x foobar $r set x foobar
set v1 [$r expire x 5] set v1 [$r expire x 5]
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment