Add new default siphash24 cookie algorithm, but keep AES as legacy

This commit changes the BIND cookie algorithms to match
draft-sury-toorop-dnsop-server-cookies-00.  Namely, it changes the Client Cookie
algorithm to use SipHash 2-4, adds the new Server Cookie algorithm using SipHash
2-4, and changes the default for the Server Cookie algorithm to be siphash24.

Add siphash24 cookie algorithm, and make it keep legacy aes as

Remove all cookie algorithms but AES, which was used as a default, for legacy purposes.

Merge branch 'ondrej/fix-leaked-memory-in-geoip_test.c' into 'master'

Fix leaked memory in geoip_test.c

See merge request isc-projects/bind9!2154

Fix the memory leaks in GeoIP unit test

Each individual test opened GeoIP databased but the database handles were never
closed.  This commit moves the open/close from the individual unit tests into
the _setup and _teardown methods where they really belong.

Merge branch 'ondrej/remove-mutexatomics-tests' into 'master'

Disable the mutexatomics tests from the regular CI (we could enable them on...

See merge request isc-projects/bind9!2150

Disable the mutexatomics tests from the regular CI (we could enable them on trigger or schedule in the future)

Merge branch 'ondrej/add-memcpy.spatch-and-fix-check-cocci-script' into 'master'

Add memcpy.spatch and fixup check-cocci script

See merge request isc-projects/bind9!2149

Don't cover the .spatch files with copyright

Make util/check-cocci modify the files in place and be more verbose

Remove #include <config.h> usage from siphash.c and siphash_test.c

Replace the only memcpy usage with memmove

Add spatch to replace memcpy usage with memmove

Fix the check-cocci script to scan bin lib fuzz dirs individually, as it was using only the last one

Merge branch 'prep-release'

prep 9.15.2

Merge branch '1139-1952-breaks-windows-build' into 'master'

Resolve "!1952 Breaks Windows Build"

Closes #1139

See merge request isc-projects/bind9!2145

Fix Windows compilation issues with !1952, contributed by Thomas Jach

Merge branch 'wpk-fix-compilation-error-statschannel' into 'master'

statschannel.c: declare dnssecsignstat_dump only if it's used (LIBXML2 or LIBJSON is available)

See merge request isc-projects/bind9!2143

statschannel.c: declare dnssecsignstat_dump only if it's used (LIBXML2 or LIBJSON is available)

Merge branch '1038-proper-atomics-use' into 'master'

Resolve "Make proper use of the atomics"

Closes #1038

See merge request isc-projects/bind9!1952

CHANGES

Add --enable-mutex-atomics build to CI

Fix a few broken atomics initializations

lib/ns/stats.c: use isc_refcount_t

lib/isc/win32/socket.c: use isc_refcount_t

lib/isc/task.c: use isc_refcount_t

lib/isc/stats.c: Use ->references instead of ->refs

lib/isc/quota.c: use proper acquire/release/relaxed memory order semantics

lib/isc/counter.c: use isc_refcount_t

lib/dns/zt.c: use isc_refcount_t and atomics

lib/dns/tsig.c: use isc_refcount_t

lib/dns/ssu.c: use isc_refcount_t

lib/dns/sdlz.c:
- use isc_refcount_t
- properly detach from nodes so that we don't destroy objects
  that have non-zero references count

lib/dns/sdb.c: use isc_refcount_t

lib/dns/resolver.c: use isc_refcount_t and atomics

lib/dns/masterdump.c: use isc_refcount_t and atomics

lib/dns/master.c: use isc_refcount_t and atomics

lib/dns/lib.c: use isc_refcount_t

lib/dns/nta.c: use isc_refcount_t

lib/dns/ecdb.c: use isc_refcount_t

lib/dns/dbtable.c: use isc_refcount_t

lib/dns/cache.c: use isc_refcount_t

dns/stats.c: use isc_refcount_t

lib/ns/client: use refcount_t for reference counting

lib/ns/lib.c: use isc_refcount_t for reference counting

Special, for-tests-only, mode with atomics emulated by a mutex-locked variable - useful for finding atomics congestions

Properly initialize all atomic variables

isc/stats: use isc_refcount_t

isc/timer: use isc_refcount_t

lib/isc/task: use isc_refcount_t

dns/zt.c: use isc_refcount_t for reference counting

interfacemgr: use isc_refcount_t for reference counting

Make portentry->refs an isc_refcount_t

Merge branch 'ondrej/missing-ax_restore_flags-geoip2' into 'master'

Add missing AX_RESTORE_FLAGS([geoip2]) to configure.ac

See merge request isc-projects/bind9!2140

Add missing AX_RESTORE_FLAGS([geoip2]) to configure.ac

Merge branch 'ondrej-disable-freebsd-again' into 'master'

Disable FreeBSD Runner in the CI

See merge request isc-projects/bind9!2138

Disable FreeBSD Runner in the CI; it's broken again.

Merge branch '1112-remove-legacy-geoip' into 'master'

Resolve "Remove legacy GeoIP support in 9.15 after GeoIP2 is merged"

Closes #1112

See merge request isc-projects/bind9!2102

Get rid of DNS_GEOIP_DATABASE_INIT explicit initializer for geoip dbs

Instead of the explicit struct initializer with all member, rely on the fact
that static variables are explicitly initialized to 0 if not explicitly
initialized.

remove --with-geoip and change --without-geoip2 to --disable-geoip in CI

convert geoip to geoip2 in win32utils/Configure

use configure --enable-geoip to turn on GeoIP2

- instead of --with-geoip2, use --enable-geoip to turn on the GeoIP
  feature
- --with-maxminddb can be used to set the prefix for maxminddb

add CHANGES note; remove references to legacy GeoIP from documentation

remove all support for legacy GeoIP

Merge branch '7-matthijs-deprecate-dnssec-lookaside' into 'master'

Deprecate dnssec-lookaside option

Closes #7

See merge request isc-projects/bind9!2104

remove spurious commas from docbook

Deprecate dnssec-lookaside option

Add the CFG_CLAUSEFLAG_DEPRECATED flag to the option so that people are
discouraged from using DLV.

Merge branch 'ondrej/kyua.result-location' into 'master'

Use $KYUA_RESULT in kyua report-html invocation for unified file location

See merge request isc-projects/bind9!2127

Set number of parallel jobs in .gitlab-ci.yml

Use $KYUA_RESULT in kyua report-html invocation for unified file location

Merge branch '1114-windows-build-broken' into 'master'

fix broken windows build

Closes #1114

See merge request isc-projects/bind9!2110

fix broken windows build

The MSVS C compiler requires every struct to have at least one member.
The dns_geoip_databases_t structure had one set of members for
HAVE_GEOIP and a different set for HAVE_GEOIP2, and none when neither
API is in use.

This commit silences the compiler error by moving the declaration of
dns_geoip_databases_t to types.h as an opaque reference, and commenting
out the contents of geoip.h when neither version of GeoIP is enabled.

Merge branch 'michal/define-ATOMIC_VAR_INIT-on-windows' into 'master'

Define ATOMIC_VAR_INIT() on Windows

See merge request isc-projects/bind9!2125

Define ATOMIC_VAR_INIT() on Windows

Commit b104a9bc503033b4c3174f088e2c5bdbc5d8bb51 introduced unconditional
use of the ATOMIC_VAR_INIT() macro in bin/dnssec/dnssec-signzone.c even
though that macro is only defined on Unix platforms.  Define it on
Windows systems as well in order to prevent build failures.

Merge branch '1026-make-dnssec-signzone-thread-sanitizer-clean' into 'master'

Resolve "Make dnssec-signzone ThreadSanitizer clean"

Closes #1026

See merge request isc-projects/bind9!1938

Protect global init_count variable in rbtdb.c by making it atomic

bin/dnssec/dnssec-signzone.c: Protect global variables by making them atomic

Both global shuttingdown and finished bool variables were prone to data race
(as reported by ThreadSanitizer).  The commit makes them both atomic.

Make isc_rwlock.c thread-safe

The ThreadSanitizer found several possible data races in our rwlock
implementation.  This commit changes all the unprotected variables to atomic and
also changes the explicit memory ordering (atomic_<foo>_explicit(..., <order>)
functions to use our convenience macros (atomic_<foo>_<order>).

Define ATOMIC_VAR_INIT in isc/stdatomic.h shim

Add atomic_compare_exchange_strong_relaxed convenience macro

Merge branch 'ondrej/with-or-without-geoip' into 'master'

Enable GeoIP2 for all builds and enable old GeoIP only for Debian amd64 stretch build

See merge request isc-projects/bind9!2120

Enable GeoIP2 for all builds and enable old GeoIP only for Debian jessie and Ubuntu xenial builds

Merge branch 'ondrej/missing-dot-in-job' into 'master'

Add missing dot in .autoconf job template that caused duplicate job

See merge request isc-projects/bind9!2121

Add missing dot in .autoconf job template that caused duplicate job

Merge branch 'ondrej/restore-freebsd-runner' into 'master'

Restore the FreeBSD Runner

See merge request isc-projects/bind9!2117

Use $(pwd) instead of $CI_PROJECT_DIR to always get absolute path

Use sudo to setup the interfaces when CI job is not running as root

Add FreeBSD 11 GitLab CI Runner

Merge branch 'ondrej/fix-freebsd-make' into 'master'

Add rules to make sure subdirs are always built before testdirs

See merge request isc-projects/bind9!2116

Add rules to make sure subdirs are always built before testdirs

Merge branch '1120-remove-so_bsdcompat-usage' into 'master'

Resolve "Remove SO_BSDCOMPAT usage"

Closes #1120

See merge request isc-projects/bind9!2113

Add CHANGES note:

5261.  [cleanup]       Remove SO_BSDCOMPAT socket option usage.

Remove SO_BSDCOMPAT usage, it's ignore since Linux 2.4

The SO_BSDCOMPAT socket option is no-op since Linux 2.4, see the manpage:

SO_BSDCOMPAT
Enable BSD bug-to-bug compatibility.  This is used by the UDP protocol
module in Linux 2.0 and 2.2.  If enabled, ICMP errors received for a UDP
socket will not be passed to the user program.  In later kernel
versions, support for this option has been phased out: Linux 2.4
silently ignores it, and Linux 2.6 generates a kernel warning (printk())
if a program uses this option.  Linux 2.0 also enabled BSD bug-to-bug
compatibility options (random header changing, skipping of the broadcast
flag) for raw sockets with this option, but that was removed in Linux
2.2.

Merge branch 'managed-keys-is-deprecated' into 'master'

Mark some managed-keys instances deprecated

See merge request isc-projects/bind9!2105

Mark some managed-keys instances deprecated

The 'managed-keys' (and 'trusted-keys') options have been deprecated
by 'dnssec-keys'.  Some documentation references to 'managed-keys'
had not yet been marked or noted as such.

Merge branch 'michal/add-and-use-keyfile_to_key_id-helper-function' into 'master'

Add and use keyfile_to_key_id() helper function

See merge request isc-projects/bind9!2087

Add and use keyfile_to_key_id() helper function

When trying to extract the key ID from a key file name, some test code
incorrectly attempts to strip all leading zeros.  This breaks tests when
keys with ID 0 are generated.  Add a new helper shell function,
keyfile_to_key_id(), which properly handles keys with ID 0 and use it in
test code whenever a key ID needs to be extracted from a key file name.

Merge branch '1093-dnstap-read-clear-buffer-before-expanding-it' into 'master'

dnstap-read: clear buffer before expanding it

Closes #1093

See merge request isc-projects/bind9!2049

Add CHANGES entry

5260. [bug] dnstap-read was producing malformed output for large
packets. [GL #1093]

dnstap-read: clear buffer before expanding it

When printing a packet, dnstap-read checks whether its text form takes
up more than the 2048 bytes allocated for the output buffer by default.
If that is the case, the output buffer is automatically expanded, but
the truncated output is left in the buffer, resulting in malformed data
being printed.  Clear the output buffer before expanding it to prevent
this issue from occurring.