lib-http: client: Drop the ioloop switch callback before freeing the global client...

This removes the possibility of recursion when the client context object somehow
triggers an ioloop switch (e.g., when debug logging starts blocking at that
inopportune moment).

diff --git a/src/lib-http/http-client.c b/src/lib-http/http-client.c
index 1c4f89ad4e4b2940bd43759379e49b173b535a8a..4b4e7cce04fbc6426e530831587d9ee441fc5108 100644 (file)
--- a/src/lib-http/http-client.c
+++ b/src/lib-http/http-client.c
@@ -691,11 +691,6 @@ void http_client_context_switch_ioloop(struct http_client_context *cctx)
        http_client_context_do_switch_ioloop(cctx);
 }
 
-static void http_client_global_context_free(void)
-{
-       http_client_context_unref(&http_client_global_context);
-}
-
 static void
 http_client_global_context_ioloop_switched(
        struct ioloop *prev_ioloop ATTR_UNUSED)
@@ -713,6 +708,15 @@ http_client_global_context_ioloop_switched(
        }
 }
 
+static void http_client_global_context_free(void)
+{
+       /* drop ioloop switch callback to make absolutely sure there is no
+          recursion. */
+       io_loop_remove_switch_callback(http_client_global_context_ioloop_switched);
+
+       http_client_context_unref(&http_client_global_context);
+}
+
 struct http_client_context *http_client_get_global_context(void)
 {
        if (http_client_global_context != NULL)