]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
dns/resolver.c: Return void when ISC_R_SUCCESS is only returned value
authorOndřej Surý <ondrej@sury.org>
Wed, 6 Oct 2021 10:59:45 +0000 (12:59 +0200)
committerOndřej Surý <ondrej@sury.org>
Wed, 13 Oct 2021 03:47:48 +0000 (05:47 +0200)
With isc_mem_get() and dns_name_dup() no longer being able to fail, some
functions can now only return ISC_R_SUCCESS.  Change the return type to
void for the following function(s):

 * dns_resolver_addalternate()

bin/named/server.c
lib/dns/include/dns/resolver.h
lib/dns/resolver.c

index 94d44d48d35cf0f810971e90636426fb341b1d4e..113a838be7ce96b2457bb9bbaec4aab91a37e054 100644 (file)
@@ -6074,8 +6074,8 @@ configure_alternates(const cfg_obj_t *config, dns_view_t *view,
                                }
                                myport = (in_port_t)val;
                        }
-                       CHECK(dns_resolver_addalternate(view->resolver, NULL,
-                                                       name, myport));
+                       dns_resolver_addalternate(view->resolver, NULL, name,
+                                                 myport);
                        continue;
                }
 
@@ -6083,7 +6083,7 @@ configure_alternates(const cfg_obj_t *config, dns_view_t *view,
                if (isc_sockaddr_getport(&sa) == 0) {
                        isc_sockaddr_setport(&sa, port);
                }
-               CHECK(dns_resolver_addalternate(view->resolver, &sa, NULL, 0));
+               dns_resolver_addalternate(view->resolver, &sa, NULL, 0);
        }
 
 cleanup:
index 918bddf928ea86816c36ca7450c97f8781d9ff6d..85d81c0c8a580b5207e8e12e733967928141f09e 100644 (file)
@@ -434,7 +434,7 @@ dns_resolver_setlamettl(dns_resolver_t *resolver, uint32_t lame_ttl);
  *\li  'resolver' to be valid.
  */
 
-isc_result_t
+void
 dns_resolver_addalternate(dns_resolver_t *resolver, const isc_sockaddr_t *alt,
                          const dns_name_t *name, in_port_t port);
 /*%<
index 7a500043107b4af0d9fdf4af79cf9361260761bb..1402dd786fa78935dc1a8efe64c211034fc188a6 100644 (file)
@@ -10708,7 +10708,7 @@ dns_resolver_setlamettl(dns_resolver_t *resolver, uint32_t lame_ttl) {
        resolver->lame_ttl = lame_ttl;
 }
 
-isc_result_t
+void
 dns_resolver_addalternate(dns_resolver_t *resolver, const isc_sockaddr_t *alt,
                          const dns_name_t *name, in_port_t port) {
        alternate_t *a;
@@ -10729,8 +10729,6 @@ dns_resolver_addalternate(dns_resolver_t *resolver, const isc_sockaddr_t *alt,
        }
        ISC_LINK_INIT(a, link);
        ISC_LIST_APPEND(resolver->alternates, a, link);
-
-       return (ISC_R_SUCCESS);
 }
 
 void