]> git.ipfire.org Git - thirdparty/bind9.git/commit
rctx_resend() increment query counters
authorColin Vidal <colin@isc.org>
Tue, 7 Apr 2026 20:18:58 +0000 (22:18 +0200)
committerMichał Kępień <michal@isc.org>
Thu, 7 May 2026 11:09:18 +0000 (13:09 +0200)
commitac3ea4ecd438c99e44f7698485df9ca30322638a
treeacfa0fb7aff3c3e76ea40c0dab3d2dd774bd6c09
parentae554715ae4607d02c643ae4898ce1babe95915b
rctx_resend() increment query counters

Calls to `rctx_resend()` are done internally within the resolver, in
flow which are not supposed to happens more than once. For instance,
if some query fails, and a specific flag "F" wasn't set, then set the
flag and try again. This wouldn't occur more than once because if the
query fails the next attempt, the flag "F" would be set already, so the
resolver would move to the next server (or give up).

However, a subtle bug missing checking a flag, for instance, could lead
to an unbounded loop re-trying to query the same server. This is now
impossible as `rctx_resend()` also increment the query counters (so if
such case occurs, it would stop once the maximum limit is reached).

The dns_resstatscounter_retry are also only incremented if the
`fctx_query()` succeeds, similar to as is done in `fctx_try()`.

(cherry picked from commit f3e74304889a2e8b69c8e88fc9a383589decda32)
lib/dns/resolver.c