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.

Only clear DNS_ZONEFLG_LOADPENDING in zone_asyncload() if zone loading is completed immediately

zone_load() is not always synchronous, it may only initiate an
asynchronous load and return DNS_R_CONTINUE, which means zone loading
has not yet been completed.  In such a case, zone_asyncload() must not
clear DNS_ZONEFLG_LOADPENDING immediately and leave that up to
zone_postload().

Lock zone before checking whether its asynchronous load is already pending

While this is not an issue in named, which only calls
dns_zone_asyncload() from task-exclusive mode, this function is exported
by libdns and thus may in theory be concurrently called for the same
zone by multiple threads.  It also does not hurt to be consistent
locking-wise with other DNS_ZONEFLG_LOADPENDING accesses.

Reduce repeated detection of mysql lib directory.
Use common part once a bit later.

Reuse new function from rt46864 for similar block elsewhere.

Merge branch 'gitlab-ci-keep-artifacts' into 'master'

Archive the full bin/tests/ directory and bin/named/named binary on build/test failure in GitLab CI

See merge request isc-projects/bind9!24

Archive the full bin/tests/ directory and bin/named/named binary on build/test failure.

Merge branch 'cherry-pick-d7975e30' into 'master'

placeholder

See merge request isc-projects/bind9!23

placeholder

(cherry picked from commit d7975e300ea1f9865bee92696aeb9ea59394152a)

Merge branch 'update-libtool-2.4.6' into 'master'

Update libtool files to 2.4.6

See merge request isc-projects/bind9!4

Merge branch 'gitlab-ci' into 'master'

Add support for running builds directly from Gitlab CI

See merge request isc-projects/bind9!20

Disable artifact uploading as it's not needed anymore with unified build and test

Run the network script in debugging mode

Check the address and routing information

Look a who am I

The interface setup needs sudo

Merge build and test steps

Use external CI Docker images

Enforce overlay2 filesystem driver for docker builds

Rename $CI_BUILD_TOKEN to $CI_JOB_TOKEN

Install libnet-dns-perl into Docker images to fix tests failures

Install procps for /bin/kill into Debian and Ubuntu docker images

Add correct dependencies for test jobs to pull artifacts from build jobs

Add net-tools for ifconfig support

Run the interface up script with 'bash' as it's not executable by itself directly

Add i386 as tested architecture

Enable private IP addresses for tests

Build docker images only in master, triggers and/or tags

Enable Gitlab-CI for the project

Merge branch 'remove-ondestroy-callbacks' into 'master'

Remove ondestroy callbacks from libisc.

See merge request isc-projects/bind9!3

Update CHANGES for isc-projects/bind9!3

Remove whole unused ondestroy callback mechanism

[master] copyrights

regen master

check warning text

(cherry picked from commit c6cd108838543042407b81fc865bbfc66dc6034b)

get the correct test file

[master] fix test error

[rt43670] more emphatic warning

4889.   [func]          Warn about the use of old root keys without the new
                        root key being present.  Warn about dlv.isc.org's
                        key being present. Warn about both managed and
                        trusted root keys being present. [RT #43670]

[master] fix typo

update copyright notice / whitespace

newcopyrights

[master] fix nsupdate test on windows

4888. [test] Initialize sockets correctly in sample-update so
that nsupdate system test will will run on Windows.
[RT #47097]

[master] fix rpzrecurse test on windows

4887. [test] Enable the rpzrecurse test to run on Windows.
[RT #47093]

Document dig -u in manpage (#47150)

regen master

update copyright notice / whitespace

newcopyrights

add note for update-policy rules changes

4885.   [security]      update-policy rules that otherwise ignore the name
                        field now require that it be set to "." to ensure
                        that any type list present is properly interpreted.
                        [RT #47126]

update copyright notice / whitespace

newcopyrights

[master] Prevent crashing due to a race during server shutdown

4884. [bug] named could crash on shutdown due to a race between
shutdown_server() and ns__client_request(). [RT #47120]

look for #! to determine if the file is PYTHON or PYTHON-BIN

regen master

[master] spelling, release note

update copyright notice / whitespace

Update license to 'MPL' [RT #47101]

newcopyrights

[master] Silence compiler warnings about comparisons between signed and unsigned integers [RT #46980]

update copyright notice / whitespace

newcopyrights

[master] improve debugging output from dnssec-cds

4883. [cleanup] Improved debugging output from dnssec-cds. [RT #47026]

Patch submitted by Tony Finch (dot@dotat.at)

update copyright notice / whitespace

newcopyrights

handle different eol coventions

[master] use 'command' instead of 'type' to find atf-run and kyua paths

[master] typo

4882.   [bug]           Address potential memory leak in
                        dns_update_signaturesinc. [RT #47084]

4881.   [bug]           Only include dst_openssl.h when OpenSSL is required.
                        [RT #47068]

4880.   [bug]           Named wasn't returning the target of a cross zone
                        CNAME between to served zones when recursion was
                        desired and available (RD=1, RA=1). Don't return
                        the CNAME target otherwise to prevent accidental
                        cache poisoning. [RT #47078]

4879.   [bug]           dns_rdata_caa:value_len is was small. [RT #47086]

update copyright notice / whitespace

newcopyrights

dns_dnssec_findmatchingkeys warning test requires crytpo to be supported

4878.   [bug]           List 'ply' as a requirement for the 'isc' python
                        package. [RT #47065]

regen master