From 9ba34ae710d28648242c07937c7ed875b3928263 Mon Sep 17 00:00:00 2001 From: Amaury Denoyelle Date: Mon, 9 Aug 2021 15:09:17 +0200 Subject: [PATCH] BUG/MINOR: check: test if server is not null in purge Test if server is not null before using free_server in the check purge operation. Currently, the null server scenario should not occured as purge is used with refcounted dynamic servers. However, this might not be always the case if purge is use in the future in other cases; thus the test is useful for extensibility. No need to backport, unless dynamic server checks are backported. This has been reported through a coverity report in github issue #1343. --- src/check.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/src/check.c b/src/check.c index b49aca61a..b1ffe5eca 100644 --- a/src/check.c +++ b/src/check.c @@ -1253,7 +1253,8 @@ struct task *process_chk_conn(struct task *t, void *context, unsigned int state) task_destroy(check->task); t = NULL; - free_server(check->server); + if (check->server) + free_server(check->server); } return t; -- 2.47.3