When ISC assertions are disabled, still execute the condition to prevent unused variable warnings/errors from the compiler

Add a GitLab CI job that runs with all assertions disabled

Turn (int & flag) into (int & flag) != 0 when implicitly typed to bool

Add small tweaks to the code to fix compilation when ISC assertions are disabled

While implementing the new unit testing framework cmocka, it was found that the
BIND 9 code doesn't compile when assertions are disabled or replaced with any
function (such as mock_assert() from cmocka unit testing framework) that's not
directly recognized as assertion by the compiler.

This made the compiler to complain about blocks of code that was recognized as
unreachable before, but now it isn't.

The changes in this commit include:

* assigns default values to couple of local variables,
* moves some return statements around INSIST assertions,
* adds __builtin_unreachable(); annotations after some INSIST assertions,
* fixes one broken assertion (= instead of ==)

Merge branch 'wpk-get-rid-of-bashism-in-CDS-test' into 'master'

Small system tests fixes for Solaris

See merge request isc-projects/bind9!988

Remove unnecessary sed from autosign test

Use 'local' variable in echo functions in tests

Get rid of bashism in CDS test

Merge branch '4-add-ac_posix_shell-m4-macro' into 'master'

Add AX_POSIX_SHELL autoconf macro to have POSIX-compatible shell in $SHELL

See merge request isc-projects/bind9!986

Add CHANGES entry for AX_POSIX_SHELL

Remove m4/ files from copyright check

Add and use AX_POSIX_SHELL autoconf macro, so we can rely on POSIX shell features

Merge branch 'wpk-fix-typo-thread-c' into 'master'

Fix a typo in lib/isc/unix/thread.c

See merge request isc-projects/bind9!990

Fix a typo in lib/isc/unix/thread.c

Merge branch '650-fix-race-in-dig' into 'master'

Add some DBC checks in dighost; fix race between clear_query and send_done in dighost

Closes #650

See merge request isc-projects/bind9!965

Add some DBC checks in dighost; fix race between clear_query and send_done in dighost

Merge branch 'wpk-multiple-task-queues' into 'master'

Multiple task queues

See merge request isc-projects/bind9!878

CHANGES note

Use larger quantum for network tasks

Use a single wake_all_queues() function to wake all queues

Switch from privileged to un-privileged mode under lock

Document isc_task_sendto properly, make sure that cpu we're sending to is always sane

Use proper memory ordering for tasks_running/tasks_ready

Comment about taskmgr exclusive mode, fix a REQUIRE.

Allow slight over-quota in'checking lame server clients are dropped at the soft limit' test

Saner exclusive task handling in taskmgr

Formatting

Get rid of isc_taskmgr_setmode, we only use it to set privileged mode

Make sure all priority tasks are done before entering normal execution

Post shutting down tasks always to manager 0

Separate structure for each thread/queue; 2-phase-locking for exclusive tasks

Always restart dispatchers on empty readyq

Taskmgr shutdown fixes

Multiple worker queues

isc_thread_setaffinity()

Merge branch 'rndc-nta-preen' into 'master'

Fixes for `rndc nta` user interface

See merge request isc-projects/bind9!845

CHANGES, release note

Fixes for `rndc nta` user interface

Tell the user explicitly about their mistakes:

* Unknown options, e.g. -list instead of -dump
  or -delete instead of -remove.

* Unknown view names.

* Excess arguments.

Include the view name in `rndc nta -dump` output, for consistency with
the NTA add and remove actions.

When removing an NTA from all views, do not abort with an error if the
NTA was not found in one of the views.

Merge branch '653-idnout-only-on-tty' into 'master'

Enable idnout output only on tty, disable it when the stdout is not a tty

Closes #653

See merge request isc-projects/bind9!973

Add CHANGES entry for GL #653

Add release notes for IDN processing update

Update the IDN documentation for nslookup

Change the dig documentation

Update the idna tests for the new non-tty defaults

Enable IDN processing (both idnin and idnout) only on tty, disable it when the stdout is not a tty

Merge branch '601-build-option-with-python-should-default-to-yes' into 'master'

Resolve "Build option --with-python should default to yes"

Closes #601

See merge request isc-projects/bind9!980

require python to be explicitly disabled

Merge branch 'u/fanf2/deprecate-ip6-int' into 'master'

Abolish ip6.int support in `dig` and `mdig`

See merge request isc-projects/bind9!969

add CHANGES note

Abolish ip6.int support in `dig` and `mdig`

Merge branch '617-if-rrl-is-configured-the-require-server-cookie-yes-is-ignored' into 'master'

Resolve "If RRL is configured the "require-server-cookie yes;" is ignored."

Closes #617

See merge request isc-projects/bind9!974

add CHANGES note

test require-server-cookie with rate-limit

check requireservercookie even if rrl is configured

Merge branch '634-unchecked-returns-in-resolver-c' into 'master'

Resolve "Unchecked returns in resolver.c"

Closes #634

See merge request isc-projects/bind9!929

two dns_name_dup calls were not checked

Merge branch '645-fix-missing-buffer-init' into 'master'

dighost.c: after invalidating buffer we need to reinit it

Closes #645

See merge request isc-projects/bind9!976

dighost.c: after invalidating buffer we need to reinit it

Merge branch '16-cleanup-finds-before-sending-final-query' into 'master'

Cleanup fctx->finds before sending 'final' query after qname minimization.

Closes #16

See merge request isc-projects/bind9!962

Cleanup fctx->finds before sending 'final' query after qname minimization.

At the beginning of qname minimization we get fctx->finds filled with what's
in the cache at this point, in worst case root servers. After doing full
run querying for NSes at different levels we need to clean it and refill
it with proper values from cache.

Merge branch 'fix-dnstap-test' into 'master'

disable QNAME minimization in the dnstap test so packet counts will be correct

See merge request isc-projects/bind9!971

disable QNAME minimization in the dnstap test so packet counts will be correct

Merge branch 'wpk-qmin-intermittent-fixes' into 'master'

Avoid a race in qmin test

See merge request isc-projects/bind9!954

Avoid a race in qmin test

Merge branch 'u/fanf2/configure-backtrace' into 'master'

Correctly report backtrace support when `configure` finishes

See merge request isc-projects/bind9!967

regen

Correctly report backtrace support when `configure` finishes

Merge branch '611-extend-serve-stale-tests' into 'master'

Extend serve-stale tests

Closes #611

See merge request isc-projects/bind9!963

Check serve-stale behavior with a cold cache

Ensure that serve-stale works as expected when returning stale answers
is enabled, the authoritative server does not respond, and there is no
cached answer available.

Check TTL of stale answers

Make sure that stale answers returned when the serve-stale feature is
enabled have a TTL matching the value of the stale-answer-ttl setting.

Merge branch 'wpk-get-rid-of-socket-v-functions' into 'master'

Get rid of socket *v functions

See merge request isc-projects/bind9!932

[squash] Rework code preparing a HTTP response for sending in lib/isc/httpd.c

CHANGES entry

Remove vector socket functions from Unix socket code and library headers

Remove the following functions in order to simplify socket code:

  - isc_socket_recvv()
  - isc_socket_sendtov()
  - isc_socket_sendtov2()
  - isc_socket_sendv()

Remove vector socket functions from Windows socket code

Remove the following functions in order to simplify socket code:

  - isc_socket_recvv()
  - isc_socket_sendtov()
  - isc_socket_sendtov2()
  - isc_socket_sendv()

Do not use vector socket functions in diagnostic tools

Refactor diagnostic tools code to no longer use:

  - isc_socket_recvv()
  - isc_socket_sendtov2()
  - isc_socket_sendv()

as these functions will be removed shortly.

Do not use vector socket functions in HTTP server code

Refactor code in lib/isc/httpd.c to no longer use isc_socket_sendv() as
this function will be removed shortly.

Merge branch '649-resolver-test-failing' into 'master'

Resolve "resolver test failing"

Closes #649

See merge request isc-projects/bind9!959

use new packet for response; rename variable to request and response

Merge branch 'wpk-fix-catz-test-race' into 'master'

catz test: wait for dom8.example to be transferred before checking its availability

See merge request isc-projects/bind9!946

Fix style in catz test

catz test: wait for dom8.example to be transferred before checking its availability

Merge branch '647-config-h' into 'master'

remove config.h references

Closes #647

See merge request isc-projects/bind9!958

remove config.h references

Merge branch 'silence-warning' into 'master'

silence warning from missing print.h

See merge request isc-projects/bind9!955

silence warning from missing print.h

Merge branch '84-destroy-task-first-when-destroying-rpzs' into 'master'

Destroy updater task first when destroying rpzs.

Closes #84

See merge request isc-projects/bind9!944

CHANGES entry

Destroy task first when destroying catzs.

    When freeing catzs structures we need to kill the updater task first.
    Otherwise we might race with the updater and there might be a crash
    on shutdown.

Destroy task first when destroying rpzs.

When freeing rpzs structures we need to kill the updater task first.
Otherwise we might race with the updater and there might be a crash
on shutdown.

Merge branch '644-fix-isc_buffer_copyregion-for-auto-reallocated-buffers' into 'master'

Fix isc_buffer_copyregion() for auto-reallocated buffers

Closes #644

See merge request isc-projects/bind9!949

Add CHANGES entry

5072. [bug] Add unit tests for isc_buffer_copyregion() and fix its
behavior for auto-reallocated buffers. [GL #644]

Fix isc_buffer_copyregion() for auto-reallocated buffers

While isc_buffer_copyregion() calls isc_buffer_reserve() to ensure the
target buffer will have enough available space to append the contents of
the source region to it, the variables used for subsequently checking
available space are not updated accordingly after that call.  This
prevents isc_buffer_copyregion() from working as expected for
auto-reallocated buffers: ISC_R_NOSPACE will be returned if enough space
is not already available in the target buffer before it is reallocated.
Fix by calling isc_buffer_used() and isc_buffer_availablelength()
directly instead of assigning their return values to local variables.

Add unit tests for isc_buffer_copyregion()

Add some basic checks for isc_buffer_copyregion() to ensure it behaves
as expected for both fixed-size buffers and buffers which can be
automatically reallocated.  Adjust the list of headers included by
lib/isc/tests/buffer_test.c so that it matches what that test program
really uses.

Merge branch 'ondrej/cleanup-leftover-fatal' into 'master'

Remove double result check leftover

See merge request isc-projects/bind9!945

Remove double result check leftover

Merge branch '631-dns_rdata_compare-for-nxt-performs-case-sensitive-name-comparison' into 'master'

Resolve "dns_rdata_compare() for NXT performs case sensitive name comparison"

Closes #631

See merge request isc-projects/bind9!917

add CHANGES note

compare_nxt compared records with identical next fields case insensitively

Merge branch '638-record-types-with-empty-rdata-fields-were-not-being-handled-correctly' into 'master'

Resolve "Record types with empty rdata fields were not being handled correctly."

Closes #638

See merge request isc-projects/bind9!919