Add semantic patch to remove checking for isc_mem_get() return value

The isc_mem_get() cannot fail gracefully now, it either gets memory of
assert()s.  The added semantic patch cleans all the blocks checking whether
the return value of isc_mem_get() was NULL.

Revert to patch generating check-cocci script

The coccinelle and util/update_copyright script have different
idea about how the whitespace should look like.  Revert the script
to the previous version, so it doesn't mangle the files in place,
and deal with just whitespace changes.

Merge branch 'ondrej/disable-LD_WRAP-without-libtool' into 'master'

Disable LD_WRAP tests when libtool is disabled

See merge request isc-projects/bind9!2192

Disable LD_WRAP tests when libtool is disabled

Merge branch 'ondrej/fix-LD_WRAP-Makefile' into 'master'

Fix the lib/dns/tests/Makefile.in to work without LD_WRAP

See merge request isc-projects/bind9!2188

Fix the lib/dns/tests/Makefile.in to work without LD_WRAP

Merge branch '1149-restore-locking-in-resume_dslookup' into 'master'

Restore locking in resume_dslookup()

Closes #1149

See merge request isc-projects/bind9!2184

Restore locking in resume_dslookup()

Commit 9da902a201b6d0e1bdbac0af067a59bb0a489c9c removed locking around
the fctx_decreference() call inside resume_dslookup().  This allows
fctx_unlink() to be called without the bucket lock being held, which
must never happen.  Ensure the bucket lock is held by resume_dslookup()
before it calls fctx_decreference().

Merge branch 'u/fanf2/arm-rrset-order-random' into 'master'

doc/arm: correct default for rrset-order

See merge request isc-projects/bind9!2146

doc/arm: correct default for rrset-order

Merge branch '195-add-dnstap-builds-to-ci' into 'master'

Add dnstap builds to CI

Closes #195

See merge request isc-projects/bind9!2160

Add dnstap builds to CI

Ensure BIND with dnstap support enabled is being continuously tested by
adding --enable-dnstap to the ./configure invocation used for CentOS 7
and Debian sid builds in GitLab CI.

Merge branch 'each-fix-ld-wrap-test' into 'master'

Fix LD_WRAP test in configure

See merge request isc-projects/bind9!1806

Make the symbol wrapping work with dynamic linking

When the unit test is linked with dynamic libraries, the wrapping
doesn't occur, probably because it's different translation unit.

To workaround the issue, we provide thin wrappers with *real* symbol
names that just call the mocked functions.

Fix the configure.ac and Makefile.in to correctly test for --wrap

Merge branch '1136-named-checkconf-should-report-missing-dnstap-output-option-when-dnstap-option-is-set' into 'master'

Resolve "named-checkconf should report missing dnstap-output option when dnstap option is set"

Closes #1136

See merge request isc-projects/bind9!2147

named-checkconf failed to report dnstap-output missing

from named.conf when dnstap was specified

Merge branch '1106-interaction-between-dns64-and-rpz-can-cause-unexpected-results' into 'master'

Resolve "Interaction between dns64 and RPZ can cause unexpected results"

Closes #1106

See merge request isc-projects/bind9!2111

add CHANGES

Do not attempt to perform a DNS64 rewrite if RPZ returns NODATA.

Check that RPZ 'CNAME *.' (NODATA) works with DNS64.

Merge branch 'michal/add-debian-buster-to-ci' into 'master'

Add Debian buster to CI

See merge request isc-projects/bind9!2162

Add Debian buster to CI

Ensure BIND is continuously tested on Debian 10 (buster) as it is the
current stable Debian release.

Merge branch '1149-fix-crashes-during-resolver-shutdown' into 'master'

Restore locking in dns_resolver_shutdown and dns_resolver_attach

Closes #1149

See merge request isc-projects/bind9!2164

Restore more locking in the lib/dns/resolver.c code

1. Restore locking in the fctx_decreference() code, because the insides of the
   function needs to be protected when fctx->references drops to 0.

2. Restore locking in the dns_resolver_attach() code, because two variables are
   accessed at the same time and there's slight chance of data race.

Restore locking in dns_resolver_shutdown and dns_resolver_attach

Although the struct dns_resolver.exiting member is protected by stdatomics, we
actually need to wait for whole dns_resolver_shutdown() to finish before
destroying the resolver object.  Otherwise, there would be a data race and some
fctx objects might not be destroyed yet at the time we tear down the
dns_resolver object.

Merge branch '605-fixes' into 'master'

Fix missing siphash_test from Kyuafile and missing endian.h from MSVC build files

Closes #605

See merge request isc-projects/bind9!2173

Add siphash_test to lib/isc/tests/Kyuafile

Add endian.h to libisc.vcxproj.* files

Merge branch '1153-fix-broken-atomics-on-windows' into 'master'

Fixup the increment vs decrement typo in lib/isc/win32/socket.c

Closes #1153

See merge request isc-projects/bind9!2172

Fixup the increment vs decrement typo in lib/isc/win32/socket.c

Merge branch '605-add-siphash24-wpk' into 'master'

Synchronize the Cookie algorithm to SipHash-2-4 with other open-source DNS vendors

Closes #605

See merge request isc-projects/bind9!1788

Add CHANGES note

Add release notes

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