From bb95666bac94b6235eda431aba788644f7de7a3f Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Thu, 24 Jan 2013 00:37:39 +0100 Subject: [PATCH] BUG/MEDIUM: checks: ensure the health_status is always within bounds health_adjust() checks for incorrect bounds for the status argument. With current code, the argument is always a constant from the valid enum so there is no impact and the check is basically a NOP. However users running local patches (eg: new checks) might want to recheck their code. This fix should be backported to 1.4 which introduced the issue. Reported-by: Dinko Korunic --- src/checks.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/checks.c b/src/checks.c index 6d527a328..a933bba65 100644 --- a/src/checks.c +++ b/src/checks.c @@ -619,7 +619,7 @@ void health_adjust(struct server *s, short status) if (s->observe >= HANA_OBS_SIZE) return; - if (status >= HCHK_STATUS_SIZE || !analyze_statuses[status].desc) + if (status >= HANA_STATUS_SIZE || !analyze_statuses[status].desc) return; switch (analyze_statuses[status].lr[s->observe - 1]) {