Remove local implementations of hash functions

Enforce crypto selection in configure

Remove -p option usage from rpz test

Update tests to not use '>&-' that closes file descriptor, but instead use correct redirection to '>/dev/null'

Remove genrandom command and all usage of specific random files throughout the system test suite

Refactor and cleanup duplicate code in the isc_hmac functions

Replace all random functions with isc_random, isc_random_buf and isc_random_uniform API.

The three functions has been modeled after the arc4random family of
functions, and they will always return random bytes.

The isc_random family of functions internally use these CSPRNG (if available):

1. getrandom() libc call (might be available on Linux and Solaris)
2. SYS_getrandom syscall (might be available on Linux, detected at runtime)
3. arc4random(), arc4random_buf() and arc4random_uniform() (available on BSDs and Mac OS X)
4. crypto library function:
4a. RAND_bytes in case OpenSSL
4b. pkcs_C_GenerateRandom() in case PKCS#11 library

Merge branch '171-lgtm-issues' into 'master'

Resolve "problems detected by LGTM static analyzer"

Closes #171

See merge request isc-projects/bind9!213

remove unnecessary comparison

silence warnings about unnecessary comparisons

- these are cases where result has been explicitly set, so
  if (result != ISC_R_SUCCESS) is unnecessary

add header guards in files that were missing them

add header guard when generating bind.keys.h

change "key" to "tsigkey" to silence "short global name" warning

fix shadowed global variables

fix duplicate include guard in dnsconf.h

- IMHO we should consider removing dnsconf.c and deprecating the
  /etc/dns.conf file, though, as I don't think it's likely anyone
  is using it

Merge branch 'fix-win32-def' into 'master'

add missing symbols

See merge request isc-projects/bind9!218

add missing symbols

Merge branch '217-remove-rbtdb64' into 'master'

Resolve "remove rbtdb64"

Closes #217

See merge request isc-projects/bind9!209

remove the rbtdb64 database implementation

Merge branch '202-cppcheck-reporting-miscellaneous-issues' into 'master'

Resolve "cppcheck reporting miscellaneous issues"

Closes #202

See merge request isc-projects/bind9!202

remove exit calls after FATAL_ERROR

remove dead code

use %u instead of %d for unsigned int arguments

Merge branch '206-nslookup-accepts-any-bogus-option-interpret-it-as-vc' into 'master'

Resolve "nslookup accepts any -bogus -option, interpret it as -vc"

Closes #206

See merge request isc-projects/bind9!205

Remove bogus comparison

Merge branch '213-add-raflag-and-tcflag-to-dig' into 'master'

Resolve "Add +raflag and +tcflag to dig."

Closes #213

See merge request isc-projects/bind9!201

dig: add the ability to set RA and TC in queries

Merge branch '145-dnskey-sigvalidity' into 'master'

Resolve "different RRSIG expiry for DNSKEY"

Closes #145

See merge request isc-projects/bind9!121

new option "dnskey-sig-validity"

- overrides "sig-validity-interval" for DNSKEY, CDNSKEY and CDS RRSIGs

Merge branch 'placeholder' into 'master'

placeholder

See merge request isc-projects/bind9!211

placeholder

Merge branch 'rm-clang-format' into 'master'

Remove .clang-format as we are going to use uncrustify anyway

See merge request isc-projects/bind9!207

We are going to use uncrustify

Merge branch 'clang-format-updates' into 'master'

Add more tweaks to .clang-format style for experiments

See merge request isc-projects/bind9!206

Add more tweaks to .clang-format style for experimenting

Merge branch '46-add-clang-format-configuration' into 'master'

Add basic clang-format configuration that match our current formatting style closest

See merge request isc-projects/bind9!204

Add basic clang-format configuration that match our current formatting style closest

Merge branch '37-implement-geoff-huston-s-trusted-key-sentinel-feature' into 'master'

Implement Geoff Huston's trusted key sentinel feature

Closes #37

See merge request isc-projects/bind9!123

CHANGES entry for root-key-sentinel

add release note for root-key-sentinel

add system test for root-key-sentinel

add named.conf option root-key-sentinel

detect and processes root-key-sentinel labels.

Merge branch '200-add-clang-to-gitlab-ci' into 'master'

Add clang (with clang specific extra options like -Wenum-conversion) to out GitLab CI

Closes #200

See merge request isc-projects/bind9!203

Run the clang-compiled unittest

Add clang to the build job matrix and build gcc and clang builds with -Wall -Wextra

Merge branch 'fix-strtok' into 'master'

fix a missing variable declaration needed by strtok_r on non-threaded builds

See merge request isc-projects/bind9!200

fixed a missing variable declaration needed by strtok_r on non-threaded builds

Merge branch '203-max-cache-ttlval' into 'master'

max-cache-ttl and other interval options should take ttlval arguments

Closes #203

See merge request isc-projects/bind9!197

Allow TTL values when configuring TTLs and time intervals.

Options updated to use ttlvals: max-cache-ttl, max-ncache-ttl,
max-policy-ttl, fstrm-set-reopen-interval, interface-interval, and
min-update-interval.

Merge branch 'u/fanf2/werror-maybe-uninitialized' into 'master'

Appease -Werror=maybe-uninitialized before calling strtok_r()

See merge request isc-projects/bind9!198

Appease -Werror=maybe-uninitialized before calling strtok_r()

Merge branch 'add-print.h' into 'master'

add missing include

See merge request isc-projects/bind9!195

add missing include

Merge branch '178-cleanup-isc-string' into 'master'

Cleanup <isc/string.h> functions

See merge request isc-projects/bind9!163

Add CHANGES entry:

4924.   [cleanup]       Clean up the isc_string_* namespace and leave
                        only strlcpy and strlcat. [GL #178]

We use too old Visual Compiler to use scoped variables

Remove usage of strcasestr, so we don't have to have replacement function on Windows

Remove isc_string_strcasestr implementation and clean up the header and headers

Replace all usage of non-reentrant strtok() with strtok_r()

Replace usage of strsep with POSIX strtok_r()

Remove unused isc_string_regiondup function.

Remove isc_string_printf and isc_string_printf_truncate.

Replace with simple snprintf() as appropriate.

Remove isc_string_copy, isc_string_copy_truncate and isc_string_append.

Use strlcpy and strlcat as appropriate instead.

Replace isc_string_touint64 with strtoull (C99)

Merge branch 'socket-options-refactoring' into 'master'

Refactoring #defines into typedef enums

See merge request isc-projects/bind9!135

Add CHANGES entry:

4923.   [cleanup]       Refactor socket and socket event options into
                        enum types. [GL !135]

Change ISC_SOCKEVENTATTR_* defines into isc_sockeventattr_t enum

Refactor ISC_SOCKET_* defines into an isc_socket_options_t enum

Merge branch '189-fix-win32-build' into 'master'

Fix runtime assertion on Windows due isc_file_template being out-of-sync

Closes #189

See merge request isc-projects/bind9!193

Sync isc_file_template functions between unix and win32, so they are same

Merge branch '189-fix-tsig-dump-keyfile-name-generation-issues' into 'master'

Resolve "Fix TSIG dump keyfile name generation issues"

Closes #189

See merge request isc-projects/bind9!164

Fix TSIG dump keyfile name buffer size issues

Define PATH_MAX on Windows as _MAX_PATH, so we can use it everywhere

Merge branch '193-fix-variable-definitions-in-bin-tests-Makefile.in' into 'master'

Fix variable definitions in bin/tests/Makefile.in

Closes #193

See merge request isc-projects/bind9!183

Alter distclean rule to prevent recursing into the same directory twice

Commit f87e0c03ee removed the "system" directory from the TESTDIRS
variable in bin/tests/Makefile.in in an attempt to fix "make distclean"
which was broken since commit 0d784de16a.  However, this change
prevented any system tests from being run when "make test" is invoked.

We now put it back into both SUBDIRS and TESTDIRS, but with a modified
rule to check for the existence of a Makefile in each subdirectory before
trying to run make there. This prevents "make distclean" from trying to
run again in a directory where it's already been run.

Fail CI pipeline when "make test" does not run any system tests

Apart from ensuring "make test" returns 0, also check whether any system
test output was generated as a result of running it.  This prevents the
CI job running system tests from succeeding unless it actually tests
something.

Merge branch 'revert-freebsd-runner' into 'master'

Revert freebsd CI runner

See merge request isc-projects/bind9!189

Revert "Merge branch 'gitlab-ci-freebsd11' into 'master'"

This reverts commit f595ab93eb6ea98167ac1c60a6d9dddc7c6f1fa7, reversing
changes made to deae7d33f1330c3f60b2c28aab61bf41f164fe5c.

Merge branch '197-dnstap-sockaddr' into 'master'

Resolve "dnstap: log actual local IPv6 address, not :: listening address"

Closes #197

See merge request isc-projects/bind9!188

CHANGES note

dnstap: log actual local IPv6 address, not :: listening address

Merge branch '194-libdns-cleanup-cleanup' into 'master'

Fixup the one last occurence of dns_dt_create2 (in dnstap enabled build)

Closes #194

See merge request isc-projects/bind9!185

Rename the last occurence of dns_dt_create2.

Merge branch 'gitlab-ci-freebsd11' into 'master'

Add FreeBSD 11 to GitLab CI

See merge request isc-projects/bind9!184

Add FreeBSD 11 GitLab CI Runner

Merge branch '183-add-dns_fixedname_initname' into 'master'

Add and use dns_fixedname_initname()

Closes #183

See merge request isc-projects/bind9!161

Add CHANGES entry:

4921.   [cleanup]       Add dns_fixedname_initname() and refactor the caller
                        code to make usage of the new function, as a part of
                        refactoring dns_fixedname_*() macros were turned into
                        functions. [GL #183]

Do not access dns_fixedname_t fields directly

Employ dns_fixedname_name() and dns_fixedname_initname() to no longer
directly access dns_fixedname_t fields.

Use dns_fixedname_initname() where possible

Replace dns_fixedname_init() calls followed by dns_fixedname_name()
calls with calls to dns_fixedname_initname() where it is possible
without affecting current behavior and/or performance.

This patch was mostly prepared using Coccinelle and the following
semantic patch:

    @@
    expression fixedname, name;
    @@
    - dns_fixedname_init(&fixedname);
     ...
    - name = dns_fixedname_name(&fixedname);
    + name = dns_fixedname_initname(&fixedname);

The resulting set of changes was then manually reviewed to exclude false
positives and apply minor tweaks.

It is likely that more occurrences of this pattern can be refactored in
an identical way.  This commit only takes care of the low-hanging fruit.

Add a dns_fixedname_initname() helper function

This also turns the dns_fixedname macros into functions.

Merge branch 'libdns-cleanup2' into 'master'

libdns cleanup

See merge request isc-projects/bind9!172

libdns refactoring: CHANGES entry

libdns refactoring: get rid of multiple versions of dns_keytable_add, dns_iptable_addprefix and dns_iptable_addprefix

libdns refactoring: get rid of multiple versions of dns_dt_create, dns_view_setcache, dns_zt_apply, dns_message_logfmtpacket, dns_message_logpacket, dns_ssutable_checkrules and dns_ttl_totext

libdns refactoring: get rid of multiple versions of dns_view_find, dns_view_findzonecut and dns_view_flushcache

libdns refactoring: get rid of multiple versions of dns_xfrin_create, dst_key_generate, dst_lib_init and dst_context_create

libdns refactoring: integrate zone->options and zone->options2 into one enum, removing unnecessary flags.

libdns refactoring: get rid of multiple versions of dns_zone_setfile, dns_zone_notifyreceive, dns_zone_dumptostream, dns_zone_getserial

libdns refactoring: get rid of 3 versions of dns_resolver_createfetch