ClientList.cxx: copy client list before iterating

It seems that it is not safe to remove the item from std::list while it's being iterated somewhere else. This is a very simple quick fix to make things work until some better solution is implemented.

ClientList.cxx: copy client list before iterating
f53dadcc · Denis Krjuchkov · 90ab65f8 · f53dadcc
Commit f53dadcc authored Jan 13, 2013 by Denis Krjuchkov
Hide whitespace changes
Inline Side-by-side

Showing with 2 additions and 1 deletion

ClientList.cxx src/ClientList.cxx +2 -1

No files found.
--- a/src/ClientList.cxx
+++ b/src/ClientList.cxx
@@ -59,7 +59,8 @@ client_list_add(Client *client)
 void
 client_list_foreach(void (*callback)(Client *client, void *ctx), void *ctx)
 {
-	for (Client *client : clients)
+	auto clients_local = std::list<Client *>(clients);
+	for (Client *client : clients_local)
 		callback(client, ctx);
 }