Commit 12504105 authored by Oran Agra's avatar Oran Agra
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 44b76a75
...@@ -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