Report non-effective primaries

When named is started with -4 or -6 and the primaries for a zone
do not have an IPv4 or IPv6 address respectively issue a log message.

(cherry picked from commit 2cd43032498752d2ab4527475ba21beee133b30b)

Zone transfers should honour -4 and -6 options

Check if the address family has been disabled when transferring
zones.

(cherry picked from commit ecdde04e63277e2bed8d1d861470de80ab0f49e1)

Add helper function isc_sockaddr_disabled

(cherry picked from commit 9be1873ef37a4f20e01dc2ad1e64112b7104d942)

Merge branch '4706-4707-add-more-label-checks-to-dangerfile.py-9.18' into 'bind-9.18'

[9.18] Add more label checks to dangerfile.py

See merge request isc-projects/bind9!9086

Fail for merge requests with "Affects v9.x" labels

Setting "Affects v9.x" labels on a merge request duplicates information
already present on the GitLab issue associated with that merge request.
For trivial merge requests that are not associated with any GitLab
issue, setting the "Affects v9.x" label(s) is considered unnecessary.
Trigger a failure for every merge request marked with at least one
"Affects v9.x" label.

(cherry picked from commit aae51cf28d4466907e152591c8a00525f7ec7038)

Warn about auto-generated merge request titles

Merge request titles auto-generated by GitLab are often a source of
confusion regarding the actual contents of a given merge request.  Warn
for merge requests containing titles that look like auto-generated ones.

(cherry picked from commit 80ec57f198e5706e653ebe2016f41dc6f856db8c)

Fail for branches using old-style version suffixes

Using "-v9_x" and "-v9.x" version suffixes for branch names is now
deprecated since some automation logic does not handle these.  Fail for
any merge request using such old-style version suffixes.

(cherry picked from commit 09851e62309f55bb3e6e3bc7299db95929827f76)

Fail for backports with "Affects v9.x" labels set

Backports are not expected to have any "Affects v9.x" labels set since
those are only meant to be set for merge requests that should have
backports created for them.

(cherry picked from commit 9359cb9c99454626b65b3863e10e710a329ca99c)

Merge branch '4708-uv-tcp-close-reset-issue-9.18' into 'bind-9.18'

[9.18] Deal with uv_tcp_close_reset more gracefully

See merge request isc-projects/bind9!9084

Add CHANGES entry for [GL #4708]

(cherry picked from commit 113968f3a04a236fbf5a7f97c1e23154ca72662f)

Call reset_shutdown if uv_tcp_close_reset failed

If uv_tcp_close_reset() returns an error code, this means the
reset_shutdown callback has not been issued, so do it now.

(cherry picked from commit c40e5c865313c8d538650a8bab71fc4c576705c2)

Do not runtime check uv_tcp_close_reset

When we reset a TCP connection by sending a RST packet, do not bother
requiring the result is a success code.

(cherry picked from commit 5b94bb21295560aed8b3f1968fbfae261c57aab9)

Merge branch '4742-tcp4clients-statistic-is-inaccurate' into 'bind-9.18'

Increase TCP4Clients/TCP6Clients after point of no failure

See merge request isc-projects/bind9!9066

Document TCP4Clients/TCP6Clients

Commit abc47f5ce4a50ab2d3b23505914e9c65f856262b added two new statistics
counters without documenting them.  Add the missing counter descriptions
to the ARM.

add CHANGES and release note for GL #4425

increase TCP4Clients/TCP6Clients after point of no failure

Failing to accept TCP/TLS connections in 9.18 detaches the quota in
isc__nm_failed_accept_cb, causing TCP4Clients and TCP6Clients statistics
to not decrease inside cleanup.

Fix by increasing the counter after the point of no failure but before
handling statistics through the client's socket is no longer valid.

Merge branch '4750-set-loop-on-incoming-transfer-9.18' into 'bind-9.18'

[9.18] Create the new database for AXFR from the dns_zone API

See merge request isc-projects/bind9!9075

Create the new database for AXFR from the dns_zone API

The `axfr_makedb()` didn't set the loop on the newly created database,
effectively killing delayed cleaning on such database.  Move the
database creation into dns_zone API that knows all the gory details of
creating new database suitable for the zone.

(cherry picked from commit 3310cac2b0b83e20a8ce2da67d3e5ede9bee4c)

Merge branch 'nicki/use-bind-team-gitlab-tokens-9.18' into 'bind-9.18'

[9.18] Use bind-team tokens for CI gitlab access

See merge request isc-projects/bind9!9068

Use BIND_TEAM_WRITE_TOKEN for autobackport bot

This is done to unify the token usage across our repositories to
simplify token maintenance.

(cherry picked from commit d391f0c9586c619a2ceead562327ebad8840e69d)

Use BIND_TEAM_API_TOKEN for danger bot

This is done to unify the token usage across our repositories to
simplify token maintenance.

(cherry picked from commit 6767172bc0fc2f7fbb44c24144ea40c0836b9913)

Merge branch 'mnowak/alpine-3.20-9.18' into 'bind-9.18'

[9.18] Add Alpine Linux 3.20

See merge request isc-projects/bind9!9060

Add Alpine Linux 3.20

(cherry picked from commit 6f22e79ef99c83498922fccb1c9e5547828f536d)

Merge branch 'mnowak/short-stress-tests-should-keep-artifacts-9.18' into 'bind-9.18'

[9.18] Short stress tests should keep artifacts

See merge request isc-projects/bind9!9057

Short stress tests should keep artifacts

(cherry picked from commit aa72747c808ff55a53d724c7786032a7b522f904)

Merge branch '4730-clang-format-header-file-changes-bind-9.18' into 'bind-9.18'

[9.18] Resolve "Clang format header file changes"

See merge request isc-projects/bind9!9053

Ignore reformat sources with up-to-date clang-format-18

Clang-format header file changes

Merge branch 'aram/isc_task_purgeevent-race-fix' into 'bind-9.18'

Fix a data race in isc_task_purgeevent()

See merge request isc-projects/bind9!8937

Add a CHANGES note for [GL !8937]

Fix a data race in isc_task_purgeevent()

When isc_task_purgeevent() is called for and 'event', the event, in
the meanwhile, could in theory get processed, unlinked, and freed.
So when the function then operates on the 'event', it causes a
segmentation fault.

The only place where isc_task_purgeevent() is called is from
timer_purge().

In order to resolve the data race, call isc_task_purgeevent() inside
the 'timer->lock' locked block, so that timerevent_destroy() won't
be able to destroy the event if it was processed in the meanwhile,
before isc_task_purgeevent() had a chance to purge it.

In order to be able to do that, move the responsibility of calling
isc_event_free() (upon a successful purge) out from the
isc_task_purgeevent() function to its caller instead, so that it can
be called outside of the timer->lock locked block.

Test a race condition between isc_timer_purge() and isc_event_free()

Let basic_tick() of 'task1' and 'basic_quick' of 'task4' run in
different threads, and insert an artificial delay in timer_purge()
to cause an existing race condition to appear.

Expose internal timer_purge() as isc_timer_purge()

This function is used in a unit test to check for data races.

Merge branch '4726-increase-formerr-coverage-in-message-parsing-9.18' into 'bind-9.18'

[9.18] Resolve "Increase FORMERR coverage in message parsing"

See merge request isc-projects/bind9!9048

add test cases for several FORMERR code paths:

  - duplicated question
  - duplicated answer
  - qtype as an answer
  - two question types
  - question names
  - nsec3 bad owner name
  - short record
  - short question
  - mismatching question class
  - bad record owner name
  - mismatched class in record
  - mismatched KEY class
  - OPT wrong owner name
  - invalid RRSIG "covers" type
  - UPDATE malformed delete type
  - TSIG wrong class
  - TSIG not the last record

(cherry picked from commit 6e9ed4983eb6ed7c018f23e1b7cd8c46f9104fc0)

Merge tag 'v9.18.27' into bind-9.18

Merge branch '4517-dnssec-verify-reports-errors-in-nsec3-chain-9.18' into 'bind-9.18'

[9.18] Resolve "dnssec-verify reports errors in NSEC3 chain"

See merge request isc-projects/bind9!9041

Add CHANGES note for [GL #4517]

(cherry picked from commit e4dbf4be8e6c305687bb7f2cc9035979a8f9d108)

Don't sign non-apex DNSKEY records

DNSKEY can only be validated if it is signed by itself.  Stop
attempting to sign non apex DNSKEY RRsets.

(cherry picked from commit dd13f41ae193e9f597ac4d18cfb4daf64714907a)

Remove invalid DNSKEY RRset from zone

(cherry picked from commit 315ad2df7adc7ff36ed435490fec067f11680b31)

Add regression test data for [GL #4517]

An obscured DNSKEY RRset at a delegation was incorrectly added to
the NSEC/NSEC3 type bit map leading to zone verification failures.
This adds such a RRset to the test zone.

(cherry picked from commit 122111f75e1c3d76b8620e3c3009be2a78e55424)

Fail if there are non apex DNSKEYs

DNSSEC only works when DNSKEYs are self signed.  This only occurs
when the DNSKEY RRset is at the apex.  Cause dnssec-signzone to
fail if it attempts to sign an non-apex DNSKEY RRset.

(cherry picked from commit b3efc15be429d940a98baa4715959071e2581502)

Properly build the NSEC/NSEC3 type bit map

DNSKEY was incorrectly being added to the NESC/NSEC3 type bit map
when it was obscured by the delegation.  This lead to zone verification
failures.

(cherry picked from commit ec3c6248148dec0696eeab62abdf53ea71116c0b)

Properly update 'maxtype'

'maxtype' should be checked to see if it should be updated whenever
a type is added to the type map.

(cherry picked from commit e84615629f52f551c7482036e4a333498fc5f089)

Merge branch '4719-prevent-passing-null-to-dns_dispatch_resume' into 'bind-9.18'

Prevent passing NULL to dns_dispatch_resume()

See merge request isc-projects/bind9!9035

Add release note for GL #4719

Add CHANGES for GL #4719

Prevent passing NULL to dns_dispatch_resume()

If a query sent using the dns_request API times out when the view it was
associated with gets torn down, the dns_dispatch_resume() call in
req_response() may be issued with the 'resp' argument set to NULL,
triggering an assertion failure.  Consider the following scenario ([A]
and [B] are thread identifiers):

 1. [A] Read timeout for a dispatch query fires.

 2. [A] udp_recv() is called.  It locks the dispatch, determines it
    timed out, prepares for calling the higher-level callback with
    ISC_R_TIMEDOUT, and unlocks the dispatch (lib/dns/dispatch.c:633).

 3. [B] The last reference to a view is released.
    dns_requestmgr_shutdown() is called, canceling all in-flight
    requests for that view.  (Note that udp_recv() in thread [A] already
    unlocked the dispatch, so its state can be modified.)  As a part of
    this process, request_cancel() calls dns_dispatch_done() on
    request->dispentry, setting it to NULL.

 4. [A] udp_recv() calls the higher-level callback (req_response()) with
    ISC_R_TIMEDOUT.

 5. [A] Since the request timed out, req_response() retries sending it.
    In the process, it calls dns_dispatch_resume(), passing
    request->dispentry as the 'resp' argument.

 6. [A] Since 'resp' is NULL, the REQUIRE(VALID_RESPONSE(resp));
    assertion in dns_dispatch_resume() fails.

Fix by checking whether the request has been canceled before calling
dns_dispatch_resume(), similarly to how it is done in req_connected()
and req_senddone().

Merge branch '4635-enhance-xferquota-zone-handling' into 'bind-9.18'

Enhance xferquota zone handling

See merge request isc-projects/bind9!9039

Address qp/rbtdb backup file style differences

qp and rbtdb produce stylistically different backup files.  This
was causing the xferquota system test to fail.  This has been
addressed by making the test independent of the stylistic differences.

(cherry picked from commit 1482e9bbb9f828fda0f0171823ce6c192b64a195)

Fix xferquota system test

The change from RBT to QP has changed the contents of generated zone
files slightly: node names are now always absolute, so instead of using
$ORIGIN and relative names, generated zone files use full names for all
records.

This caused a failure in the xferquota system test, which was looking
for a relative name in secondary zone files. Replace the string
matching with a regular expression to fix the test.

(cherry picked from commit 618c963cb75b3677d5d2e4ff9059a965f09fcb13)

Merge branch 'nicki/add-mailmap-9.18' into 'bind-9.18'

[9.18] Add .mailmap to the git repo

See merge request isc-projects/bind9!9030

Add .mailmap to the git repo

Mailmap can be used to unify the display name in standard git tools.

(cherry picked from commit 986013a7083cc17335107a11ba3b669d91bf6744)

Merge branch 'mnowak/pytest_rewrite_include-multiplecfg-9.18' into 'bind-9.18'

[9.18] Rewrite include-multiplecfg system test to pytest

See merge request isc-projects/bind9!9029

Rewrite include-multiplecfg system test to pytest

(cherry picked from commit e9e87533061d0e5ca1e976606e16ef9c4b276339)

Use isctest.run.cmd() helper function in tests

(cherry picked from commit 77a42f887549bf4c8f0a213a939a266201c59e76)

Add isctest.run.cmd() helper function

(cherry picked from commit a9f3fb3db5b473b7223bf7699e42b0371808dd6a)

Merge branch '4502-missing-reference-9.18' into 'bind-9.18'

[9.18] Resolve "Missing reference?"

See merge request isc-projects/bind9!9024

Add CHANGES for [GL #4502]

(cherry picked from commit 53698ad58d5c8c908f0f8a55bdd05a2387962a3b)

Use dns_view_findzone instead of dns_zt_find

With weak zone attachments being used for catzs, catzs->view->zonetable
may be NULL so we need to account for this which dns_view_findzone
does.  This is already done in main.

catz should use weak attachment to the view

(cherry picked from commit 941ad2b3c526defc416d35e42451fcfec70520c2)

catzs->view should maintain a view reference

Use dns_view_weakattach and dns_view_weakdetach to maintain a
reference to the view referenced through catzs->view.

(cherry picked from commit 307e3ed9a66a2a90f38c59f7854529ffb76d93f8)

Merge branch 'pspacek/doc-rfc-ref-update-9.18' into 'bind-9.18'

[9.18] Update RFC references in documentation

See merge request isc-projects/bind9!9021

Update reference for Catalog Zones spec to RFC9432

(cherry picked from commit ead4a110ad05faa806841af53c2ea8dc36e5e9c9)

Update KSK root sentinel references

The mechanism was published as RFC 8509. I've briefly looked at diff
between versions -08 and the RFC and did not find significant protocol
change. Quick manual check confirms what we seem to comply with the
published protocol.

(cherry picked from commit 153311da2d97f1febd7e952842fd7e4cf8f9b673)

Mention RFC 9276 Guidance for NSEC3 Parameter Settings

Draft was eventually published as RFC 9276 but we did not update our
docs. Also add couple mentions in relevant places in the ARM and
dnssec-signzone man page, mainly around "do not touch" places.

(cherry picked from commit 8e4c0329c3a61239e023926a73591029168ea7a3)

Update DNSSEC Operational Practices references to Version 2

RFC 4641 was obsoleted by 6781.

(cherry picked from commit c5d6769e11cfbce258017f13a8164005644747ff)

Merge branch '4704-svcb-https-bind9-and-dig-fail-to-parse-packet-if-zero-svcpriority-and-existing-svcparams-9.18' into 'bind-9.18'

[9.18] Resolve "SVCB/HTTPS: Bind9 and Dig fail to parse packet if zero SvcPriority and existing SvcParams"

See merge request isc-projects/bind9!9019

Add CHANGES note for [GL #4704]

(cherry picked from commit e0b5c85f5a3a03a5541a6fc1fe171b0bd56c2f56)

Only check SVBC alias forms at higher levels

Allow SVBC (HTTPS) alias form with parameters to be accepted from
the wire and when transfered.  This is for possible future extensions.

(cherry picked from commit 799046929cb1f23cfec7ae7391e5408e01baface)

Merge branch '4357-treat-notfile-as-eof-9.18' into 'bind-9.18'

[9.18] Remove infinite loop on ISC_R_NOFILE

See merge request isc-projects/bind9!9018

Add CHANGES note for [GL #4357]

(cherry picked from commit 236a38a7c90cfdd0fdc4afc9c26426622faa2614)

Test including a directory in a zone file

(cherry picked from commit e697d20f008ed1c77d59317e4b297a09d951669c)

Remove infinite loop on ISC_R_NOFILE

When parsing a zonefile named-checkzone (and others) could loop
infinitely if a directory was $INCLUDED.  Record the error and treat
as EOF when looking for multiple errors.

This was found by Eric Sesterhenn from X41.

(cherry picked from commit efd27bb82d89ad0ff0e52b93b30d0ef2cc5e2d8e)

Merge branch '4353-fix-nibbles-9.18' into 'bind-9.18'

[9.18] Address infinite loop when processing $GENERATE

See merge request isc-projects/bind9!9017

Add CHANGES note for [GL #4353]

(cherry picked from commit f154187fd05dcc255d6fc0af3c9b3d0c686f1d36)

Test $GENERATE in nibble mode with a negative value

Negative values used to cause $GENERATE to loop forever.

(cherry picked from commit 32535de856db26e70b657ccf17a22c4e891672e6)

Address infinite loop when processing $GENERATE

In nibble mode if the value to be converted was negative the parser
would loop forever.  Process the value as an unsigned int instead
of as an int to prevent sign extension when shifting.

This was found by Eric Sesterhenn from X41.

(cherry picked from commit 371824f0789d6e491216f266bf62955a73b49858)

Merge branch 'mnowak/fedora-40-9.18' into 'bind-9.18'

[9.18] Add Fedora 40

See merge request isc-projects/bind9!9015

Add Fedora 40

(cherry picked from commit 1a09f822addaafe4512ae3a18d4697346310df09)

Merge branch 'mnowak/ubuntu-24.04-noble-numbat-9.18' into 'bind-9.18'

[9.18] Add Ubuntu 24.04 LTS (Noble Numbat)

See merge request isc-projects/bind9!9013

Configure Ubuntu "focal" with --disable-doh

At least one CI job should be configured with disabled DoH to catch
build problems.

Add Ubuntu 24.04 LTS (Noble Numbat)

(cherry picked from commit d02ed1a0ebd7957127a843279a331e44e8412700)

Merge branch 'mnowak/openbsd-7.5-9.18' into 'bind-9.18'

[9.18] Add OpenBSD 7.5

See merge request isc-projects/bind9!9011

Add OpenBSD 7.5

(cherry picked from commit 102fccdcba3105e2801652a3636009cf62290903)

Merge branch '4711-stress-test-finer-runtime-granularity-9.18' into 'bind-9.18'

[9.18] Add short version of "stress" tests to MR-triggered pipelines

See merge request isc-projects/bind9!9009

Add short "stress" test jobs

(cherry picked from commit f33b746353116b581fc6c4f0b5d58aeea17da0be)

Rename "stress" tests to long "stress" tests

(cherry picked from commit 58a1a50e76bf36c7d3675a3734d7ee1f0627e735)

Revert "Drop parallel build from stress tests"

This reverts commit 3fd7e7c81f81e21adebde2930fe511fe335f825e.

In AWS this is no longer a concern.

(cherry picked from commit 1d77a3b3c61c310c8b104aa2a68fcfcf53b3e083)

Merge branch '3323-different-rpz-soa-ttl-9.18' into 'bind-9.18'

[9.18] RPZ response's SOA record is incorrectly set to 1

See merge request isc-projects/bind9!9008

Add release note and CHANGES for #3323

(cherry picked from commit 2726b569b1042a4fe2a7dde1a4fdd0af21ed8e6b)

RPZ response's SOA record is incorrectly set to 1

An RPZ response's SOA record TTL is set to 1 instead of the SOA TTL,
a boolean value is passed on to query_addsoa, which is supposed to be
a TTL value. I don't see what value is appropriate to be used for
overriding, so we will pass UINT32_MAX.

(cherry picked from commit 5d7e613e8103b81cf47a22c72310a38737fbe3c6)

Merge branch 'mnowak/set-up-version-and-release-notes-for-bind-9.18.28' into 'bind-9.18'

Set up version and release notes for BIND 9.18.28

See merge request isc-projects/bind9!9007

Set up release notes for BIND 9.18.28

Update BIND version to 9.18.28-dev

Update BIND version for release

Add a CHANGES marker

Merge branch 'mnowak/prepare-documentation-for-bind-9.18.27' into 'v9.18.27-release'

Prepare documentation for BIND 9.18.27

See merge request isc-private/bind9!685

Tweak and reword release notes

Prepare release notes for BIND 9.18.27