]> git.ipfire.org Git - thirdparty/bind9.git/log
thirdparty/bind9.git
12 hours agonew: test: Add pyyaml to system test requirements main
Nicki Křížek [Tue, 28 Jul 2026 13:10:21 +0000 (15:10 +0200)] 
new: test: Add pyyaml to system test requirements

Needed for: https://gitlab.isc.org/isc-projects/bind9/-/merge_requests/12446

Merge branch 'nicki/requie-pyyaml' into 'main'

See merge request isc-projects/bind9!12468

12 hours agoAdd pyyaml to system test requirements 12468/head
Nicki Křížek [Tue, 28 Jul 2026 13:06:37 +0000 (15:06 +0200)] 
Add pyyaml to system test requirements

14 hours agorem: ci: Drop FreeBSD 13 support
Michal Nowak [Tue, 28 Jul 2026 10:38:21 +0000 (12:38 +0200)] 
rem: ci: Drop FreeBSD 13 support

FreeBSD 13 has reached its end of life, so stop testing on it: remove
the CI build, system test, and unit test jobs together with the runner
tag anchor they used, and move the release from the regularly tested
platforms to the best-effort list in the ARM.

The is_host_freebsd_13 helper existed solely to grant one rerun to two
tests that were flaky on that release, so it goes away with them.

Assisted-by: Claude:claude-opus-5[1m]
Merge branch 'mnowak/drop-freebsd-13' into 'main'

See merge request isc-projects/bind9!12464

14 hours agoDrop FreeBSD 13 support 12464/head
Michal Nowak [Tue, 28 Jul 2026 08:47:41 +0000 (10:47 +0200)] 
Drop FreeBSD 13 support

FreeBSD 13 has reached its end of life, so stop testing on it: remove
the CI build, system test, and unit test jobs together with the runner
tag anchor they used, and move the release from the regularly tested
platforms to the best-effort list in the ARM.

The is_host_freebsd_13 helper existed solely to grant one rerun to two
tests that were flaky on that release, so it goes away with them.

Assisted-by: Claude:claude-opus-5[1m]
15 hours agochg: dev: Use sanitizer.contains() instead of the "in" operator
Michal Nowak [Tue, 28 Jul 2026 09:52:15 +0000 (11:52 +0200)] 
chg: dev: Use sanitizer.contains() instead of the "in" operator

muon's static analyzer aborts on the four "'address' in sanitizer"
expressions because it cannot typecheck the "in" operator against the
complex combo type it infers for b_sanitize, which makes "muon analyze"
unusable on the whole tree.  Reported upstream as
https://github.com/muon-build/muon/issues/289.

Assisted-by: Claude:claude-fable-5
Merge branch 'mnowak/meson-sanitizer-contains' into 'main'

See merge request isc-projects/bind9!12459

34 hours agoUse sanitizer.contains() instead of the "in" operator 12459/head
Michal Nowak [Mon, 27 Jul 2026 14:44:40 +0000 (16:44 +0200)] 
Use sanitizer.contains() instead of the "in" operator

muon's static analyzer aborts on the four "'address' in sanitizer"
expressions because it cannot typecheck the "in" operator against the
complex combo type it infers for b_sanitize, which makes "muon analyze"
unusable on the whole tree.  Reported upstream as
https://github.com/muon-build/muon/issues/289.

Assisted-by: Claude:claude-fable-5
39 hours agochg: dev: use project lints as tests in meson
Aydın Mercan [Mon, 27 Jul 2026 10:29:10 +0000 (13:29 +0300)] 
chg: dev: use  project lints as tests in meson

The check scripts run in the CI job `misc` is also useful in development
enough that it warrants an easy way to execute them all at once.

Add these scripts as unit tests with the suite `lint`.

Merge branch 'aydin/meson-misc-checks' into 'main'

See merge request isc-projects/bind9!12437

41 hours agouse the lint test suite in misc job 12437/head
Aydın Mercan [Wed, 22 Jul 2026 15:08:49 +0000 (18:08 +0300)] 
use the lint test suite in misc job

The `misc` job in CI already needs to have a build directory configured
when checking for rdata registration. Thus, it is safe to transition the
CI job to use the `lint` test suite instead.

41 hours agoadd various project lints as tests to meson
Aydın Mercan [Fri, 10 Jul 2026 09:10:05 +0000 (12:10 +0300)] 
add various project lints as tests to meson

The check scripts run in the CI job `misc` is also useful in development
enough that it warrants an easy way to execute them all at once.

Add these scripts as unit tests with the suite `lint`.

44 hours agofix: usr: Restore SMF support on Solaris and illumos
Mark Andrews [Mon, 27 Jul 2026 05:07:21 +0000 (15:07 +1000)] 
fix: usr: Restore SMF support on Solaris and illumos

SMF support on Solaris and illumos was silently dropped by a build
system rewrite in 2018; it is now detected and enabled again.

Closes #6096

Merge branch '6096-named-smf-add-message-prototype-mismatch' into 'main'

See merge request isc-projects/bind9!12369

45 hours agoAdd missing putnull() to named_smf_add_message() 12369/head
Michal Nowak [Thu, 23 Jul 2026 09:18:34 +0000 (11:18 +0200)] 
Add missing putnull() to named_smf_add_message()

The control channel response text needs to be NUL-terminated.
named_smf_add_message() did not do that, so 'rndc stop' or
'rndc halt' on an SMF-managed named running chrooted could echo
uninitialized heap contents after the 'use svcadm(1M) to manage
named' message.

Assisted-by: Claude:claude-fable-5
45 hours agoRestore SMF support in the build system
Michal Nowak [Thu, 23 Jul 2026 09:18:34 +0000 (11:18 +0200)] 
Restore SMF support in the build system

libscf detection (AC_CHECK_LIB(scf, smf_enable_instance)) was lost
in the automake rewrite (978c7b2e89aa) and was not added back when
the build system was replaced with meson, so HAVE_LIBSCF was never
defined and the Solaris SMF code in bin/named was dead code.

Detect libscf on SunOS hosts and link named against it.

Assisted-by: Claude:claude-fable-5
45 hours agoFix named_smf_add_message prototype
Mark Andrews [Wed, 8 Jul 2026 13:53:09 +0000 (23:53 +1000)] 
Fix named_smf_add_message prototype

There was a prototype mismatch for named_smf_add_message.  This
has been fixed.

4 days agofix: test: Drop a redundant racy check in fetchlimit test
Nicki Křížek [Fri, 24 Jul 2026 10:44:11 +0000 (12:44 +0200)] 
fix: test: Drop a redundant racy check in fetchlimit test

This particular `fetchlimit` failure has been happening quite often since https://gitlab.isc.org/isc-projects/bind9/-/merge_requests/12124 removed the test re-runs with `flaky` (after fixing other instabilities inside the test).

Example failure: https://gitlab.isc.org/isc-projects/bind9/-/jobs/7780648

Merge branch 'nicki/fix-fetchlimit-recovery-quota-flaky' into 'main'

See merge request isc-projects/bind9!12393

4 days agoDrop a redundant racy check in fetchlimit test 12393/head
Nicki Křížek [Thu, 9 Jul 2026 09:52:45 +0000 (09:52 +0000)] 
Drop a redundant racy check in fetchlimit test

The "lame server recovery" step required the adaptive per-server quota
for the lame server to have dropped further after it started answering
again. That decrease is not reliable: the quota only drops while the
timeout ratio (atr) exceeds the high watermark, and once the server
answers atr falls, so a further drop cannot be relied upon.

Remove the check to improve the fetchlimit test stability in slower
platforms in CI. The quota at the start of the recovery is still
recorded -- it is checked later on that it is increased once the server
recovers.

Assisted-by: Claude:claude-opus-4-8
5 days agofix: usr: Prevent ddns-confgen self-domain update-policy injection
Mark Andrews [Fri, 24 Jul 2026 00:33:18 +0000 (10:33 +1000)] 
fix: usr: Prevent ddns-confgen self-domain update-policy injection

In `ddns-confgen`, the user-supplied domain name to be updated was not properly escaped and put inside a double-quoted string when generating the example code fragment.  This has been corrected.

Closes #6072

Merge branch '6072-ddns-confgen-self-domain-update-policy-injection' into 'main'

See merge request isc-projects/bind9!12409

5 days agoPrevent ddns-confgen self-domain update-policy injection 12409/head
Mark Andrews [Wed, 15 Jul 2026 07:53:22 +0000 (17:53 +1000)] 
Prevent ddns-confgen self-domain update-policy injection

In ddns-confgen, the user-supplied domain name to be updated
was not properly escaped and put inside a double quoted string
when generating the example code fragment.  This has been corrected.

5 days agofix: dev: Restore arc4random() detection dropped in the v9.21.14 merge
Michal Nowak [Thu, 23 Jul 2026 16:24:10 +0000 (18:24 +0200)] 
fix: dev: Restore arc4random() detection dropped in the v9.21.14 merge

Commit 4db9e5d90e2 ("Use arc4random for CSPRNG when available", part
of the CVE-2025-40780 fix) guarded the arc4random() code paths in
lib/isc/random.h and lib/isc/random.c with HAVE_ARC4RANDOM and added
the corresponding function check to meson.build.  The manual conflict
resolution in merge c2a672bbaef ("Merge tag 'v9.21.14'") kept the code
changes but dropped the meson.build hunk, so HAVE_ARC4RANDOM was never
defined and platforms with arc4random() (macOS and the BSDs) silently
fell back to the internal ChaCha-based CSPRNG.  Restore the check.

Assisted-by: Claude:claude-fable-5
Merge branch 'mnowak/restore-arc4random-meson-check' into 'main'

See merge request isc-projects/bind9!12451

5 days agoRestore arc4random() detection dropped in the v9.21.14 merge 12451/head
Michal Nowak [Thu, 23 Jul 2026 14:47:31 +0000 (16:47 +0200)] 
Restore arc4random() detection dropped in the v9.21.14 merge

Commit 4db9e5d90e2 ("Use arc4random for CSPRNG when available", part
of the CVE-2025-40780 fix) guarded the arc4random() code paths in
lib/isc/random.h and lib/isc/random.c with HAVE_ARC4RANDOM and added
the corresponding function check to meson.build.  The manual conflict
resolution in merge c2a672bbaef ("Merge tag 'v9.21.14'") kept the code
changes but dropped the meson.build hunk, so HAVE_ARC4RANDOM was never
defined and platforms with arc4random() (macOS and the BSDs) silently
fell back to the internal ChaCha-based CSPRNG.  Restore the check.

Assisted-by: Claude:claude-fable-5
5 days agofix: nil: Sync root hints with primary source
Nicki Křížek [Thu, 23 Jul 2026 16:08:55 +0000 (18:08 +0200)] 
fix: nil: Sync root hints with primary source

Should match https://www.internic.net/zones/named.root

Assisted-by: Claude:claude-fable-5
Merge branch 'nicki/sync-root-hints' into 'main'

See merge request isc-projects/bind9!12445

5 days agoSync root hints with primary source 12445/head
Nicki Křížek [Thu, 23 Jul 2026 08:16:41 +0000 (08:16 +0000)] 
Sync root hints with primary source

Should match https://www.internic.net/zones/named.root

Assisted-by: Claude:claude-fable-5
5 days agofix: ci: Make the scan-build CI job actually run the static analyzer
Michal Nowak [Thu, 23 Jul 2026 12:24:19 +0000 (14:24 +0200)] 
fix: ci: Make the scan-build CI job actually run the static analyzer

scan-build was a silent no-op: the native file pinned CC to a name ccc-analyzer never intercepts.  Run meson and ninja under scan-build with --use-cc, drop the native file and LTO, and gate on all default checkers via --status-bugs.

Merge branch '6165-scan-build-actually-run-analyzer' into 'main'

See merge request isc-projects/bind9!12298

5 days agoSilence scan-build leak false positive in isccc_cc_checkdup() 12298/head
Michal Nowak [Mon, 29 Jun 2026 12:41:03 +0000 (12:41 +0000)] 
Silence scan-build leak false positive in isccc_cc_checkdup()

On success the symtab owns 'key' and frees it via symtab_undefine(), but
scan-build cannot see that ownership pass through the const key
parameter of isc_symtab_define() and reports a leak.  free(key) under
__clang_analyzer__ only, which placates the checker without affecting
real builds:

    lib/isccc/cc.c:1015:9: warning: Potential leak of memory pointed to by 'key' [unix.Malloc]

Assisted-by: Claude:claude-opus-4-8
5 days agoInitialize rrbuffer in towire_answer()
Michal Nowak [Mon, 29 Jun 2026 11:42:13 +0000 (11:42 +0000)] 
Initialize rrbuffer in towire_answer()

Initialize rrbuffer from *target, like the adjacent savedbuffer, to
silence a scan-build uninitialized-read false positive:

    lib/dns/rdataset.c:423:3: warning: 2nd function call argument is an uninitialized value [core.CallAndMessage]

Assisted-by: Claude:claude-opus-4-8
5 days agoAssert current_version in qpzone_destroy()
Michal Nowak [Mon, 29 Jun 2026 11:41:57 +0000 (11:41 +0000)] 
Assert current_version in qpzone_destroy()

current_version is a lifetime invariant and never NULL.  Assert it to
silence a scan-build false positive:

    lib/dns/qpzone.c:593:2: warning: Dereference of null pointer [core.NullDereference]

Assisted-by: Claude:claude-opus-4-8
5 days agoAssert rpz_ready() yields an rdataset in rpz_find_p()
Michal Nowak [Mon, 29 Jun 2026 11:41:37 +0000 (11:41 +0000)] 
Assert rpz_ready() yields an rdataset in rpz_find_p()

A successful rpz_ready() always leaves *rdatasetp non-NULL.  Assert it
to silence a scan-build false positive:

    lib/ns/query.c:3136:7: warning: Access to field 'type' results in a dereference of a null pointer [core.NullDereference]

Assisted-by: Claude:claude-opus-4-8
5 days agoAssert rdataset precondition in query_addrrset()
Michal Nowak [Mon, 29 Jun 2026 11:41:20 +0000 (11:41 +0000)] 
Assert rdataset precondition in query_addrrset()

rdataset is always non-NULL and is dereferenced unconditionally below.
Make the precondition explicit to silence a scan-build false positive:

    lib/ns/query.c:2254:11: warning: Access to field 'type' results in a dereference of a null pointer (loaded from variable 'rdataset') [core.NullDereference]

Assisted-by: Claude:claude-opus-4-8
5 days agoAssert fd validity in start_tcp_child()
Michal Nowak [Mon, 29 Jun 2026 11:41:09 +0000 (11:41 +0000)] 
Assert fd validity in start_tcp_child()

load_balance_sockets is fixed at netmgr init, so dup(fd) runs only when
the caller supplied a valid descriptor.  Assert it to silence a
scan-build false positive:

    lib/isc/netmgr/tcp.c:468:15: warning: The 1st argument to 'dup' is -1 but should be >= 0 [unix.StdCLibraryFunctions]

Assisted-by: Claude:claude-opus-4-8
5 days agoAssert fd validity in start_udp_child()
Michal Nowak [Mon, 29 Jun 2026 11:40:50 +0000 (11:40 +0000)] 
Assert fd validity in start_udp_child()

load_balance_sockets is fixed at netmgr init, so dup(fd) runs only when
the caller supplied a valid descriptor.  Assert it to silence a
scan-build false positive:

    lib/isc/netmgr/udp.c:194:15: warning: The 1st argument to 'dup' is -1 but should be >= 0 [unix.StdCLibraryFunctions]

Assisted-by: Claude:claude-opus-4-8
5 days agoRemove dead stores reported by scan-build
Michal Nowak [Mon, 22 Jun 2026 19:24:33 +0000 (21:24 +0200)] 
Remove dead stores reported by scan-build

The syncplugin test driver overwrote two dns_name_fromstring() results
without checking them; wrap the calls in CHECK():

    bin/tests/system/hooks/driver/test-syncplugin.c:156:3: warning: Value stored to 'result' is never read [deadcode.DeadStores]
    bin/tests/system/hooks/driver/test-syncplugin.c:158:3: warning: Value stored to 'result' is never read [deadcode.DeadStores]

Assisted-by: Claude:claude-opus-4-8
5 days agoRun scan-build static analysis in CI
Michal Nowak [Mon, 22 Jun 2026 16:25:16 +0000 (18:25 +0200)] 
Run scan-build static analysis in CI

scan-build was a silent no-op: the native file pinned CC to a name
ccc-analyzer never intercepts.  Run meson and ninja under scan-build
with --use-cc, drop the native file and LTO, and gate on all default
checkers via --status-bugs.

Assisted-by: Claude:claude-opus-4-8
5 days agofix: dev: Use memmove in isc_sockaddr_fromin/isc_sockaddr_fromin6
Arаm Sаrgsyаn [Thu, 23 Jul 2026 10:59:33 +0000 (10:59 +0000)] 
fix: dev: Use memmove in isc_sockaddr_fromin/isc_sockaddr_fromin6

Use memmove instead of direct assignment from the source pointer
because the source pointer is not guaranteed to be correctly
aligned.

Closes #6260

Merge branch '6260-proxyv2-direct-parser-alignment-issue' into 'main'

See merge request isc-projects/bind9!12434

5 days agoAdd a new check in the proxyheader unit test 12434/head
Aram Sargsyan [Wed, 22 Jul 2026 12:20:20 +0000 (12:20 +0000)] 
Add a new check in the proxyheader unit test

Check that isc_proxy2_header_handle_directly() can handle
data that is not aligned on the usual boundary.

5 days agoUse memmove in isc_sockaddr_fromin/isc_sockaddr_fromin6
Aram Sargsyan [Wed, 22 Jul 2026 12:18:59 +0000 (12:18 +0000)] 
Use memmove in isc_sockaddr_fromin/isc_sockaddr_fromin6

Use memmove instead of direct assignment from the source pointer
because the source pointer is not guaranteed to be correctly
aligned.

5 days agonew: test: Mark each test's boundaries in the named instance logs
Nicki Křížek [Thu, 23 Jul 2026 10:58:26 +0000 (12:58 +0200)] 
new: test: Mark each test's boundaries in the named instance logs

Use `rndc null` command to mark each test boundaries in named.run logs.

Since calling the `rndc` binary caused a lot of overhead, import the rndc python interface from [python-rndc](https://gitlab.isc.org/isc-projects/python-rndc/) and adapt it to our use-case.

Merge branch 'nicki/rndc-null-in-system-tests' into 'main'

See merge request isc-projects/bind9!12415

5 days agoMark each test's boundaries in the named instance logs 12415/head
Nicki Křížek [Fri, 10 Jul 2026 11:50:51 +0000 (11:50 +0000)] 
Mark each test's boundaries in the named instance logs

Send an `rndc null` command with a test ID marker at the start and end
of each test function to all named instances.

This makes it easy to identify the precise point where each test was
executed when reading the named logs. This becomes especially useful for
modules which run multiple test functions, since those are executed on
the same named instances.

Assisted-by: Claude:claude-fable-5
5 days agoIntegrate the Python RNDC client into isctest
Nicki Křížek [Tue, 14 Jul 2026 11:42:25 +0000 (11:42 +0000)] 
Integrate the Python RNDC client into isctest

Register the rndc module in the isctest package and provide a
NamedInstance.rndc_client() factory next to rndc(), so tests can grab
a client for an instance's control channel without repeating the
address and port details.

Assisted-by: Claude:claude-fable-5
5 days agoRework the Python RNDC client API around isctest conventions
Nicki Křížek [Tue, 14 Jul 2026 11:41:39 +0000 (11:41 +0000)] 
Rework the Python RNDC client API around isctest conventions

Rename the class to RNDCClient and align its interface with the
existing isctest.run helpers so that the binary and the Python RNDC
interfaces feel the same:

- take separate ip/port arguments and accept commands as str
- default the HMAC algorithm and secret to the _common/rndc.key
  values used by virtually all control channels in the system tests
- call() returns an RNDCResult mirroring CmdResult (rc/out/err) and
  raises RNDCException on a non-zero result code, with the same
  raise_on_exception escape hatch as isctest.run.cmd(); the decoded
  protocol response stays available as RNDCResult.response
- support the context manager protocol and add close(); the socket
  gets a timeout so a stuck server can't hang a test, and a failed
  login closes the socket instead of leaking it
- use sendall() and an exact-read loop to avoid short reads/writes
  (MSG_WAITALL is ineffective on a socket with a timeout, which
  operates in non-blocking mode)
- raise RNDCProtocolError for truncated, malformed, or
  unauthenticated responses instead of overloading OSError
- draw the initial message serial from the full 32-bit space like
  the rndc binary, keeping the odds of tripping named's duplicate
  message suppression negligible

Assisted-by: Claude:claude-fable-5
5 days agoAdd type annotations to rndc.py
Nicki Křížek [Tue, 14 Jul 2026 11:39:50 +0000 (11:39 +0000)] 
Add type annotations to rndc.py

Annotate the module for the mypy CI gate. The locals in _command()
that were reused for values of a different type are split so each name
keeps a single type.

Assisted-by: Claude:claude-fable-5
5 days agoAlign rndc.py naming with project conventions
Nicki Křížek [Tue, 14 Jul 2026 11:38:25 +0000 (11:38 +0000)] 
Align rndc.py naming with project conventions

Rename locals that shadow the hash, input, and type builtins and drop
the double-underscore name mangling from private members in favor of
the single-underscore style used across isctest.

Assisted-by: Claude:claude-fable-5
5 days agoPort rndc.py to Python 3.10 idioms
Nicki Křížek [Tue, 14 Jul 2026 11:37:40 +0000 (11:37 +0000)] 
Port rndc.py to Python 3.10 idioms

Replace percent formatting of str with f-strings and the deprecated
IOError alias with OSError. The bytes interpolation in the protocol
serialization is left alone; f-strings have no bytes counterpart.

Assisted-by: Claude:claude-fable-5
5 days agoSort rndc.py imports with ruff
Nicki Křížek [Tue, 14 Jul 2026 11:36:44 +0000 (11:36 +0000)] 
Sort rndc.py imports with ruff

Assisted-by: Claude:claude-fable-5
5 days agoFormat rndc.py with black
Nicki Křížek [Tue, 14 Jul 2026 11:36:29 +0000 (11:36 +0000)] 
Format rndc.py with black

Assisted-by: Claude:claude-fable-5
5 days agoUse the standard license header in the imported rndc.py
Nicki Křížek [Tue, 14 Jul 2026 11:36:16 +0000 (11:36 +0000)] 
Use the standard license header in the imported rndc.py

Replace the upstream header with the SPDX-tagged header used across the
repository and drop the stale __version__ marker, which has no meaning
for an in-tree copy. The module provenance is recorded in the import
commit.

Assisted-by: Claude:claude-fable-5
5 days agoImport rndc.py from the python-rndc project verbatim
Nicki Křížek [Tue, 14 Jul 2026 08:47:30 +0000 (08:47 +0000)] 
Import rndc.py from the python-rndc project verbatim

Bring in the pure-Python RNDC protocol client as-is, with no
modifications, so that subsequent cleanup and integration changes can
be reviewed against the pristine upstream source.

Source: python-rndc rndc.py (version 0.2.0)

Co-authored-by: Matthew Pounsett <matt@conundrum.com>
Assisted-by: Claude:claude-fable-5
5 days agochg: test: Deduplicate controls configuration in named.conf test templates
Nicki Křížek [Thu, 23 Jul 2026 09:21:46 +0000 (11:21 +0200)] 
chg: test: Deduplicate controls configuration in named.conf test templates

The named.conf.j2 files in system tests had a copy-pasted snippets for control channel configuration. These were replaced with the `_common/controls.conf.j2` template.

The control channel was turned on for all named test instances.

Merge branch 'nicki/pytest-named-conf-controls' into 'main'

See merge request isc-projects/bind9!12386

5 days agoCover quoted key names in checkconf 12386/head
Nicki Křížek [Fri, 10 Jul 2026 11:47:46 +0000 (11:47 +0000)] 
Cover quoted key names in checkconf

The logfileconfig configs used to define an unquoted hyphenated
`key rndc-key` referenced as `keys { "rndc-key"; }`, incidentally
covering quoted/unquoted key-name interchangeability.  Now that those
configs use the shared controls include, keep that parser coverage in
a checkconf test.

Assisted-by: Claude:claude-fable-5
5 days agoMark the configs that deliberately keep a custom controls config
Nicki Křížek [Wed, 8 Jul 2026 18:36:20 +0000 (18:36 +0000)] 
Mark the configs that deliberately keep a custom controls config

Every remaining deviation from the shared _common/controls.conf.j2
include is intentional; annotate each with a one-line grep-able
comment stating why:

- rndc/ns2 and ns5 test multi-key and read-only channels; ns4 gets a
  runtime-generated multi-key channel appended by setup.sh;
- resolver/ns9 runs an IPv6-only channel, which the IPv4-only shared
  template cannot express;
- shutdown/resolver lives in a non-ns<N> directory, so the template's
  ns.ip variable is not injected there.

Assisted-by: Claude:claude-fable-5
5 days agoReplace empty controls statements with the controls template
Nicki Křížek [Wed, 8 Jul 2026 18:34:01 +0000 (18:34 +0000)] 
Replace empty controls statements with the controls template

None of those tests relies on the controls channel being absent, so
make the control channel available same as for all other named test
instances.

Assisted-by: Claude:claude-fable-5
5 days agoProvide a control channel on named instances that had none
Nicki Křížek [Wed, 8 Jul 2026 18:29:39 +0000 (18:29 +0000)] 
Provide a control channel on named instances that had none

Insert _common/controls.conf.j2 right after the options block in every
started named instance config that declared no control channel at all.
This ensure that each named instance exposes its control channel for the
tests.

Note that some configs may include the control channel via a named.conf
include mechanism. Those are left as is, since the control channel is
already configured via that include mechanism -- and those configs
aren't part of this commit at all.

Assisted-by: Claude:claude-fable-5
5 days agoUse controls template in named configs where rndc.key was included
Nicki Křížek [Wed, 8 Jul 2026 18:23:47 +0000 (18:23 +0000)] 
Use controls template in named configs where rndc.key was included

Same sweep as the previous commit, for the configs that obtained the
key via a native `include ".../_common/rndc.key";`.

Commit was split up for easier review-ability.

Assisted-by: Claude:claude-fable-5
5 days agoUse controls template in named configs where rndc key was inlined
Nicki Křížek [Wed, 8 Jul 2026 18:13:26 +0000 (18:13 +0000)] 
Use controls template in named configs where rndc key was inlined

Replace all occurences of rndc_key and controls channel config with the
standard _common/controls.conf.j2 template.

The server's IP address will be automatically filled in based on the
instance's directory name (nsX -> 10.53.0.X) using the @ns.ip@
dir-specific template variable.

The controls template also include the `rndc_key`, so any uses of that
remain functional.

If present, the empty controls section were removed, since they were
redundant.

The logfileconfig configs used quoted `"rndc-key"` and were switched to
`rndc_key` instead. The specific use-case of quoted key using a hyphen
will be re-added in one of the followup commits as a dedicated test.

Assisted-by: Claude:claude-fable-5
5 days agoFix controls statements bound to another instance's address
Nicki Křížek [Wed, 8 Jul 2026 18:06:19 +0000 (18:06 +0000)] 
Fix controls statements bound to another instance's address

Some of the system test named configs had a mismatched control channel
IP address. As there is no evidence this was intentional (no uses of the
control channel), it was most likely introduced as a copy-paste bug. Fix
the control channel IP to match the instance number.

Assisted-by: Claude:claude-fable-5
5 days agofix: nil: Fix the logged ZONEVERSION serial number
Evan Hunt [Thu, 23 Jul 2026 05:59:56 +0000 (05:59 +0000)] 
fix: nil: Fix the logged ZONEVERSION serial number

The serial number in the ZONEVERSION log message was computed
incorrectly; this has been fixed.

Closes #6208

Merge branch '6208-log-zoneversion' into 'main'

See merge request isc-projects/bind9!12338

5 days agoFix the logged ZONEVERSION serial number 12338/head
Evan Hunt [Wed, 1 Jul 2026 20:01:42 +0000 (13:01 -0700)] 
Fix the logged ZONEVERSION serial number

The serial number in the ZONEVERSION log message was computed
incorrectly; this has been fixed.

5 days agofix: nil: host -a now uses TCP
Evan Hunt [Thu, 23 Jul 2026 05:58:24 +0000 (05:58 +0000)] 
fix: nil: host -a now uses TCP

`host -a` is documented as being equivalent to `host -v -t any`,
but previously it used UDP instead of TCP. This has been corrected.

Closes #2072

Merge branch '2072-host-a-tcp' into 'main'

See merge request isc-projects/bind9!12440

5 days agohost -a should use TCP 12440/head
Evan Hunt [Wed, 22 Jul 2026 21:43:07 +0000 (14:43 -0700)] 
host -a should use TCP

"host -a" is documented as being equivalent to "host -v -t any",
but it uses UDP while the other uses TCP. This has been corrected.

5 days agofix: nil: Don't use wildcard keys in update-policy in ddns-confgen
Evan Hunt [Thu, 23 Jul 2026 05:55:05 +0000 (05:55 +0000)] 
fix: nil: Don't use wildcard keys in update-policy in ddns-confgen

In `ddns-confgen`, if the requested key name is a wildcard, don't generate
an `update-policy` statement, since using a wildcard as an SSU identity
allows other keys that match the wildcard to update records.

Closes #6235

Merge branch '6235-wildcard-ddns-confgen' into 'main'

See merge request isc-projects/bind9!12439

5 days agoDon't use wildcard keys in update-policy in ddns-confgen 12439/head
Evan Hunt [Wed, 22 Jul 2026 21:35:19 +0000 (14:35 -0700)] 
Don't use wildcard keys in update-policy in ddns-confgen

In ddns-confgen, if the requested key name is a wildcard, don't generate
an update-policy statement, since using a wildcard as an SSU identity
allows other keys that match the wildcard to update records.

6 days agochg: ci: Revert "allow_failure: true" for the "cross-version-config-tests" job andoni/fix-issues-work-items-reference-parsing 12436/head
Michał Kępień [Wed, 22 Jul 2026 13:24:02 +0000 (15:24 +0200)] 
chg: ci: Revert "allow_failure: true" for the "cross-version-config-tests" job

With the `v9.21.24` release tag now merged back into `main`, revert the
`allow_failure: true` change introduced in !12264 as the
`cross-version-config-tests` job is no longer expected to fail.

Merge branch 'michal/revert-allow_failure-true-for-cross-version-config-tests' into 'main'

See merge request isc-projects/bind9!12435

6 days agoRevert "Add allow_failure to cross-version-config tests" 12435/head
Michał Kępień [Wed, 22 Jul 2026 13:09:14 +0000 (15:09 +0200)] 
Revert "Add allow_failure to cross-version-config tests"

This reverts commit d3345763cb2671713004e6e9434714bfe1112d4a.

6 days agoMerge tag 'v9.21.24'
Michał Kępień [Wed, 22 Jul 2026 12:23:39 +0000 (14:23 +0200)] 
Merge tag 'v9.21.24'

6 days agonew: dev: Add more unit tests for isc_time API
Martin Basti [Wed, 22 Jul 2026 11:27:28 +0000 (11:27 +0000)] 
new: dev: Add more unit tests for isc_time API

While working on internal 64-bit time for BIND 9, the unit test suite for isc_time API has been extended.

Backport the unit tests from 64-bit time branch to the main branch. This should make the unit tests for isc_time API (mostly) complete.

Related to #2959

Merge branch '2959-add-more-time.c-unit-tests' into 'main'

See merge request isc-projects/bind9!5760

6 days agoMake isc_time_nowplusinterval consistent with other functions 5760/head
Martin Basti [Thu, 9 Jul 2026 12:25:33 +0000 (14:25 +0200)] 
Make isc_time_nowplusinterval consistent with other functions

isc_time_nowplusinterval was using different code than the
rest of functions in the time.c. Replace custom overflow logic
with ckd_add macro.

6 days agoAdd more unit tests for isc_test API
Ondřej Surý [Wed, 26 Jan 2022 10:50:12 +0000 (11:50 +0100)] 
Add more unit tests for isc_test API

While working on internal 64-bit time for BIND 9, the unit test suite
for isc_time API has been extended.

Backport the unit tests from 64-bit time branch to the main branch.
This should make the unit tests for isc_time API (mostly) complete.

6 days agoAdd more validity checks to the isc_time API
Ondřej Surý [Wed, 26 Jan 2022 11:08:10 +0000 (12:08 +0100)] 
Add more validity checks to the isc_time API

This commits backport few things from the 64-bit time branch:

* Add ISC_VALID_TIME() and ISC_VALID_INTERVAL() macros that checks
  the valid range for nanoseconds

* Add more specific buffer sizes for formatting the dates and require
  the buffers passed to the formatting functions to be at least that
  big.

* When error happens when formatting the string, always return a valid
  buffer with formatted mock datetime string instead of garbage.
  Because the formatting functions doesn't return status, the caller
  could print garbage (unterminated string).

6 days agofix: usr: Treat an unusable NSEC3 chain as a verification failure
Evan Hunt [Wed, 22 Jul 2026 07:09:06 +0000 (07:09 +0000)] 
fix: usr: Treat an unusable NSEC3 chain as a verification failure

When transferring in a mirror zone, DNSSEC verification could incorrectly succeed when the zone had an invalid `NSEC3PARAM` record, leading to subsequent validation failures. This has been fixed.

Closes #6136

Merge branch '6136-zoneverify-nsec3param' into 'main'

See merge request isc-projects/bind9!12408

6 days agoTreat unusable NSEC3PARAM as a verification failure 12408/head
Evan Hunt [Wed, 15 Jul 2026 06:02:11 +0000 (23:02 -0700)] 
Treat unusable NSEC3PARAM as a verification failure

dns_zoneverify_dnssec() could previously return success if the
NSEC3PARAM record was unusable, but no other NSEC/NSEC3 chains were
available. This has been fixed.

Co-Authored-By: Mark Andrews <marka@isc.org>
7 days agoAllow null nameserver for jinja2 test-zone templates
Evan Hunt [Wed, 15 Jul 2026 22:03:50 +0000 (15:03 -0700)] 
Allow null nameserver for jinja2 test-zone templates

It is now possible to generate a zone file from a jinja2 template that
isn't written to a nameserver directory, but to "zones/" in the top
level of the system test directory, by specifying the server name
"NO_NS" instead of NS1, NS2, etc. This can be used for testing command
line tools such as dnssec-verify on generated zone files, without
starting an unnecessary name server.

7 days agofix: usr: dig +yaml producing invalid YAML when a lookup fails
Martin Basti [Tue, 21 Jul 2026 20:01:42 +0000 (20:01 +0000)] 
fix: usr: dig +yaml producing invalid YAML when a lookup fails

When "dig +yaml" was run and no server could be reached, dig printed its plain-text startup banner (the "; \<\<\>\> DiG ..." and ";; global options" lines) ahead of the machine-readable output, so the result was not valid YAML and could not be parsed. dig no longer emits that banner in YAML mode. As part of the same change, the banner is now built only after the whole command line has been read, so options given after the query name (such as +nocmd, +short and +yaml) are correctly reflected in it.

Closes #1230

Merge branch '1230-dig-fix-error-comments' into 'main'

See merge request isc-projects/bind9!12416

7 days agoTest that +short does not leak the ";; " comments 12416/head
Martin Basti [Tue, 21 Jul 2026 13:49:00 +0000 (13:49 +0000)] 
Test that +short does not leak the ";; " comments

Plain +short already turns lookup->comments off, so the regression it
guards against only surfaces with "+short +comments", where +comments
re-enables the comment toggle while short form is still in effect.  The
existing silent-then-servfail scenario is reused because its retry path
reliably prints a "Got SERVFAIL reply from ..." comment; the +comments
case (without +short) is asserted first so the absence check is
meaningful.

Assisted-by: Claude:claude-opus-4-8
7 days agoDon't emit comments when +short form is enabled
Mark Andrews [Tue, 21 Jul 2026 07:22:00 +0000 (17:22 +1000)] 
Don't emit comments when +short form is enabled

7 days agoDon't print dig's startup banner with +nocmd when the lookup fails
Martin Basti [Wed, 15 Jul 2026 17:15:25 +0000 (19:15 +0200)] 
Don't print dig's startup banner with +nocmd when the lookup fails

This only affects the failure path.  On a successful lookup dig
reprints the banner and re-evaluates the final options as it does so,
so a late +nocmd is honored; but the "no servers could be reached"
error path prints the pre-built banner verbatim, without that
re-check, and that banner had been built too early to be correct.

printgreeting() was called the moment the first query name was seen,
so options appearing later on the command line (e.g. +[no]cmd, +short,
+yaml) were not yet in effect and the banner captured stale state.
That is why a failing "dig . soa @host +yaml" emitted the ";"-prefixed
banner ahead of the error, and why "+nocmd" placed after the query
name was ignored.

Assisted-by: Claude:claude-opus-4-8
7 days agoSuppress the dig startup banner in +yaml error output
Martin Basti [Wed, 15 Jul 2026 16:58:39 +0000 (18:58 +0200)] 
Suppress the dig startup banner in +yaml error output

The two "no servers could be reached" error paths printed the
lookup's startup banner (";"-prefixed "<<>> DiG ..." lines) ahead of
the "- type: DIG_ERROR" block.  Those lines are not valid YAML, so
with +yaml the whole output failed to parse whenever every server was
unreachable.  Guard the banner with the existing yaml flag, matching
how the successful path in printmessage() already gates it.

Assisted-by: Claude:claude-opus-4-8
7 days agofix: dev: QUIC crypto fixes
Arаm Sаrgsyаn [Tue, 21 Jul 2026 13:40:54 +0000 (13:40 +0000)] 
fix: dev: QUIC crypto fixes

The isc_crypto_aead_open() and isc_crypto_aead_seal() functions
in ossl3.c didn't check if 'additional_data' exists before using
it. The checks were in place in the ossl1_1.c implementation. Use
the same conditions in the ossl3.c implementation too.

Additionally, the ciphertext length passed to the EVP_DecryptUpdate()
function included the tag length too which caused errors when decrypting.
Use the 'len' variable instead which doesn't include the tag length.

Merge branch 'aram/quic-crypto-fixes' into 'main'

See merge request isc-projects/bind9!12413

7 days agoFix isc_crypto_aead_open() for OpenSSL 1.1.x 12413/head
Aram Sargsyan [Wed, 15 Jul 2026 13:19:17 +0000 (13:19 +0000)] 
Fix isc_crypto_aead_open() for OpenSSL 1.1.x

The ciphertext length passed to the EVP_DecryptUpdate() function
included the tag length too. Use the 'len' variable instead which
doesn't include the tag length.

7 days agoCheck AEAD additional data existence before use
Aram Sargsyan [Wed, 15 Jul 2026 13:14:42 +0000 (13:14 +0000)] 
Check AEAD additional data existence before use

The isc_crypto_aead_open() and isc_crypto_aead_seal() functions
in ossl3.c didn't check if 'additional_data' exists before using
it. The checks were in place in the ossl1_1.c implementation. Use
the same conditions in the ossl3.c implementation too.

7 days agofix: test: Mark the qpdb unit test as flaky
Nicki Křížek [Tue, 21 Jul 2026 08:11:28 +0000 (10:11 +0200)] 
fix: test: Mark the qpdb unit test as flaky

Closes #6267

Merge branch '6267-mark-qpdb-unit-test-flaky' into 'main'

See merge request isc-projects/bind9!12427

8 days agoMark the qpdb unit test as flaky 12427/head
Nicki Křížek [Mon, 20 Jul 2026 12:55:11 +0000 (12:55 +0000)] 
Mark the qpdb unit test as flaky

The qpdb unit test intermittently aborts on the
`isc_mem_inuse(mctx) >= hiwater` assertion in the overmempurge tests
(#6267) and is not reliably reproducible. Extend the flaky-test retry
mechanism from tests/isc to tests/dns so CI re-runs qpdb before
declaring the job failed.

Assisted-by: Claude:claude-fable-5
8 days agonew: usr: Disclose active Negative Trust Anchors with Extended DNS Error 33
Ondřej Surý [Mon, 20 Jul 2026 10:50:09 +0000 (12:50 +0200)] 
new: usr: Disclose active Negative Trust Anchors with Extended DNS Error 33

A Negative Trust Anchor (RFC 7646) turns off DNSSEC validation for a domain,
so a name that would normally fail validation resolves instead. named now
marks such answers with Extended DNS Error code 33, "Negative Trust Anchor",
so operators can see at a glance when a response came back only because
an NTA was in effect.

Closes #6268

Merge branch '6268-implement-nta-ede' into 'main'

See merge request isc-projects/bind9!12424

8 days agoTest Negative Trust Anchor disclosure via Extended DNS Error 33 12424/head
Ondřej Surý [Mon, 20 Jul 2026 08:46:09 +0000 (10:46 +0200)] 
Test Negative Trust Anchor disclosure via Extended DNS Error 33

Extend the EDE unit test to cover the newly registered code 33 and to
verify it does not collide with lower codes in the used-code bitmap.

Add a resolver system test: a name whose validation is suppressed by a
Negative Trust Anchor is answered with EDE 33 present, while the same
name without an NTA still fails validation and carries no such error.

Assisted-by: Claude:claude-opus-4-8
8 days agoDisclose Negative Trust Anchors with Extended DNS Error 33
Ondřej Surý [Mon, 20 Jul 2026 08:45:50 +0000 (10:45 +0200)] 
Disclose Negative Trust Anchors with Extended DNS Error 33

A Negative Trust Anchor (RFC 7646) makes a validating resolver treat an
otherwise-secure name as insecure, but there was no in-band way for a
client to tell that an answer which should have failed DNSSEC validation
was returned because an NTA was in place.

Register Extended DNS Error INFO-CODE 33, "Negative Trust Anchor"
(draft-farrokhi-dnsop-ede-nta), and attach it to a response whenever a
covering NTA suppresses validation for the queried name.

8 days agofix: usr: Resolver could return expired records instead of a negative answer
Ondřej Surý [Mon, 20 Jul 2026 06:30:06 +0000 (08:30 +0200)] 
fix: usr: Resolver could return expired records instead of a negative answer

When an unvalidated negative answer (such as one obtained for
a query with the "checking disabled" flag set) arrived for
a name that had DNSSEC-validated records in the cache, those
records blocked the negative answer from being cached even
after they had passed their TTL, and the expired records could
be returned to the client instead. Validated records that have
expired no longer prevent negative answers from being cached.

Merge branch '5877-skip-inactive-headers' into 'main'

See merge request isc-projects/bind9!12423

8 days agoTest that an expired RRset does not block negative caching 12423/head
Ondřej Surý [Mon, 20 Jul 2026 05:52:03 +0000 (07:52 +0200)] 
Test that an expired RRset does not block negative caching

Cover the inverse of the CD=1 NXDOMAIN guard: a validated RRset that
has passed its TTL, but has not yet been cleaned from the cache, must
not block the unvalidated negative entry. Before the fix the CD=1
query failed with SERVFAIL until the expired header was reaped.

Assisted-by: Claude:claude-fable-5
8 days agoOnly active secure data should block negative cache entries
Ondřej Surý [Mon, 20 Jul 2026 04:53:36 +0000 (06:53 +0200)] 
Only active secure data should block negative cache entries

Expired secure headers linger in the node's header list until they
are reaped, so the secure-data check could match a header already
past its TTL, reject the new negative entry, and hand the expired
data back to the caller. Require the matching header to be active.

11 days agochg: dev: Do not attach authdb
Alessio Podda [Fri, 17 Jul 2026 13:07:57 +0000 (13:07 +0000)] 
chg: dev: Do not attach authdb

The authdb variable is used either to check that, on restarts, we
do not cross to a different zone unless recursion is enabled, and to
lookup the zone version when filling the additional section.

Neither use requires the pointer to be attached, and attaching the
pointer causes scalability issues. This commit solves the problem by
turning the pointer into an integer id.

Merge branch 'alessio/authdb-version-lookup' into 'main'

See merge request isc-projects/bind9!12375

11 days agoFix mirror zone auth issues 12375/head
Alessio Podda [Mon, 13 Jul 2026 15:32:28 +0000 (17:32 +0200)] 
Fix mirror zone auth issues

Previously query_validatezonedb would not save the current version of
mirror zones, which would cause the mirror zone not to be found when
non-glue additional data.

We fix this by storing the mirror zone version in the client, and
deferring the acl check in query_validatezonedb.

11 days agoAdd test for mirror zone additional data
Alessio Podda [Thu, 9 Jul 2026 12:26:46 +0000 (14:26 +0200)] 
Add test for mirror zone additional data

Existing mirror-zone system tests only covered referral glue.
With this commit, we add a test that also checks for non-glue
additional data.

11 days agoMake authdb version lookup return isc_result_t
Alessio Podda [Thu, 9 Jul 2026 11:58:10 +0000 (13:58 +0200)] 
Make authdb version lookup return isc_result_t

Returning isc_result_t is more idiomatic. Also, there was some shared
code between ns_client_findversionid and ns_client_findversion that
has now been extracted into an helper function.

11 days agoTurn authdb into a enum
Alessio Podda [Mon, 6 Jul 2026 08:26:31 +0000 (10:26 +0200)] 
Turn authdb into a enum

Given that now authdb is now an integer id, we can fold the authdb
variable into it by turning it into a sentinel value.

11 days agoDo not attach authdb
Alessio Podda [Mon, 6 Jul 2026 08:15:13 +0000 (10:15 +0200)] 
Do not attach authdb

The authdb variable is used either to check that, on restarts, we
do not cross to a different zone unless recursion is enabled, and to
lookup the zone version when filling the additional section.

Neither use requires the pointer to be attached, and attaching the
pointer causes scalability issues. This commit solves the problem by
turning the pointer into an integer id.

12 days agofix: usr: Unterminated OpenSSL private-key `Label:` field can be read past its parser...
Mark Andrews [Thu, 16 Jul 2026 20:07:48 +0000 (06:07 +1000)] 
fix: usr: Unterminated OpenSSL private-key `Label:` field can be read past its parser buffer

Check that the string encoded in the Label: field of the .private
file of a key pair is NUL terminated and the correct length.  Reject
the .private file if it is not.

Closes #6193

Merge branch '6193-check-that-label-is-nul-terminated' into 'main'

See merge request isc-projects/bind9!12364

12 days agoMalformed Label: data in .private files was not detected 12364/head
Mark Andrews [Wed, 8 Jul 2026 01:18:44 +0000 (11:18 +1000)] 
Malformed Label: data in .private files was not detected

Check that the string encoded in the Label: field of the .private
file of a key pair is NUL terminated and the correct length.  Reject
the .private file if it is not.

13 days agofix: nil: Extend meson.build to support python 3.13 and 3.14
Mark Andrews [Wed, 15 Jul 2026 23:36:30 +0000 (09:36 +1000)] 
fix: nil: Extend meson.build to support python 3.13 and 3.14

The later releases of python are not supported by meson.build
requiring the built environment to maintain / install older
versions of python to be able build and test.

Merge branch 'marka-support-python-3.14' into 'main'

See merge request isc-projects/bind9!12389

13 days agoExtend meson.build to know about python 3.13 and 3.14 12389/head
Mark Andrews [Mon, 13 Jul 2026 07:00:40 +0000 (17:00 +1000)] 
Extend meson.build to know about python 3.13 and 3.14

13 days agofix: dev: Fix TSIG keys creation/eviction ordering issue
Arаm Sаrgsyаn [Wed, 15 Jul 2026 15:02:23 +0000 (15:02 +0000)] 
fix: dev: Fix TSIG keys creation/eviction ordering issue

When adding a new key into a full list, the newly inserted key
could be evicted just after the insertion if all the existing
keys were marked as visited. This has been fixed.

Closes #6263

Merge branch '6263-tkey-quota-bug-fix' into 'main'

See merge request isc-projects/bind9!12400

13 days agoFix TSIG keys creation/eviction bug 12400/head
Aram Sargsyan [Tue, 14 Jul 2026 11:26:50 +0000 (11:26 +0000)] 
Fix TSIG keys creation/eviction bug

When adding a new key to the SIEVE list, make the eviction decision
first, and then add the new key, so that it doesn't get evicted
immediately after insertion.

13 days agoAdd a new check in the tsig unit test
Aram Sargsyan [Tue, 14 Jul 2026 11:25:09 +0000 (11:25 +0000)] 
Add a new check in the tsig unit test

The new check creates more TSIG keys than the maximum allowed number,
and expectes that all the newly created keys are findable just after
they were created. I.e., when full, the newly created key should not
evicted.

13 days agofix: test: Increase timeout for zone update in multisigner test
Nicki Křížek [Wed, 15 Jul 2026 14:43:27 +0000 (16:43 +0200)] 
fix: test: Increase timeout for zone update in multisigner test

In slower platforms in CI (e.g. freebsd14), the 10s timeout could be too
short causing the test to fail. Increase it to make the test stable even
during heavy CI load.

---

failing job example: https://gitlab.isc.org/isc-private/bind9/-/jobs/7762354

Merge branch 'nicki/pytest-multisigner-increase-timeout' into 'main'

See merge request isc-projects/bind9!12373

13 days agoIncrease timeout for zone update in multisigner test 12373/head
Nicki Křížek [Thu, 9 Jul 2026 08:16:40 +0000 (10:16 +0200)] 
Increase timeout for zone update in multisigner test

In slower platforms in CI (e.g. freebsd14), the 10s timeout could be too
short causing the test to fail. Increase it to make the test stable even
during heavy CI load.