From 7a666ac4194997363f6098996ddda141e78e315a 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 be709322..75a4e59d 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) {