From e48365e2c2507e2b3eac0b5fda07c5f8c46e3ff7 Mon Sep 17 00:00:00 2001 From: antirez Date: Tue, 17 Dec 2013 14:50:24 +0100 Subject: [PATCH] Cluster: set n->slaves to NULL in clusterNodeResetSlaves(). The value was otherwise undefined, so next time the node was promoted again from slave to master, adding a slave to the list of slaves would likely crash the server or result into undefined behavior. --- src/cluster.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/cluster.c b/src/cluster.c index be709322f..75a4e59d2 100644 --- a/src/cluster.c +++ b/src/cluster.c @@ -537,6 +537,7 @@ int clusterNodeAddSlave(clusterNode *master, clusterNode *slave) { void clusterNodeResetSlaves(clusterNode *n) { zfree(n->slaves); n->numslaves = 0; + n->slaves = NULL; } void freeClusterNode(clusterNode *n) {