gamzinav/haproxy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

BUG/MEDIUM: bwlim: Be sure to never set the analyze expiration date in past

Browse Source 
Every time a bandwidth limitation is evaluated on a channel, the analyze
expiration date is renewed, mainly based on the internal bandwidth
limitation filter expiration date. However, when the filter is called while
there is no data to filter, we skip all limitation computations to jump at
the end of the function. At this stage, the analyze expiration date is
renewed before exiting. But here the internal expiration date may be expired
and not reset.

To sum up, it is possible to set the analyze expiration date of a channel in
the past. It is unexpected and this could lead to a loop in process_stream.

To fix the issue, we just now take care to reset the internal expiration
date, if needed, before exiting.

This patch should fix the issue #2634. It must be backported as far as 2.8.

(cherry picked from commit 2cb5b7dca688e59146256833153ad700302004ba)
Signed-off-by: Willy Tarreau <w@1wt.eu>
This commit is contained in:
Christopher Faulet 2024-07-09 18:51:43 +02:00 committed by Willy Tarreau
parent 49b64e6539
commit 9613cd06a5
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
src/flt_bwlim.c
View File

@ -85,10 +85,13 @@ static int bwlim_apply_limit(struct filter *filter, struct channel *chn, unsigne
/* Don't forward anything if there is nothing to forward or the waiting
* time is not expired
*/
if (!len || (tick_isset(st->exp) && !tick_is_expired(st->exp, now_ms)))
if (tick_isset(st->exp) && !tick_is_expired(st->exp, now_ms))
goto end;
st->exp = TICK_ETERNITY;
if (!len)
goto end;
ret = len;
if (conf->flags & BWLIM_FL_SHARED) {
void *ptr;
@ -177,6 +180,7 @@ static int bwlim_apply_limit(struct filter *filter, struct channel *chn, unsigne
end:
chn->analyse_exp = tick_first((tick_is_expired(chn->analyse_exp, now_ms) ? TICK_ETERNITY : chn->analyse_exp),
st->exp);
BUG_ON(tick_is_expired(chn->analyse_exp, now_ms));
return ret;
}