BUG/MINOR: threads: Don't forget to init each thread toremove_lock.

Don't forget to use HA_SPIN_INIT() on each toremove_lock, or DEBUG_THREAD may
not work reliably with it.

This should be backported to 2.1 and 2.0.

(cherry picked from commit f21695bd8b59d582efc99c85f9a1afac200eda81)
[wt: the lock is called toremove_lock[i] in 2.1]
Signed-off-by: Willy Tarreau <w@1wt.eu>

diff --git a/src/cfgparse.c b/src/cfgparse.c
index 1fe0781..30df16b 100644 (file)
--- a/src/cfgparse.c
+++ b/src/cfgparse.c
@@ -3661,6 +3661,7 @@ out_uri_auth_compat:
                                                        goto err;
                                                idle_conn_cleanup[i]->process = srv_cleanup_toremove_connections;
                                                idle_conn_cleanup[i]->context = NULL;
+                                               HA_SPIN_INIT(&toremove_lock[i]);
                                                MT_LIST_INIT(&toremove_connections[i]);
                                        }
                                }