From 926f34bc36311fe037e6fea4b917661f94047984 Mon Sep 17 00:00:00 2001 From: Remi Tricot-Le Breton Date: Tue, 28 Feb 2023 17:46:18 +0100 Subject: [PATCH] MINOR: ssl: Destroy ocsp update http_client during cleanup If a deinit is started while an OCSP update is in progress we might end up with a dangling http_client instance that should be destroyed properly. --- src/ssl_ocsp.c | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/src/ssl_ocsp.c b/src/ssl_ocsp.c index 99edfc875..ef624b073 100644 --- a/src/ssl_ocsp.c +++ b/src/ssl_ocsp.c @@ -846,6 +846,11 @@ void ssl_destroy_ocsp_update_task(void) ssl_sock_free_ocsp(ssl_ocsp_task_ctx.cur_ocsp); ssl_ocsp_task_ctx.cur_ocsp = NULL; + + if (ssl_ocsp_task_ctx.hc) { + httpclient_stop_and_destroy(ssl_ocsp_task_ctx.hc); + ssl_ocsp_task_ctx.hc = NULL; + } } static inline void ssl_ocsp_set_next_update(struct certificate_ocsp *ocsp) -- 2.47.3