From c2e5be04214acea0bb93f21155333853ab73641a Mon Sep 17 00:00:00 2001 From: antirez Date: Mon, 3 Dec 2018 18:02:57 +0100 Subject: [PATCH] RESP3: fix zrangeGenericCommand() proto dependent array len. --- src/t_zset.c | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) diff --git a/src/t_zset.c b/src/t_zset.c index 2efdcd260..0427ee887 100644 --- a/src/t_zset.c +++ b/src/t_zset.c @@ -2445,9 +2445,13 @@ void zrangeGenericCommand(client *c, int reverse) { if (end >= llen) end = llen-1; rangelen = (end-start)+1; - /* Return the result in form of a multi-bulk reply */ - if (withscores && c->resp == 2) rangelen *= 2; - addReplyArrayLen(c, rangelen); + /* Return the result in form of a multi-bulk reply. RESP3 clients + * will receive sub arrays with score->element, while RESP2 returned + * a flat array. */ + if (withscores && c->resp == 2) + addReplyArrayLen(c, rangelen*2); + else + addReplyArrayLen(c, rangelen); if (zobj->encoding == OBJ_ENCODING_ZIPLIST) { unsigned char *zl = zobj->ptr;