Permalink
Browse files

Cluster: make sortGetKeys() able to handle multiple STORE options.

It does not make sense to pass multiple store options, so, better to
handle it ;-)
  • Loading branch information...
antirez committed Mar 10, 2014
1 parent c4ef1d6 commit 5b864617bc50dbfc34dcc3393b78b3d07e0f291b
Showing with 4 additions and 1 deletion.
  1. +4 −1 src/db.c
View
@@ -1064,7 +1064,10 @@ int *sortGetKeys(struct redisCommand *cmd, robj **argv, int argc, int *numkeys)
i += skiplist[j].skip;
break;
} else if (!strcasecmp(argv[i]->ptr,"store") && i+1 < argc) {
- keys[num++] = i+1; /* <store-key> */
+ /* Note: we don't increment "num" here and continue the loop
+ * to be sure to process the *last* "STORE" option if multiple
+ * ones are provided. This is same behavior as SORT. */
+ keys[num] = i+1; /* <store-key> */
break;
}
}

0 comments on commit 5b86461

Please sign in to comment.