Commit 676445ad authored by Oran Agra's avatar Oran Agra Committed by antirez
Browse files

fix disconnectSlaves, to try to free each slave.

the recent change in that loop (iteration rather than waiting for it to
be empty) was intended to avoid an endless loop in case some slave would
refuse to be freed.

but the lookup of the first client remained, which would have caused it
to try the first one again and again instead of moving on.
parent 4846c0c8
...@@ -1019,7 +1019,6 @@ void disconnectSlaves(void) { ...@@ -1019,7 +1019,6 @@ void disconnectSlaves(void) {
listNode *ln; listNode *ln;
listRewind(server.slaves,&li); listRewind(server.slaves,&li);
while((ln = listNext(&li))) { while((ln = listNext(&li))) {
listNode *ln = listFirst(server.slaves);
freeClient((client*)ln->value); freeClient((client*)ln->value);
} }
} }
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment