From d6f0a9ac72034d95e6005662eb992eefe161bab2 Mon Sep 17 00:00:00 2001 From: artix Date: Fri, 2 Nov 2018 17:08:25 +0100 Subject: [PATCH] Cluster Manager: fixed string parsing issue in clusterManagerGetConfigSignature --- src/redis-cli.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/redis-cli.c b/src/redis-cli.c index e6e20b6f..3db212f4 100644 --- a/src/redis-cli.c +++ b/src/redis-cli.c @@ -3307,8 +3307,8 @@ static sds clusterManagerGetConfigSignature(clusterManagerNode *node) { nodename = token; tot_size = (p - token); name_len = tot_size++; // Make room for ':' in tot_size - } else if (i == 8) break; - i++; + } + if (++i == 8) break; } if (i != 8) continue; if (nodename == NULL) continue; @@ -3347,7 +3347,7 @@ static sds clusterManagerGetConfigSignature(clusterManagerNode *node) { char *sp = cfg + name_len; *(sp++) = ':'; for (i = 0; i < c; i++) { - if (i > 0) *(sp++) = '|'; + if (i > 0) *(sp++) = ','; int slen = strlen(slots[i]); memcpy(sp, slots[i], slen); sp += slen;