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.
This commit is contained in:
Olivier Houchard 2017-10-24 19:03:30 +02:00 committed by Willy Tarreau
parent 31d4dbe825
commit 390485a68d

View File

@ -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);