Fix to optional parameter (EX/NX) ordering for set command

madsovesen · michael-grunder · commit 3b021ea70814 · 2016-06-02T08:04:20.000-07:00
diff --git a/redis_commands.c b/redis_commands.c
@@ -1273,9 +1273,9 @@ int redis_set_cmd(INTERNAL_FUNCTION_PARAMETERS, RedisSock *redis_sock,
     /* Now let's construct the command we want */
     if(exp_type && set_type) {
         /* SET <key> <value> NX|XX PX|EX <timeout> */
-        *cmd_len = redis_cmd_format_static(cmd, "SET", "ssssl", key, key_len,
-                                           val, val_len, set_type, 2, exp_type,
-                                           2, expire);
+        *cmd_len = redis_cmd_format_static(cmd, "SET", "sssls", key, key_len,
+                                           val, val_len, exp_type, 2, expire,
+                                           set_type, 2);
     } else if(exp_type) {
         /* SET <key> <value> PX|EX <timeout> */
         *cmd_len = redis_cmd_format_static(cmd, "SET", "sssl", key, key_len,
