Wait until a cache dump completes instead of waiting for a fixed amount of time

Dumping the cache is an asynchronous operation, so sleeping for a fixed
amount of time after running "rndc dumpdb" is imperfect as dumping cache
contents may take longer than expected on slower machines.  Instead of
always sleeping for 1 second, wait until the "; Dump complete" line
appears in the dump or 10 seconds pass, whichever comes first.

Do not overwrite cache dumps

Unless configured otherwise in named.conf, "rndc dumpdb" causes a cache
dump to be written to a file called "named_dump.db" in the working
directory of the given named instance.  Repeatedly using this command
throughout different checks in the cacheclean system test causes cache
dumps for older checks to be overwritten, which hinders failure
diagnosis.  Prevent this by moving each cache dump to a check-specific
location after running "rndc dumpdb".

Furthermore, during the "check flushtree clears adb correctly" check,
dump_cache() is called twice without renaming the resulting files.
Prevent the first cache dump from being overwritten by moving it to a
different file before calling "rndc dumpdb" for the second time.

Assign an index to each check in the cacheclean system test

This will enable storing per-check forensic data, which facilitates
troubleshooting test failures.

[master] Merge branch 'master' of ssh://repo.isc.org/proj/git/prod/bind9

regen master

update copyright notice / whitespace

newcopyrights

[master] add missing release notes, update README

[master] synchronize

Merge branch 'master' into 'master'

Master

See merge request isc-projects/bind9!36

Merge branch 'gitlab-ci-limit-runners' into 'master'

Limit the combination of architectures and distributions that are normally run

See merge request isc-projects/bind9!33

Merge branch 'fix-dnstap-output-file-rolling' into 'master'

Fix dnstap output file rolling

See merge request isc-projects/bind9!21

Limit the combination of architectures and distributions that are normally run

Add CHANGES entry

4894. [bug] named could crash while rolling a dnstap output file.
[RT #46942]

Make dns_dt_send() call dns_dt_reopen() asynchronously

Instead of checking current dnstap output file size and potentially
synchronously calling dns_dt_reopen() upon every call to dns_dt_send():

  - call dns_dt_reopen() asynchronously by queuing an event to the task
    specified at dnstap environment creation time,

  - ensure no roll event is outstanding before checking dnstap output
    file size and potentially queuing another roll event.

This causes dnstap output files to exceed their configured size limits,
but prevents any two threads from performing the roll simultaneously
(which causes crashes).

Make dns_dt_reopen() request task-exclusive mode on its own

Instead of relying on the caller to set up task-exclusive mode, make
dns_dt_reopen() enforce task-exclusive mode itself, using the task
specified at dnstap environment creation time.

Add dns_dt_create2()

Implement a new variant of dns_dt_create() to enable a dnstap
environment structure to hold the task in the context of which
dns_dt_reopen() will be executed.

4893.   [bug]           Address various issues reported by cppcheck. [GL #51]

See merge request isc-projects/bind9!26

regen master

s/cppchecker/cppcheck

add GL issue number

[master] add missing release note

add CHANGES note

newcopyrights

add POST(len);

conditionally declare stacksize

add POST(len);

Clarify calculation precedence for '&' and '?'

add POST(tl)

return failure count

conditionally declare strbuf

test for == 0 rather than <= as value is unsigned

add brackets

reorder test to silence cppcheck

remove event which is only used for sizeof

remember the masters count

reorder test to silence cppcheck

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

case to unsigned; reorder expression

simplify expression

use %u and preserve unsigned property

make declaration and use of setnocookie fully conditional

cast to unsigned

preserve unsigned property

use %u instead of %d

make both arguments of & unsigned

use %u instead of %d

use %u instead of %d

use %u instead of %d

shift unsigned (~0U) rather than signed (~0) contant

use %u instead of %d

unsigned constants

unsigned constants

use %u instead of %d

use %u instead of %d

use %u instead of %d

prevent implict conversion to signed

use %u instead of %d

INSIST ipnum6 != NULL

use %u instead of %d

use %u instead of %d

test for >= 0 and use %d instead of %u

formally cast to int; use %u instead of %d

formally cast to int

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

copyrights

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

make declarations consistent

report argument error in human form

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %d

use %u instead of %i

use %u instead of %d

use %u rather than %d; pass a unsigned int pointer

clear *logp

use %u instead of %d

signed vs unsigned fixes

Merge branch 'fix-loadpending-handling' into 'master'

Fix DNS_ZONEFLG_LOADPENDING handling

See merge request isc-projects/bind9!22

[master] copyrights

Add CHANGES entry

4892. [bug] named could leak memory when "rndc reload" was invoked
before all zone loading actions triggered by a previous
"rndc reload" command were completed. [RT #47076]

Do not recheck DNS_ZONEFLG_LOADPENDING in zone_asyncload()

Remove a block of code which dates back to commit 8a2ab2b9203, when
dns_zone_asyncload() did not yet check DNS_ZONEFLG_LOADPENDING.
Currently, no race in accessing DNS_ZONEFLG_LOADPENDING is possible any
more, because:

  - dns_zone_asyncload() is still the only function which may queue
    zone_asyncload(),

  - dns_zone_asyncload() accesses DNS_ZONEFLG_LOADPENDING under a lock
    (and potentially queues an event under the same lock),

  - DNS_ZONEFLG_LOADPENDING is not cleared until the load actually
    completes.

Thus, the rechecking code can be safely removed from zone_asyncload().

Note that this also brings zone_asyncload() to a state in which the
completion callback is always invoked.  This is required to prevent
leaking memory in case something goes wrong in zone_asyncload() and a
zone table the zone belongs to is indefinitely left with a positive
reference count.

Asynchronous zone load events have no way of getting canceled

Code handling cancellation of asynchronous zone load events was likely
copied over from other functions when asynchronous zone loading was
first implemented in commit 8a2ab2b9203.  However, unlike those other
functions, asynchronous zone loading events currently have no way of
getting canceled once they get posted, which means the aforementioned
code is effectively dead.  Remove it to prevent confusion.