From 3a8c63d48d45cc18b68ab85f1237632e6f4c4f46 Mon Sep 17 00:00:00 2001 From: Tim Duesterhus Date: Sat, 22 Apr 2023 17:47:31 +0200 Subject: [PATCH] MINOR: Make `tasklet_free()` safe to be called with `NULL` Make this freeing function safe, like other freeing functions are as discussed in GitHub issue #2126. --- include/haproxy/task.h | 3 +++ 1 file changed, 3 insertions(+) diff --git a/include/haproxy/task.h b/include/haproxy/task.h index e31e9dac4..390db0f8c 100644 --- a/include/haproxy/task.h +++ b/include/haproxy/task.h @@ -621,6 +621,9 @@ static inline void task_destroy(struct task *t) /* Should only be called by the thread responsible for the tasklet */ static inline void tasklet_free(struct tasklet *tl) { + if (!tl) + return; + if (MT_LIST_DELETE(list_to_mt_list(&tl->list))) _HA_ATOMIC_DEC(&ha_thread_ctx[tl->tid >= 0 ? tl->tid : tid].rq_total);