mirrors/haproxy
1
0
Fork 0
mirror of https://git.haproxy.org/git/haproxy.git/ synced 2025-09-22 14:21:25 +02:00
Code Issues Projects Releases Wiki Activity

BUG/MEDIUM: resolvers: Use tick_first() to update the resolvers task timeout

Browse Source 
In resolv_update_resolvers_timeout(), the resolvers task timeout is updated
by checking running and waiting resolutions. However, to find the next
wakeup date, MIN() operator is used to compare ticks. Ticks must never be
compared with such operators, tick helper functions must be used, to
properly handled TICK_ETERNITY value. In this case, tick_first() must be
used instead of MIN().

It is an old bug but it is pretty visible since the commit fdecaf6ae4
("BUG/MINOR: resolvers: do not run the timeout task when there's no
resolution"). Because of this bug, the resolvers task timeout may be set to
TICK_ETERNITY, stopping periodic resolutions.

This patch should solve the issue #1962. It must be backported to all stable
versions.
This commit is contained in:
Christopher Faulet 2022-12-14 10:26:25 +01:00
parent 427293420f
commit 819d48b14e
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/resolvers.c
View File

@ -291,11 +291,11 @@ static void resolv_update_resolvers_timeout(struct resolvers *resolvers)
if (!LIST_ISEMPTY(&resolvers->resolutions.curr)) { if (!LIST_ISEMPTY(&resolvers->resolutions.curr)) {
res = LIST_NEXT(&resolvers->resolutions.curr, struct resolv_resolution *, list); res = LIST_NEXT(&resolvers->resolutions.curr, struct resolv_resolution *, list);
next = tick_add(now_ms, resolvers->timeout.resolve); next = tick_add(now_ms, resolvers->timeout.resolve);
next = MIN(next, tick_add(res->last_query, resolvers->timeout.retry)); next = tick_first(next, tick_add(res->last_query, resolvers->timeout.retry));
} }
list_for_each_entry(res, &resolvers->resolutions.wait, list) list_for_each_entry(res, &resolvers->resolutions.wait, list)
next = MIN(next, tick_add(res->last_resolution, resolv_resolution_timeout(res))); next = tick_first(next, tick_add(res->last_resolution, resolv_resolution_timeout(res)));
resolvers->t->expire = next; resolvers->t->expire = next;
task_queue(resolvers->t); task_queue(resolvers->t);