upstream/bind9
1
0
Fork 0
mirror of https://github.com/isc-projects/bind9.git synced 2026-03-01 13:01:40 -05:00
Code Issues Projects Releases Packages Wiki Activity Actions

'{&dns}' is as valid as '{?dns}' in a SVCB's dohpath

Browse source 
See for example section 1.2. "Levels and Expression Types" of rfc6570.
This commit is contained in:
Remi Gacogne 2024-09-05 15:11:21 +02:00 committed by Mark Andrews
parent 54889fd2af
commit e74052ea71
2 changed files with 6 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
lib/dns/rdata/in_1/svcb_64.c
View file

@ -158,7 +158,7 @@ svcb_validate(uint16_t key, isc_region_t *region) {
/*
* Minimum valid dohpath is "/{?dns}" as
* it MUST be relative (leading "/") and
* MUST contain "{?dns}".
* MUST contain "{?dns}" or "{&dns}".
*/
if (region->length < 7) {
return DNS_R_FORMERR;
@ -173,8 +173,10 @@ svcb_validate(uint16_t key, isc_region_t *region) {
{
return DNS_R_FORMERR;
}
/* MUST contain "{?dns}" */
/* MUST contain "{?dns}" or "{&dns}" */
if (strnstr((char *)region->base, "{?dns}",
region->length) == NULL &&
strnstr((char *)region->base, "{&dns}",
region->length) == NULL)
{
return DNS_R_FORMERR;

2
tests/dns/rdata_test.c
View file

@ -2701,6 +2701,8 @@ ISC_RUN_TEST_IMPL(https_svcb) {
"1 example.net. key7=\"/{?dns}\""),
TEXT_VALID_LOOPCHG(1, "1 example.net. dohpath=/some/path{?dns}",
"1 example.net. key7=\"/some/path{?dns}\""),
TEXT_VALID_LOOPCHG(1, "1 example.net. dohpath=/some/path?key=value{&dns}",
"1 example.net. key7=\"/some/path?key=value{&dns}\""),
TEXT_INVALID("1 example.com. dohpath=no-slash"),
TEXT_INVALID("1 example.com. dohpath=/{?notdns}"),
TEXT_INVALID("1 example.com. dohpath=/notvariable"),