]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
dns-domain: dns_name_is_empty() is redundant
authorLennart Poettering <lennart@poettering.net>
Thu, 5 Jan 2023 15:18:31 +0000 (16:18 +0100)
committerLennart Poettering <lennart@poettering.net>
Thu, 5 Jan 2023 20:44:26 +0000 (21:44 +0100)
dns_name_is_empty() was added later, but does the same thing as the more
accurately called dns_name_is_root(), hence drop the former.

src/resolve/resolved-dns-scope.c
src/resolve/resolved-dns-synthesize.c
src/shared/dns-domain.c
src/shared/dns-domain.h

index 0c062efc985c0c8ca0bf27bb5392df04823d92ab..7b82ae2b4589ed539c765bd3e8cd332d93e1cffc 100644 (file)
@@ -649,11 +649,11 @@ DnsScopeMatch dns_scope_good_domain(
                 DnsScopeMatch m;
                 int n_best = -1;
 
-                if (dns_name_is_empty(domain)) {
+                if (dns_name_is_root(domain)) {
                         DnsResourceKey *t;
                         bool found = false;
 
-                        /* Refuse empty name if only A and/or AAAA records are requested. */
+                        /* Refuse root name if only A and/or AAAA records are requested. */
 
                         DNS_QUESTION_FOREACH(t, question)
                                 if (!IN_SET(t->type, DNS_TYPE_A, DNS_TYPE_AAAA)) {
index 51e06bb91ea0debd50730915687b1ec1de47f613..5bde29c704b775e11fbac80010831420358275ee 100644 (file)
@@ -463,7 +463,7 @@ int dns_synthesize_answer(
 
                 name = dns_resource_key_name(key);
 
-                if (dns_name_is_empty(name)) {
+                if (dns_name_is_root(name)) {
                         /* Do nothing. */
 
                 } else if (dns_name_dont_resolve(name)) {
index d209e17f952dc91cc305e8a28cb0e0135508c1b4..620b156563378da5201a02c6b2e60aff66520695 100644 (file)
@@ -832,7 +832,6 @@ int dns_name_address(const char *p, int *ret_family, union in_addr_union *ret_ad
 }
 
 bool dns_name_is_root(const char *name) {
-
         assert(name);
 
         /* There are exactly two ways to encode the root domain name:
index 7a35ae43bcb56cb8460fd257bd4409c96abd7252..331fb89637c52f30fca61172d79a0e7d94e64391 100644 (file)
@@ -60,10 +60,6 @@ static inline int dns_name_is_valid_ldh(const char *s) {
         return 1;
 }
 
-static inline bool dns_name_is_empty(const char *s) {
-        return isempty(s) || streq(s, ".");
-}
-
 void dns_name_hash_func(const char *s, struct siphash *state);
 int dns_name_compare_func(const char *a, const char *b);
 extern const struct hash_ops dns_name_hash_ops;