From: Michał Kępień Date: Thu, 18 Oct 2018 13:48:44 +0000 (+0200) Subject: Fix the configuration type used by the "server-addresses" option X-Git-Tag: v9.13.4~100^2~1 X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=b3245768584225fbc77d1d4aa2aab9954c02419b;p=thirdparty%2Fbind9.git Fix the configuration type used by the "server-addresses" option Contrary to what the documentation states, the "server-addresses" static-stub zone option does not accept custom port numbers. Fix the configuration type used by the "server-addresses" option to ensure documentation matches source code. Remove a check_zoneconf() test which is unnecessary with this fix in place. --- diff --git a/bin/named/named.conf.5 b/bin/named/named.conf.5 index dcea32d72e2..61048326819 100644 --- a/bin/named/named.conf.5 +++ b/bin/named/named.conf.5 @@ -899,8 +899,7 @@ view \fIstring\fR [ \fIclass\fR ] { request\-expire \fIboolean\fR; request\-ixfr \fIboolean\fR; serial\-update\-method ( date | increment | unixtime ); - server\-addresses { ( \fIipv4_address\fR | \fIipv6_address\fR ) [ - port \fIinteger\fR ]; \&.\&.\&. }; + server\-addresses { ( \fIipv4_address\fR | \fIipv6_address\fR ); \&.\&.\&. }; server\-names { \fIstring\fR; \&.\&.\&. }; sig\-signing\-nodes \fIinteger\fR; sig\-signing\-signatures \fIinteger\fR; @@ -1007,8 +1006,7 @@ zone \fIstring\fR [ \fIclass\fR ] { request\-expire \fIboolean\fR; request\-ixfr \fIboolean\fR; serial\-update\-method ( date | increment | unixtime ); - server\-addresses { ( \fIipv4_address\fR | \fIipv6_address\fR ) [ port - \fIinteger\fR ]; \&.\&.\&. }; + server\-addresses { ( \fIipv4_address\fR | \fIipv6_address\fR ); \&.\&.\&. }; server\-names { \fIstring\fR; \&.\&.\&. }; sig\-signing\-nodes \fIinteger\fR; sig\-signing\-signatures \fIinteger\fR; diff --git a/bin/named/named.conf.docbook b/bin/named/named.conf.docbook index 04fc8476a87..49aa2104f9b 100644 --- a/bin/named/named.conf.docbook +++ b/bin/named/named.conf.docbook @@ -13,7 +13,7 @@ - 2018-09-04 + 2018-10-23 ISC @@ -879,8 +879,7 @@ view string [ class ] { request-expire boolean; request-ixfr boolean; serial-update-method ( date | increment | unixtime ); - server-addresses { ( ipv4_address | ipv6_address ) [ - port integer ]; ... }; + server-addresses { ( ipv4_address | ipv6_address ); ... }; server-names { string; ... }; sig-signing-nodes integer; sig-signing-signatures integer; @@ -983,8 +982,7 @@ zone string [ class ] { request-expire boolean; request-ixfr boolean; serial-update-method ( date | increment | unixtime ); - server-addresses { ( ipv4_address | ipv6_address ) [ port - integer ]; ... }; + server-addresses { ( ipv4_address | ipv6_address ); ... }; server-names { string; ... }; sig-signing-nodes integer; sig-signing-signatures integer; diff --git a/bin/named/named.conf.html b/bin/named/named.conf.html index 2e4bff72bcb..7169f14cebd 100644 --- a/bin/named/named.conf.html +++ b/bin/named/named.conf.html @@ -868,8 +868,7 @@ view request-expire boolean;
request-ixfr boolean;
serial-update-method ( date | increment | unixtime );
- server-addresses { ( ipv4_address | ipv6_address ) [
-     port integer ]; ... };
+ server-addresses { ( ipv4_address | ipv6_address ); ... };
server-names { string; ... };
sig-signing-nodes integer;
sig-signing-signatures integer;
@@ -973,8 +972,7 @@ zone request-expire boolean;
request-ixfr boolean;
serial-update-method ( date | increment | unixtime );
- server-addresses { ( ipv4_address | ipv6_address ) [ port
-     integer ]; ... };
+ server-addresses { ( ipv4_address | ipv6_address ); ... };
server-names { string; ... };
sig-signing-nodes integer;
sig-signing-signatures integer;
diff --git a/doc/arm/Bv9ARM.ch05.html b/doc/arm/Bv9ARM.ch05.html index 6c08c0a4940..36939a6ea3c 100644 --- a/doc/arm/Bv9ARM.ch05.html +++ b/doc/arm/Bv9ARM.ch05.html @@ -9465,7 +9465,7 @@ view "external" { forward ( first | only ); forwarders [ port integer ] [ dscp integer ] { ( ipv4_address | ipv6_address ) [ port integer ] [ dscp integer ]; ... }; max-records integer; - server-addresses { ( ipv4_address | ipv6_address ) [ port integer ]; ... }; + server-addresses { ( ipv4_address | ipv6_address ); ... }; server-names { string; ... }; zone-statistics ( full | terse | none | boolean ); }; diff --git a/doc/arm/man.named.conf.html b/doc/arm/man.named.conf.html index 8dab2cff4fc..3c91638ea3b 100644 --- a/doc/arm/man.named.conf.html +++ b/doc/arm/man.named.conf.html @@ -886,8 +886,7 @@ view request-expire boolean;
request-ixfr boolean;
serial-update-method ( date | increment | unixtime );
- server-addresses { ( ipv4_address | ipv6_address ) [
-     port integer ]; ... };
+ server-addresses { ( ipv4_address | ipv6_address ); ... };
server-names { string; ... };
sig-signing-nodes integer;
sig-signing-signatures integer;
@@ -991,8 +990,7 @@ zone request-expire boolean;
request-ixfr boolean;
serial-update-method ( date | increment | unixtime );
- server-addresses { ( ipv4_address | ipv6_address ) [ port
-     integer ]; ... };
+ server-addresses { ( ipv4_address | ipv6_address ); ... };
server-names { string; ... };
sig-signing-nodes integer;
sig-signing-signatures integer;
diff --git a/doc/arm/static-stub.zoneopt.xml b/doc/arm/static-stub.zoneopt.xml index 4aadf689367..beb1f105622 100644 --- a/doc/arm/static-stub.zoneopt.xml +++ b/doc/arm/static-stub.zoneopt.xml @@ -18,7 +18,7 @@ forward ( first | only ); forwarders [ port integer ] [ dscp integer ] { ( ipv4_address | ipv6_address ) [ port integer ] [ dscp integer ]; ... }; max-records integer; - server-addresses { ( ipv4_address | ipv6_address ) [ port integer ]; ... }; + server-addresses { ( ipv4_address | ipv6_address ); ... }; server-names { string; ... }; zone-statistics ( full | terse | none | boolean ); }; diff --git a/doc/misc/options b/doc/misc/options index 546e5144d4c..e845e3e52f1 100644 --- a/doc/misc/options +++ b/doc/misc/options @@ -803,8 +803,7 @@ view [ ] { request-expire ; request-ixfr ; serial-update-method ( date | increment | unixtime ); - server-addresses { ( | ) [ - port ]; ... }; + server-addresses { ( | ); ... }; server-names { ; ... }; sig-signing-nodes ; sig-signing-signatures ; @@ -908,8 +907,7 @@ zone [ ] { request-expire ; request-ixfr ; serial-update-method ( date | increment | unixtime ); - server-addresses { ( | ) [ port - ]; ... }; + server-addresses { ( | ); ... }; server-names { ; ... }; sig-signing-nodes ; sig-signing-signatures ; diff --git a/doc/misc/static-stub.zoneopt b/doc/misc/static-stub.zoneopt index 809daf38e73..f89d46248a4 100644 --- a/doc/misc/static-stub.zoneopt +++ b/doc/misc/static-stub.zoneopt @@ -5,7 +5,7 @@ zone [ ] { forward ( first | only ); forwarders [ port ] [ dscp ] { ( | ) [ port ] [ dscp ]; ... }; max-records ; - server-addresses { ( | ) [ port ]; ... }; + server-addresses { ( | ); ... }; server-names { ; ... }; zone-statistics ( full | terse | none | ); }; diff --git a/lib/bind9/check.c b/lib/bind9/check.c index 451bfbeeacd..40608518881 100644 --- a/lib/bind9/check.c +++ b/lib/bind9/check.c @@ -2461,13 +2461,6 @@ check_zoneconf(const cfg_obj_t *zconfig, const cfg_obj_t *voptions, obj = cfg_listelt_value(element); sa = *cfg_obj_assockaddr(obj); - if (isc_sockaddr_getport(&sa) != 0) { - result = ISC_R_FAILURE; - cfg_obj_log(obj, logctx, ISC_LOG_ERROR, - "port is not configurable for " - "static stub server-addresses"); - } - isc_netaddr_fromsockaddr(&na, &sa); if (isc_netaddr_getzone(&na) != 0) { result = ISC_R_FAILURE; diff --git a/lib/isccfg/namedconf.c b/lib/isccfg/namedconf.c index d6881de77f1..3b51a080bab 100644 --- a/lib/isccfg/namedconf.c +++ b/lib/isccfg/namedconf.c @@ -98,7 +98,7 @@ static cfg_type_t cfg_type_addrmatchelt; static cfg_type_t cfg_type_bracketed_aml; static cfg_type_t cfg_type_bracketed_dscpsockaddrlist; static cfg_type_t cfg_type_bracketed_namesockaddrkeylist; -static cfg_type_t cfg_type_bracketed_sockaddrlist; +static cfg_type_t cfg_type_bracketed_netaddrlist; static cfg_type_t cfg_type_bracketed_sockaddrnameportlist; static cfg_type_t cfg_type_controls; static cfg_type_t cfg_type_controls_sockaddr; @@ -566,10 +566,10 @@ static cfg_type_t cfg_type_bracketed_dscpsockaddrlist = { &cfg_type_sockaddrdscp }; -static cfg_type_t cfg_type_bracketed_sockaddrlist = { - "bracketed_sockaddrlist", cfg_parse_bracketed_list, +static cfg_type_t cfg_type_bracketed_netaddrlist = { + "bracketed_netaddrlist", cfg_parse_bracketed_list, cfg_print_bracketed_list, cfg_doc_bracketed_list, &cfg_rep_list, - &cfg_type_sockaddr + &cfg_type_netaddr }; static const char *autodnssec_enums[] = { @@ -2281,7 +2281,7 @@ zone_only_clauses[] = { { "pubkey", &cfg_type_pubkey, CFG_CLAUSEFLAG_MULTI | CFG_CLAUSEFLAG_OBSOLETE }, - { "server-addresses", &cfg_type_bracketed_sockaddrlist, + { "server-addresses", &cfg_type_bracketed_netaddrlist, CFG_ZONE_STATICSTUB }, { "server-names", &cfg_type_namelist,