From 23d86d157edd6da8d3fe40d7883a3d2beeed3189 Mon Sep 17 00:00:00 2001 From: Christopher Faulet Date: Thu, 25 Jan 2018 11:36:35 +0100 Subject: [PATCH] BUG/MEDIUM: checks: Don't try to release undefined conn_stream when a check is freed When a healt-check is released, the attached conn_stream may be undefined. For instance, this happens when 'no-check' option is used on a server line. So we must check it is defined before trying to release it. This patch must be backported in 1.8. --- src/checks.c | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/src/checks.c b/src/checks.c index 56c9d609d..0d4893e69 100644 --- a/src/checks.c +++ b/src/checks.c @@ -3098,10 +3098,12 @@ void free_check(struct check *check) check->bi = NULL; free(check->bo); check->bo = NULL; - free(check->cs->conn); - check->cs->conn = NULL; - cs_free(check->cs); - check->cs = NULL; + if (check->cs) { + free(check->cs->conn); + check->cs->conn = NULL; + cs_free(check->cs); + check->cs = NULL; + } } void email_alert_free(struct email_alert *alert)