Git log: commit 946180701191ea7e552fa5a939140a635d47b566
Author: Simon Kelley <simon@thekelleys.org.uk>
Date: Thu Mar 16 15:16:17 2023 +0000
Remove limitation on --dynamic-host.
Dynamic-host was implemented to ignore interface addresses with /32
(or /128 for IPv6) prefix lengths, since they are not useful for
synthesising addresses.
Due to a bug before 2.88, this didn't work for IPv4, and some have
used --dynamic-host=example.com,0.0.0.0,eth0 to do the equivalent of
--interface-name for such interfaces. When the bug was fixed in 2.88
these uses broke.
Since this behaviour seems to violate the principle of least surprise,
and since the 2.88 fix is breaking existing imstallations, this
commit removes the check on /32 and /128 prefix lengths to solve both
problems.
commit 00be8b39e240934e404533deda08cbae2aae25a8
Author: Simon Kelley <simon@thekelleys.org.uk>
Date: Wed Mar 15 21:12:55 2023 +0000
Fix DHCPv6 "use multicast" response which previously failed
to set the message type correctly.
Thanks to Petr Menšík for spotting the problem.
commit ef5aac95d4391fb1290fd76a3826b2851e589bbc
Author: Clayton Craft <clayton@craftyguy.net>
Date: Wed Mar 8 15:35:05 2023 +0000
Allow configuring filter-A/AAAA via dbus.
commit ef8e930e4295265b8f46898a8e166f17d7f8ddc8
Author: Simon Kelley <simon@thekelleys.org.uk>
Date: Tue Mar 7 22:46:44 2023 +0000
Generalise cached NXDOMAIN replies.
We can cache an NXDOMAIN reply to a query for any RRTYPE
and reply from a cached NXDOMAIN to any RRTYPE.
commit eb92fb32b746f2104b0f370b5b295bb8dd4bd5e5
Author: Simon Kelley <simon@thekelleys.org.uk>
Date: Tue Mar 7 22:07:46 2023 +0000
Set the default maximum DNS UDP packet size to 1232.
http://www.dnsflagday.net/2020/ refers.
Thanks to Xiang Li for the prompt.
commit 9a698434dd9cc0f3abbf98f9b266c491d322d20f
Author: Simon Kelley <simon@thekelleys.org.uk>
Date: Mon Mar 6 23:30:36 2023 +0000
Bump version in Debian changelog.
commit f5ef0f064c3f06b250a9eeda36dc239227658b00
Author: Simon Kelley <simon@thekelleys.org.uk>
Date: Mon Mar 6 23:00:58 2023 +0000
Fix possible SEGV when no servers defined.
If there exists a --address=/<domain>/ or --server=/<domain>/#
configuration but no upstream server config unqualified by
domain then when a query which doesnt match the domain is
recieved it will use the qualfied server config and in the process
possibly make an out-of-bounds memory access.
Thanks to Daniel Danzberger for spotting the bug.
commit 997982f78bd3f8c311b9557e1ef825555e7290bb
Author: Dominik Derigs <dl6er@dl6er.de>
Date: Fri Mar 3 18:05:26 2023 +0100
Fix --rev-server option. It was broken in 1db9943c6879c160a5fbef885d5ceadd3668b74d when resolving upstream servers by name was extended to --rev-server without accounting for the fact that re-using one and the same upstream server for each of the x.y.z.in-addr.arpa is actually a wanted feature
Signed-off-by: DL6ER <dl6er@dl6er.de>
commit 7d6b68c5d7016aca5372f12e9f0c25f0a108644d
Author: Simon Kelley <simon@thekelleys.org.uk>
Date: Mon Mar 6 13:06:03 2023 +0000
Document suppressing deafult options in --dhcp-option.
commit 137ae2e9cf0dc3596641e7c8b91d15307a35319e
Author: Taylor R Campbell <campbell+dnsmasq@mumble.net>
Date: Sat Feb 25 15:00:30 2023 +0000
Avoid undefined behaviour with the ctype(3) functions.
As defined in the C standard:
In all cases the argument is an int, the value of which shall
be representable as an unsigned char or shall equal the value
of the macro EOF. If the argument has any other value, the
behavior is undefined.
This is because they're designed to work with the int values returned
by getc or fgetc; they need extra work to handle a char value.
If EOF is -1 (as it almost always is), with 8-bit bytes, the allowed
inputs to the ctype(3) functions are:
{-1, 0, 1, 2, 3, ..., 255}.
However, on platforms where char is signed, such as x86 with the
usual ABI, code like
char *arg = ...;
... isspace(*arg) ...
may pass in values in the range:
{-128, -127, -126, ..., -2, -1, 0, 1, ..., 127}.
This has two problems:
1. Inputs in the set {-128, -127, -126, ..., -2} are forbidden.
2. The non-EOF byte 0xff is conflated with the value EOF = -1, so
even though the input is not forbidden, it may give the wrong
answer.
Casting char to int first before passing the result to ctype(3)
doesn't help: inputs like -128 are unchanged by this cast. It is
necessary to cast char inputs to unsigned char first; you can then
cast to int if you like but there's no need because the functions
will always convert the argument to int by definition. So the above
fragment needs to be:
char *arg = ...;
... isspace((unsigned char)*arg) ...
This patch inserts unsigned char casts where necessary, and changes
int casts to unsigned char casts where the input is char.
I left alone int casts where the input is unsigned char already --
they're not immediately harmful, although they would have the effect
of suppressing some compiler warnings if the input is ever changed to
be char instead of unsigned char, so it might be better to remove
those casts too.
I also left alone calls where the input is int to begin with because
it came from getc; casting to unsigned char here would be wrong, of
course.
