[9.20] fix: dev: 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.

Backport of MR !11303

Merge branch 'backport-colin/getipandkeylist-uinitstate' into 'bind-9.20'

See merge request isc-projects/bind9!11306
This commit is contained in:
Colin Vidal 2025-11-28 16:18:03 +01:00
commit 5ed0cf091b

View file

@ -819,6 +819,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);
@ -844,7 +845,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) {