BUG/MINOR: checks: Don't forget to release the connection on error case.

When switching the check code to a non-permanent connection, the new code
forgot to free the connection if an error happened and was returned by
connect_conn_chk(), leading to the check never be ran again.

diff --git a/src/checks.c b/src/checks.c
index e6cc42a..880b47b 100644 (file)
--- a/src/checks.c
+++ b/src/checks.c
@@ -2118,6 +2118,10 @@ static struct task *process_chk_conn(struct task *t)
                }
 
                /* here, we have seen a synchronous error, no fd was allocated */
+               if (conn) {
+                       conn_free(conn);
+                       check->conn = conn = NULL;
+               }
 
                check->state &= ~CHK_ST_INPROGRESS;
                check_notify_failure(check);