Do not fail on NULL passed to OpenSSL_free

Some plugins might call it after deconstruction. Do not crash if there
is no reason for it.

(cherry picked from commit a26673a08863cdcb48ea91dd9c5e0ef4386ce931)

Merge branch '766-querytrace-doc-v9_12-v9_11' into 'v9_11'

Resolve "--enable-querytrace has negative performance impact - update the documentation to say this"

See merge request isc-projects/bind9!1420

added querytrace documentation in README

also rearranged things a little, adding a "dependencies" section

(cherry picked from commit 51048ca06ecc3c57bae7f8c16b3ca9e3b1235619)
(cherry picked from commit 87b0098b4ea8725efde9f7917770187af38c262c)

Merge branch 'michal/add-fedora-29-to-ci-v9_11' into 'v9_11'

[v9_11] Add Fedora 29 to CI

See merge request isc-projects/bind9!1416

Add Fedora 29 to CI

Modify .gitlab-ci.yml so that every CI pipeline also builds and tests
BIND on Fedora 29.

(cherry picked from commit 38b6c5c444c1ce9a2595ad5bcd960ac0cc7b98ee)

Merge branch '259-named-args-v9_11' into 'v9_11'

add properly-formatted -D options to named.args files

See merge request isc-projects/bind9!1412

also add -D options for transient named processes started in tests.sh

(cherry picked from commit dd45831acc06b15e5814b7ce3eec903d2db23904)

add properly-formatted -D options to named.args files

this prevents servers that use arguments specified in named.args
from appearing different in 'ps' output from servers run with arguments
from start.pl

(cherry picked from commit 175d6e9bfb7085a70051d35a70b98debff3fddbe)

Merge branch 'each-fix-logfileconfig-v9_11' into 'v9_11'

reset SYSTEMTESTTOP when changing directories

See merge request isc-projects/bind9!1410

reset SYSTEMTESTTOP when changing directories

(cherry picked from commit 70f36a25e4880635645b5a6fd850421da0a543e8)

Merge branch 'each-win32-tests-v9_11' into 'v9_11'

fix win32 tests (v9_11)

See merge request isc-projects/bind9!1402

fix runtime test

named doesn't understand cygwin paths, so we need to use
`cygpath -aw .` instead of `pwd` to get the absolute path to the
working directory.

fix rrl test

strip CR characters before using awk/sed

fix rpz test

- work around a CR newline problem
- use rndc to stop servers

fix rpzrecurse test

- backport from v9_12 changes to enable rpzrecurse test to
  suspend and resume servers on win32
- use rndc to stop servers

fix dnssec test

- work around CR issues
- use UTC for time comparisons
- use $DIFF instead of cmp

fix legacy test

use rndc rather than signals to stop the server

fix fetchlimit test

use TCP for the test queries in between UDP bursts; this avoids
congestion issues that interfered with the test on windows

fix sfcache test

use a lame server configuration to force SERVFAILs instead of killing ns2.
this prevents test failures that occurred due to a different behavior of
the netowrking stack in windows.

fix nsupdate test

rndc_reload was failing on windows

fix rndc test

use regex instead of exact string matching to deal with CR at end of line

fix statistics test

the active sockets test is supposed to be commented out on win32, but
only part of it was

fix redirect test

strip CR characters before using sed

fix notify test

test the average delay between notifies instead of the minimum delay;
this helps avoid unnecessary test failures on systems with bursty
network performance.

fix masterformat test

use stop.pl instead of rndc to stop server

fix inline test

use regex instead of exact string matching, to deal with CR at end of ine

fix forward test

strip CR characters before using sed

fix cookie test

strip CR characters before comparing files

use $DIFF instead of diff

set and use SYSTEMTESTTOP consistently

more reliable method for killing "ans" servers on windows

as perl and python are both native to cygwin, we don't want to use
the "kill -f" option to terminate them.

update ifconfig.bat with current test interfaces

the addresses set up in ifconfig.bat were out of sync with the
ones in ifconfig.sh

update conf.sh.win32 test list

- the test lists in conf.sh.in and conf.sh.win32 were out of sync

Merge branch '560-dnssec-keymgr-root-v9_11' into 'v9_11'

improve handling of trailing dots in dnssec-keymgr and dnssec-coverage

See merge request isc-projects/bind9!1395

improve handling of trailing dots in dnssec-keymgr and dnssec-coverage

- mishandling of trailing dots caused bad behavior with the
  root zone or names like "example.com."
- fixing this exposed an error in dnssec-coverage caused the
  wrong return value if there were KSK errors but no ZSK errors
- incidentally silenced the dnssec-keygen output in the coverage
  system test

(cherry picked from commit 1ccf4e6c169993a20305dfec5a23a55e1d920fe1)

Merge branch 'v9_11-dig-sigchase-topdown' into 'v9_11'

Try to fix crash at +sigchase +topdown

See merge request isc-projects/bind9!1391

Try to fix crash at +sigchase +topdown

When multilabel name is already cached, child_of_zone fails check when
zone_name is direct child of name. Error is ignored and crashes on
expectation child_name was initialized. Handle the error and relax the
check.

Reproducer:
dig isc.org
dig +sigchase +topdown isc.org

Merge branch 'revert-7663d075' into 'v9_11'

Revert "Merge branch 'ondrej/fix-race-condition-in-dnstap-v9_11' into 'v9_11'"

See merge request isc-projects/bind9!1384

Revert "Merge branch 'ondrej/fix-race-condition-in-dnstap-v9_11' into 'v9_11'"

This reverts merge request !1345

Merge branch '117-running-dnssec-keymgr-with-old-keys-inactivates-deletes-them-immediately-v9_12-v9_11' into 'v9_11'

Resolve "Running dnssec-keymgr with old keys inactivates/deletes them immediately"

See merge request isc-projects/bind9!1380

introducing keymgr need to preserve functionality

(cherry picked from commit 083b730ec7fba6b7ad457451996c806223815c70)
(cherry picked from commit 15b42407647e1b898c10c7edd57b86dfac91bdc8)

Merge branch 'each-remove-fprintf-v9_11' into 'v9_11'

clean up debugging fprintf

See merge request isc-projects/bind9!1370

removed a debugging fprintf

(cherry picked from commit 6c478a3daeab6e6df1bb005d4d0666967d75cda3)

Merge branch 'michal/gitlab-ci-cleanup-v9_11' into 'v9_11'

[v9_11] GitLab CI cleanup

See merge request isc-projects/bind9!1364

Add comments

(cherry picked from commit a27b9dff18efa692162c6de550b6e4082ad024ed)

Unify YAML style used throughout .gitlab-ci.yml

(cherry picked from commit a61bf3968899ec71ba491991c81ebd5982d7b9b3)

Tweak pipeline triggering settings

In an attempt to ensure that:

  - all important changes to repository contents are tested,
  - pipelines are not automatically created for every single push,
  - some flexibility is allowed for corner cases,

change pipeline triggering settings so that:

  - full build & test pipelines are only automatically created for merge
    requests and tags (both for creation and updates),

  - pipelines for other repository changes (e.g. pushes to arbitrary
    branches) can only be created manually, using GitLab's web
    interface,

  - merging a merge request only causes jobs pushing the updated ARM to
    GitLab Pages to be run (as semi-linear Git history is enforced and
    thus testing a MR is identical to testing the target branch
    post-merge in terms of code),

  - repository synchronization does not trigger duplicate pipelines in
    projects which are set as mirroring targets.

(cherry picked from commit 1c8c1815e4bf65e8f367c1f3141adf70a20a4295)

Reorder job definitions

Group jobs by build type and operating system to make the layout of
.gitlab-ci.yml more consistent and improve locality of YAML references.

(cherry picked from commit a1dbec3b0850b05b3b3872ced44344fee337713f)

Use a common naming pattern for all jobs

Make sure all jobs are named using the following pattern:

    [<job-type>:]<build-type>:<system>:<architecture>

where specifying <job-type> is optional for "precheck" and "build" jobs.

This should make it easier to quickly recognize:

  - what kind of actions are performed by each job,
  - which BIND build flavor is used by each job,
  - which operating system image is used by each job.

(cherry picked from commit 1fe432c6c385fc36ff4aa49b692b98ba043b413e)

Define a template for precheck jobs

(cherry picked from commit e0b5dd67a1ca45bd341369e125fbb71cf3c8d304)

Merge install_test_job template into the job definition as it is only used once

(cherry picked from commit de1948398d1055ab3b3e17e2f2512c2d6baebf55)

Shorten Docker image definitions by using YAML anchors for runner tags

While we are at it, drop use of the "docker" tag since all BIND CI jobs
are currently run inside Docker containers.

(cherry picked from commit 7dd329d3857d767b28e4d88393108d332d0f5ffb)

Build ARM in the build stage

There is no need to build BIND binaries before building docs and thus
the job building the current version of the ARM can be moved to the
build stage of CI.

(cherry picked from commit 41a67147fee92018bd4065a4b0cc965f5df987a7)

Replace double colon with a single colon

(cherry picked from commit 75b26d3f5068fa478444b2e0cd64645efb6b12c9)

Remove unused parts of .gitlab-ci.yml

Remove the following from .gitlab-ci.yml:

  - unused variable definitions,
  - unused Docker image definitions,
  - commands which have no effect,
  - sections which were commented out.

(cherry picked from commit 9893bd32463a3808bc1048a3539f854ac6ea1bec)

Merge branch '781-fix-python-check-in-configure-script-v9_11' into 'v9_11'

[v9_11] Resolve "configure script in BIND 9.13.5 release fails to detect python"

See merge request isc-projects/bind9!1363

Add some combinations of --with-python=<path>/--with-python/--without-python to GitLab CI

(cherry picked from commit 99cf0faec0fb68165bae0aefef4ab25a35d1c465)

When --with-python is given to configure, use default search list for python binaries instead of 'yes'

(cherry picked from commit 3951cb053b4d451fdb0580294fb78bc792b828da)

Merge branch '752-disable-forwarders-when-priming-v9_12-v9_11' into 'v9_11'

If possible don't use forwarders when priming the resolver.

See merge request isc-projects/bind9!1357

If possible don't use forwarders when priming the resolver.

If we try to fetch a record from cache and need to look into
hints database we assume that the resolver is not primed and
start dns_resolver_prime(). Priming query is supposed to return
NSes for "." in ANSWER section and glue records for them in
ADDITIONAL section, so that we can fill that info in 'regular'
cache and not use hints db anymore.
However, if we're using a forwarder the priming query goes through
it, and if it's configured to return minimal answers we won't get
the addresses of root servers in ADDITIONAL section. Since the
only records for root servers we have are in hints database we'll
try to prime the resolver with every single query.

This patch adds a DNS_FETCHOPT_NOFORWARD flag which avoids using
forwarders if possible (that is if we have forward-first policy).
Using this flag on priming fetch fixes the problem as we get the
proper glue. With forward-only policy the problem is non-existent,
as we'll never ask for root server addresses because we'll never
have a need to query them.

Also added a test to confirm priming queries are not forwarded.

(cherry picked from commit b49310ac06ac87733dc2867828e61370a84b2a9a)
(cherry picked from commit f8963ad70e222edad0c1e64f855f7fb41fb13c3c)

Merge branch 'v9_11-normalize-files' into 'v9_11'

V9 11 normalize files

See merge request isc-projects/bind9!1352

Normalize windows files and shell scripts

Commands used:
git add --chmod=+x bin/tests/system/ifconfig.sh mkinstalldirs
git add --renormalize 'win32utils/**.txt' '**.dsw'

Change eol of windows files

Release tarballs have different eol of windows only files. Change them
also in git, so checkout matches archvies as much as possible.

Merge branch 'pkcs11-pubattr-check-v9_11' into 'v9_11'

Make sure null atributes are never used

See merge request isc-projects/bind9!1355

Make sure null atributes are never used

Add INSIST to pubattr fetching where null might occur in therory. Make
sure null is never dereferenced.

(cherry picked from commit fe9ef0d9f57a2e3f2902cf93d5f00aac3286dd99)

Merge branch '792-bind9-doesn-t-tcp-retransmit-v9_11' into 'v9_11'

Resolve "bind9 doesn't tcp retransmit"

See merge request isc-projects/bind9!1349

adjust timeout to allow for ECN negotiation failures

(cherry picked from commit dadb924be766eb8bfb922eacc8015c33c08d206b)

Merge branch 'ondrej/fix-race-condition-in-dnstap-v9_11' into 'v9_11'

(v9_11) Fix race condition in cleanup part of dns_dt_create()

See merge request isc-projects/bind9!1345

Add CHANGES entry for GL!1323

(cherry picked from commit 4b5337e06b545c18a1177e8c63e9b205c475386a)

Fix race condition in cleanup part of dns_dt_create()

(cherry picked from commit 482dd7eed379382d627e81822bad10e876501dc4)

Merge branch '590-win32-sample-gai-c-should-call-wsastartup-v9_11' into 'v9_11'

Resolve "[Win32] sample-gai.c should call WSAStartup()"

See merge request isc-projects/bind9!1343

add CHANGES

(cherry picked from commit 4500d9e91af4a09b3b7c47d0de4740b7cdd00399)

ensure that WSAStartup is called before getservbyname

(cherry picked from commit ac01359871ae1e80b3915b7744373422ddec7ea1)

Merge branch 'u/fanf2/rndc-managed-keys-v9_12-v9_11' into 'v9_11'

Fix a few cosmetic issues with `rndc managed-keys`

See merge request isc-projects/bind9!1341

add multi-view server and tests

(cherry picked from commit 7122b5786db12b0811f868977740e12e540488a0)
(cherry picked from commit 22b77f45b739a211c9524ce2528a2b41a809f19e)

add CHANGES note

(cherry picked from commit 2d3f1597079b6814a3edbd10af381b46a3818ad1)

Fix a few cosmetic issues with `rndc managed-keys`

The handling of class and view arguments was broken, because the code
didn't realise that next_token() would overwrite the class name when
it parsed the view name. The code was trying to implement a syntax
like `refresh [[class] view]`, but it was documented to have a syntax
like `refresh [class [view]]`. The latter is consistent with other rndc
commands, so that is how I have fixed it.

Before:

$ rndc managed-keys refresh in rec
rndc: 'managed-keys' failed: unknown class/type
unknown class 'rec'

After:

$ rndc managed-keys refresh in rec
refreshing managed keys for 'rec'

There were missing newlines in the output from `rndc managed-keys
refresh` and `rndc managed-keys destroy`.

Before:

$ rndc managed-keys refresh
refreshing managed keys for 'rec'refreshing managed keys for 'auth'

After:

$ rndc managed-keys refresh
refreshing managed keys for 'rec'
refreshing managed keys for 'auth'

(cherry picked from commit 6a3b851f72929802fc10a51f17b170db54988021)
(cherry picked from commit bc984ace12ef57ddbcb8ec6a87dd541b58c80f63)

Merge branch 'fix-dnssec-test-v9_12-v9_11' into 'v9_11'

b/t/s/dnssec/tests.sh: Cleanup showprivate() function

See merge request isc-projects/bind9!1337

b/t/s/dnssec/tests.sh: Cleanup showprivate() function

(cherry picked from commit b241dc58ec9400d690b43ed6a04cd953fe549de8)

Merge branch 'each-fix-dnssec-test-error-v9_11' into 'v9_11'

DNSSEC test error

See merge request isc-projects/bind9!1334

fix testing errors

- the checkprivate function in the dnssec test set ret=0, erasing
  results from previous tests and making the test appear to have passed
  when it shouldn't have
- checkprivate needed a delay loop to ensure there was time for all
  private signing records to be updated before the test

(cherry picked from commit 82e83d5dc7abdb05908c7971571eaa2d673d8691)

Merge branch 'u/fanf2/rndc-alphabetize-v9_11' into 'v9_11'

cleanup: alphabetize rndc command dispatch

See merge request isc-projects/bind9!1332

cleanup: alphabetize rndc command dispatch

Merge branch '784-bind-9-12-3-p1-fatal-error-v9_11' into 'v9_11'

Resolve "bind 9.12.3-P1 fatal error"

See merge request isc-projects/bind9!1325

add CHANGES

(cherry picked from commit 9dd39d53853e6afb7b042cf4ee1e4f9bbdb2efac)

update refreshkeytime

(cherry picked from commit ca977e39767902612498776db4609456b600d108)

Merge branch 'marka-maybe_numeric-and-nul-v9_11' into 'v9_11'

maybe_numeric failed to handle NUL in text region.

See merge request isc-projects/bind9!1322

maybe_numeric failed to handle NUL in text region.

(cherry picked from commit ee23780246e89affc31c739bbc4cbd429410fba2)

Merge branch 'marka-fail-when-required-field-is-missing-v9_11' into 'v9_11'

Ensure base64/base32/hex fields in DNS records that should be non-empty are.

See merge request isc-projects/bind9!1321

Ensure base64/base32/hex fields in DNS records that should be non-empty are.

(cherry picked from commit 5e8b772ad15f8e6d26c6b3d8d243ad9d59b58d29)

Merge branch '804-large-nsec3-responses-cause-failure-in-adding-records-to-ncache-and-eventually-formerr-instead-of-nxdomain-v9_11' into 'v9_11'

Resolve "Large NSEC3 responses cause failure in adding records to ncache and, eventually, FORMERR (instead of NXDOMAIN)"

See merge request isc-projects/bind9!1316

add CHANGES entry

(cherry picked from commit 3328116b572605880cd54e84a0b320ff20581370)

allow for up 100 records or 64K of data to be in a ncache entry

(cherry picked from commit 604889e6274d80845ffce529a4247818c3c4b50d)

Merge branch '804-large-nsec3-responses-cause-failure-in-adding-records-to-ncache-and-eventually-formerr-instead-of-nxdomain-2-v9_11' into 'v9_11'

Resolve "Large NSEC3 responses cause failure in adding records to ncache and, eventually, FORMERR (instead of NXDOMAIN)"

See merge request isc-projects/bind9!1315

add CHANGES

(cherry picked from commit 188c1843810fe0953f0332db5339cb8b946a966c)

explictly convert ISC_R_NOSPACE from dns_message_parse to DNS_R_FORMERR and remove from dns_result_torcode

(cherry picked from commit 0c42a9c0aba9a791a8ec7ec835c979634b5bc2ab)

Merge branch '812-cookie-test-failed-v9_11' into 'v9_11'

Resolve "cookie test failed."

See merge request isc-projects/bind9!1313

wait longer for dump to complete

(cherry picked from commit 8a8d378deff2b8e38ca3213b92f7d3558c5b2066)

Merge branch 'ondrej/disable-CI-in-release-branches-v9_11' into 'v9_11'

(v9_11) Run the regular pipelines only for merge requests, and run only the Debian sid...

See merge request isc-projects/bind9!1311

Run the regular pipelines only for merge requests, and run only the Debian sid build for release branches

(cherry picked from commit 24961f606882939b62f461f843933da9c2d44ea8)