shaba/systemd
1
0
Fork 0
mirror of https://github.com/systemd/systemd.git synced 2024-10-31 16:21:26 +03:00
Code

resolved: reuse check for link-local IP address lookups

Browse Source 
Let's reuse accept_link_local_reverse_lookups() at one more place, where
we check for the list of link local reverase address domains. Since we
don't actually accept the domains here (but rather the opposite, not
accept), let's rename the function a bit more generically with accept_ →
match_.

While we are at it invert the if branches, to make things more easily
understandable: filter out the unwatnted stuff and have the "all good"
state as main codepath.
This commit is contained in:
Lennart Poettering 2020-11-09 18:37:13 +01:00
parent bb3443d4f6
commit c78735eb79
1 changed files with 13 additions and 17 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

30
src/resolve/resolved-dns-scope.c
View File

@ -459,7 +459,7 @@ int dns_scope_socket_tcp(DnsScope *s, int family, const union in_addr_union *add
return dns_scope_socket(s, SOCK_STREAM, family, address, server, port, ret_socket_address); return dns_scope_socket(s, SOCK_STREAM, family, address, server, port, ret_socket_address);
} }
static DnsScopeMatch accept_link_local_reverse_lookups(const char *domain) { static DnsScopeMatch match_link_local_reverse_lookups(const char *domain) {
assert(domain); assert(domain);
if (dns_name_endswith(domain, "254.169.in-addr.arpa") > 0) if (dns_name_endswith(domain, "254.169.in-addr.arpa") > 0)
@ -568,29 +568,25 @@ DnsScopeMatch dns_scope_good_domain(
return DNS_SCOPE_YES_BASE + n_best; return DNS_SCOPE_YES_BASE + n_best;
} }
/* See if this scope is suitable as default route. */ /* Exclude link-local IP ranges */
if (match_link_local_reverse_lookups(domain) >= DNS_SCOPE_YES_BASE ||
/* If networks use .local in their private setups, they are supposed to also add .local
* to their search domains, which we already checked above. Otherwise, we consider .local
* specific to mDNS and won't send such queries ordinary DNS servers. */
dns_name_endswith(domain, "local") > 0)
return DNS_SCOPE_NO;
/* If there was no match at all, then see if this scope is suitable as default route. */
if (!dns_scope_is_default_route(s)) if (!dns_scope_is_default_route(s))
return DNS_SCOPE_NO; return DNS_SCOPE_NO;
/* Exclude link-local IP ranges */ return DNS_SCOPE_MAYBE;
if (dns_name_endswith(domain, "254.169.in-addr.arpa") == 0 &&
dns_name_endswith(domain, "8.e.f.ip6.arpa") == 0 &&
dns_name_endswith(domain, "9.e.f.ip6.arpa") == 0 &&
dns_name_endswith(domain, "a.e.f.ip6.arpa") == 0 &&
dns_name_endswith(domain, "b.e.f.ip6.arpa") == 0 &&
/* If networks use .local in their private setups, they are supposed to also add .local to their search
* domains, which we already checked above. Otherwise, we consider .local specific to mDNS and won't
* send such queries ordinary DNS servers. */
dns_name_endswith(domain, "local") == 0)
return DNS_SCOPE_MAYBE;
return DNS_SCOPE_NO;
} }
case DNS_PROTOCOL_MDNS: { case DNS_PROTOCOL_MDNS: {
DnsScopeMatch m; DnsScopeMatch m;
m = accept_link_local_reverse_lookups(domain); m = match_link_local_reverse_lookups(domain);
if (m >= 0) if (m >= 0)
return m; return m;
@ -609,7 +605,7 @@ DnsScopeMatch dns_scope_good_domain(
case DNS_PROTOCOL_LLMNR: { case DNS_PROTOCOL_LLMNR: {
DnsScopeMatch m; DnsScopeMatch m;
m = accept_link_local_reverse_lookups(domain); m = match_link_local_reverse_lookups(domain);
if (m >= 0) if (m >= 0)
return m; return m;