diff --git a/doc/arm/changelog.rst b/doc/arm/changelog.rst index 0069fc9a5f..3779e44607 100644 --- a/doc/arm/changelog.rst +++ b/doc/arm/changelog.rst @@ -18,6 +18,7 @@ Changelog development. Regular users should refer to :ref:`Release Notes ` for changes relevant to them. +.. include:: ../changelog/changelog-9.20.3.rst .. include:: ../changelog/changelog-9.20.2.rst .. include:: ../changelog/changelog-9.20.1.rst .. include:: ../changelog/changelog-history.rst diff --git a/doc/arm/notes.rst b/doc/arm/notes.rst index 4ebe00686c..9ca186283c 100644 --- a/doc/arm/notes.rst +++ b/doc/arm/notes.rst @@ -37,6 +37,7 @@ https://www.isc.org/download/. There you will find additional information about each release, and source code. .. include:: ../notes/notes-known-issues.rst +.. include:: ../notes/notes-9.20.3.rst .. include:: ../notes/notes-9.20.2.rst .. include:: ../notes/notes-9.20.1.rst .. include:: ../notes/notes-9.20.0.rst diff --git a/doc/changelog/changelog-9.20.3.rst b/doc/changelog/changelog-9.20.3.rst new file mode 100644 index 0000000000..0234af0681 --- /dev/null +++ b/doc/changelog/changelog-9.20.3.rst @@ -0,0 +1,214 @@ +.. Copyright (C) Internet Systems Consortium, Inc. ("ISC") +.. +.. SPDX-License-Identifier: MPL-2.0 +.. +.. This Source Code Form is subject to the terms of the Mozilla Public +.. License, v. 2.0. If a copy of the MPL was not distributed with this +.. file, you can obtain one at https://mozilla.org/MPL/2.0/. +.. +.. See the COPYRIGHT file distributed with this work for additional +.. information regarding copyright ownership. + +BIND 9.20.3 +----------- + +New Features +~~~~~~~~~~~~ + +- Log query response status to the query log. ``cee11c8610f`` + + Log a query response summary using the new category `responses`. + Logging can be controlled by the option `responselog` and `rndc + responselog`. :gl:`#459` :gl:`!9526` + +- Added WALLET type. ``dad3fafe9eb`` + + Add the new record type WALLET (262). This provides a mapping from a + domain name to a cryptographic currency wallet. Multiple mappings can + exist if multiple records exist. :gl:`#4947` :gl:`!9554` + +Feature Changes +~~~~~~~~~~~~~~~ + +- Set logging category for notify/xfer-in related messages. + ``1f553c61f76`` + + Some 'notify' and 'xfer-in' related log messages were logged at the + 'general' category instead of their own category. This has been fixed. + :gl:`#2730` :gl:`!9514` + +- Restore the number of threadpool threads back to original value. + ``a0eada53883`` + + The issue of long-running operations potentially blocking query + resolution has been fixed. Revert this temporary workaround and + restore the number of threadpool threads. :gl:`#4898` :gl:`!9532` + +- Allow IXFR-to-AXFR fallback on DNS_R_TOOMANYRECORDS. ``30c4cbd4035`` + + This change allows fallback from an IXFR failure to AXFR when the + reason is `DNS_R_TOOMANYRECORDS`. This is because this error condition + could be temporary only in an intermediate version of IXFR + transactions and it's possible that the latest version of the zone + doesn't have that condition. In such a case, the secondary would never + be able to update the zone (even if it could) without this fallback. + + This fallback behavior is particularly useful with the recently + introduced `max-records-per-type` and `max-types-per-name` options: + the primary may not have these limitations and may temporarily + introduce "too many" records, breaking IXFR. If the primary side + subsequently deletes these records, this fallback will help recover + the zone transfer failure automatically; without it, the secondary + side would first need to increase the limit, which requires more + operational overhead and has its own adverse effect. :gl:`#4928` + :gl:`!9471` + +- Remove statslock from dnssec-signzone. ``12eb16186ff`` + + Silence Coverity CID 468757 and 468767 (DATA RACE read not locked) by + converting dnssec-signzone to use atomics for statistics counters + rather than using a lock. :gl:`#4939` :gl:`!9500` + +- Use release memory ordering when incrementing reference counter. + ``19e3cd0cd2c`` + + As the relaxed memory ordering doesn't ensure any memory + synchronization, it is possible that the increment will succeed even + in the case when it should not - there is a race between + atomic_fetch_sub(..., acq_rel) and atomic_fetch_add(..., relaxed). + Only the result is consistent, but the previous value for both calls + could be same when both calls are executed at the same time. + :gl:`!9567` + +Bug Fixes +~~~~~~~~~ + +- Fix a statistics channel counter bug when 'forward only' zones are + used. ``2287dc0ac0d`` + + When resolving a zone with a 'forward only' policy, and finding out + that all the forwarders are marked as "bad", the 'ServerQuota' counter + of the statistics channel was incorrectly increased. This has been + fixed. :gl:`#1793` :gl:`!9502` + +- Fix a bug in the static-stub implementation. ``72626cf9405`` + + Static-stub addresses and addresses from other sources were being + mixed together, resulting in static-stub queries going to addresses + not specified in the configuration, or alternatively, static-stub + addresses being used instead of the correct server addresses. + :gl:`#4850` :gl:`!9571` + +- Don't allow statistics-channel if libxml2 and libjson-c are + unsupported. ``02822b70eee`` + + When the libxml2 and libjson-c libraries are not supported, the + statistics channel can't return anything useful, so it is now + disabled. Use of `statistics-channel` in `named.conf` is a fatal + error. :gl:`#4895` :gl:`!9486` + +- Separate DNSSEC validation from the long-running tasks. + ``c0022f68025`` + + As part of the KeyTrap \[CVE-2023-50387\] mitigation, the DNSSEC CPU- + intensive operations were offloaded to a separate threadpool that we + use to run other tasks that could affect the networking latency. + + If that threadpool is running some long-running tasks like RPZ, + catalog zone processing, or zone file operations, it would delay + DNSSEC validations to a point where the resolving signed DNS records + would fail. + + Split the CPU-intensive and long-running tasks into separate + threadpools in a way that the long-running tasks don't block the CPU- + intensive operations. :gl:`#4898` :gl:`!9495` + +- Fix assertion failure when processing access control lists. + ``a15d975dbe2`` + + The named process could terminate unexpectedly when processing access + control lists (ACLs). This has been fixed. :gl:`#4908` :gl:`!9466` + +- Fix bug in Offline KSK that is using ZSK with unlimited lifetime. + ``3f115d3cdae`` + + If the ZSK has unlimited lifetime, the timing metadata "Inactive" and + "Delete" cannot be found and is treated as an error, preventing the + zone to be signed. This has been fixed. :gl:`#4914` :gl:`!9453` + +- Fix data race in offloaded dns_message_checksig() ``3b5c4f94d70`` + + When verifying a message in an offloaded thread there is a race with + the worker thread which writes to the same buffer. Clone the message + buffer before offloading. :gl:`#4929` :gl:`!9490` + +- Limit the outgoing UDP send queue size. ``251b90c25e0`` + + If the operating system UDP queue gets full and the outgoing UDP + sending starts to be delayed, BIND 9 could exhibit memory spikes as it + tries to enqueue all the outgoing UDP messages. Try a bit harder to + deliver the outgoing UDP messages synchronously and if that fails, + drop the outgoing DNS message that would get queued up and then + timeout on the client side. :gl:`#4930` :gl:`!9511` + +- Do not set SO_INCOMING_CPU. ``6c9f3d0d1ed`` + + We currently set SO_INCOMING_CPU incorrectly, and testing by Ondrej + shows that fixing the issue by setting affinities is worse than + letting the kernel schedule threads without constraints. So we should + not set SO_INCOMING_CPU anymore. :gl:`#4936` :gl:`!9504` + +- Fix the 'rndc dumpdb' command's error reporting. ``d35f654d674`` + + The 'rndc dumpdb' command wasn't reporting errors which occurred when + starting up the database dump process by named, like, for example, a + permission denied error for the 'dump-file' file. This has been fixed. + Note, however, that 'rndc dumpdb' performs asynchronous writes, so + errors can also occur during the dumping process, which will not be + reported back to 'rndc', but which will still be logged by named. + :gl:`#4944` :gl:`!9553` + +- Fix long-running incoming transfers. ``c5cadd29d87`` + + Incoming transfers that took longer than 30 seconds would stop reading + from the TCP stream and the incoming transfer would be indefinitely + stuck causing BIND 9 to hang during shutdown. + + This has been fixed and the `max-transfer-time-in` and `max-transfer- + idle-in` timeouts are now honoured. :gl:`#4949` :gl:`!9536` + +- Fix assertion failure when receiving DNS responses over TCP. + ``e2058ab4619`` + + When matching the received Query ID in the TCP connection, an invalid + received Query ID can very rarely cause assertion failure. :gl:`#4952` + :gl:`!9582` + +- Don't ignore the local port number in dns_dispatch_add() for TCP. + ``97fad455d73`` + + The dns_dispatch_add() function registers the 'resp' entry in + 'disp->mgr->qids' hash table with 'resp->port' being 0, but in + tcp_recv_success(), when looking up an entry in the hash table after a + successfully received data the port is used, so if the local port was + set (i.e. it was not 0) it fails to find the entry and results in an + unexpected error. + + Set the 'resp->port' to the given local port value extracted from + 'disp->local'. :gl:`#4969` :gl:`!9581` + +- Add a missing rcu_read_unlock() call on exit path. ``5db2ec07395`` + + An exit path in the dns_dispatch_add() function fails to get out of + the RCU critical section when returning early. Add the missing + rcu_read_unlock() call. :gl:`!9564` + +- Don't enable REUSEADDR on outgoing UDP sockets. ``a6692e793c3`` + + The outgoing UDP sockets enabled `SO_REUSEADDR` that allows sharing of + the UDP sockets, but with one big caveat - the socket that was opened + the last would get all traffic. The dispatch code would ignore the + invalid responses in the dns_dispatch, but this could lead to + unexpected results. :gl:`!9583` + + diff --git a/doc/notes/notes-9.20.3.rst b/doc/notes/notes-9.20.3.rst new file mode 100644 index 0000000000..639250794c --- /dev/null +++ b/doc/notes/notes-9.20.3.rst @@ -0,0 +1,130 @@ +.. Copyright (C) Internet Systems Consortium, Inc. ("ISC") +.. +.. SPDX-License-Identifier: MPL-2.0 +.. +.. This Source Code Form is subject to the terms of the Mozilla Public +.. License, v. 2.0. If a copy of the MPL was not distributed with this +.. file, you can obtain one at https://mozilla.org/MPL/2.0/. +.. +.. See the COPYRIGHT file distributed with this work for additional +.. information regarding copyright ownership. + +Notes for BIND 9.20.3 +--------------------- + +New Features +~~~~~~~~~~~~ + +- Log query response status to the query log. + + Log a query response summary using the new ``responses`` category. + Logging can be controlled via the :any:`responselog` option and via + :option:`rndc responselog`. :gl:`#459` + +- Added WALLET type. + + Add the new record type WALLET (262). This provides a mapping from a + domain name to a cryptographic currency wallet. Multiple mappings can + exist if multiple records exist. :gl:`#4947` + +Feature Changes +~~~~~~~~~~~~~~~ + +- Set logging category for ``notify``/``xfer-in``-related messages. + + Some ``notify`` and ``xfer-in``-related log messages were logged at + the "general" category level instead of their own category. This has + been fixed. :gl:`#2730` + +- Allow IXFR-to-AXFR fallback on ``DNS_R_TOOMANYRECORDS``. + + This change allows fallback from an IXFR failure to AXFR when the + reason is ``DNS_R_TOOMANYRECORDS``. :gl:`#4928` + +Bug Fixes +~~~~~~~~~ + +- Fix a statistics channel counter bug when "forward only" zones are + used. + + When resolving a zone with a "forward only" policy, and finding out + that all the forwarders were marked as "bad", the "ServerQuota" + counter of the statistics channel was incorrectly increased. This has + been fixed. :gl:`#1793` + +- Fix a bug in the static-stub implementation. + + Static-stub addresses and addresses from other sources were being + mixed together, resulting in static-stub queries going to addresses + not specified in the configuration, or alternatively, static-stub + addresses being used instead of the correct server addresses. + :gl:`#4850` + +- Don't allow :any:`statistics-channels` if libxml2 and libjson-c are + not configured. + + When BIND 9 is not configured with the libxml2 and libjson-c + libraries, the use of the :any:`statistics-channels` option is a fatal + error. :gl:`#4895` + +- Separate DNSSEC validation from long-running tasks. + + Split CPU-intensive and long-running tasks into separate threadpools + in a way that the long-running tasks - like RPZ, catalog zone + processing, or zone file operations - don't block CPU-intensive + operations like DNSSEC validations. :gl:`#4898` + +- Fix an assertion failure when processing access control lists. + + The :iscman:`named` process could terminate unexpectedly when + processing ACLs. This has been fixed. :gl:`#4908` + +- Fix a bug in Offline KSK using a ZSK with an unlimited lifetime. + + If the ZSK had an unlimited lifetime, the timing metadata ``Inactive`` + and ``Delete`` could not be found and were treated as an error, + preventing the zone from being signed. This has been fixed. + :gl:`#4914` + +- Limit the outgoing UDP send queue size. + + If the operating system UDP queue got full and the outgoing UDP + sending started to be delayed, BIND 9 could exhibit memory spikes as + it tried to enqueue all the outgoing UDP messages. It now tries to + deliver the outgoing UDP messages synchronously; if that fails, it + drops the outgoing DNS message that would get queued up and then + timeout on the client side. :gl:`#4930` + +- Do not set ``SO_INCOMING_CPU``. + + Remove the ``SO_INCOMING_CPU`` setting as kernel scheduling performs + better without constraints. :gl:`#4936` + +- Fix the :option:`rndc dumpdb` command's error reporting. + + The :option:`rndc dumpdb` command was not reporting errors that + occurred when :iscman:`named` started up the database dump process. + This has been fixed. :gl:`#4944` + +- Fix long-running incoming transfers. + + Incoming transfers that took longer than 30 seconds would stop reading + from the TCP stream and the incoming transfer would be indefinitely + stuck, causing BIND 9 to hang during shutdown. + + This has been fixed, and the :any:`max-transfer-time-in` and + :any:`max-transfer-idle-in` timeouts are now honored. :gl:`#4949` + +- Fix an assertion failure when receiving DNS responses over TCP. + + When matching the received Query ID in the TCP connection, an invalid + Query ID could cause an assertion failure. This has been fixed. + :gl:`#4952` + + +Known Issues +~~~~~~~~~~~~ + +- There are no new known issues with this release. See :ref:`above + ` for a list of all known issues affecting this + BIND 9 branch.