mir/bind
2
0
Fork 0
mirror of https://gitlab.isc.org/isc-projects/bind9 synced 2025-08-30 05:57:52 +00:00
Code Issues Releases Wiki Activity

Don't skip the counting if fcount_incr() is called with force==true

Browse Source 
The fcount_incr() was incorrectly skipping the accounting for the
fetches-per-zone if the force argument was set to true.  We want to skip
the accounting only when the fetches-per-zone is completely disabled,
but for individual names we need to do the accounting even if we are
forcing the result to be success.
This commit is contained in:
Ondřej Surý 2024-06-20 18:59:56 +02:00
parent b1c99c4458
commit a513d4c07f
1 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
lib/dns/resolver.c
View File

@ -1439,8 +1439,9 @@ fcount_incr(fetchctx_t *fctx, bool force) {
REQUIRE(res != NULL);
INSIST(fctx->counter == NULL);
/* Skip any counting if fetches-per-zone is disabled */
spill = atomic_load_acquire(&res->zspill);
if (force || spill == 0) {
if (spill == 0) {
return (ISC_R_SUCCESS);
}
@ -1486,7 +1487,7 @@ fcount_incr(fetchctx_t *fctx, bool force) {
INSIST(spill > 0);
LOCK(&counter->lock);
if (++counter->count > spill) {
if (!force && ++counter->count > spill) {
counter->count--;
INSIST(counter->count > 0);
counter->dropped++;