fix unitiailized pointer check on getipandkeylist

Function `named_config_getipandkeylist` could, in case of error in the
early code attempting to get the `port` or `tls-port`, make a pointer
check on a non-initialized value. This is now fixed.

diff --git a/bin/named/config.c b/bin/named/config.c
index 32918feba692b7b36d9fefc428403619a2f3f960..81eb498925e2b39673fb0759123231d72cfe6f94 100644 (file)
--- a/bin/named/config.c
+++ b/bin/named/config.c
@@ -555,6 +555,7 @@ named_config_getipandkeylist(const cfg_obj_t *config, const cfg_obj_t *list,
        isc_result_t result;
        in_port_t def_port;
        in_port_t def_tlsport;
+       getipandkeylist_state_t s = {};
 
        REQUIRE(ipkl != NULL);
        REQUIRE(ipkl->count == 0);
@@ -580,7 +581,6 @@ named_config_getipandkeylist(const cfg_obj_t *config, const cfg_obj_t *list,
        /*
         * Process the (nested) list(s).
         */
-       getipandkeylist_state_t s = {};
        result = getipandkeylist(def_port, def_tlsport, config, list,
                                 (in_port_t)0, NULL, NULL, mctx, &s);
        if (result != ISC_R_SUCCESS) {