- Fix rpz that copies the cname override completely to the temp

  region, so there are no references to the rpz region.

diff --git a/doc/Changelog b/doc/Changelog
index fe106cdcb92e6300efb7ac01e28c7e20035e574c..9deac82ff5f7e25a0a6a59503c4d2243f7b5d0f3 100644 (file)
--- a/doc/Changelog
+++ b/doc/Changelog
@@ -2,6 +2,8 @@
        - Fix that rpz CNAME content is limited to the max number of cnames.
        - Fix rpz, it follows iterator CNAMEs for nsip and nsdname and sets
          the reply query_info values, that is better for debug logging.
+       - Fix rpz that copies the cname override completely to the temp
+         region, so there are no references to the rpz region.
 
 15 March 2024: Yorgos
        - Merge #1030: Persist the openssl and expat directories for repeated

diff --git a/services/rpz.c b/services/rpz.c
index bcc0322bb777c37b7557081a79208853b1cf21ed..0637e326241fcd5e86dae7085b4af92b52747b3d 100644 (file)
--- a/services/rpz.c
+++ b/services/rpz.c
@@ -1888,8 +1888,7 @@ rpz_apply_cname_override_action(struct rpz* r,
                sizeof(struct local_rrset));
        if(qinfo->local_alias == NULL)
                return 0; /* out of memory */
-       qinfo->local_alias->rrset = regional_alloc_init(temp,
-               r->cname_override, sizeof(*r->cname_override));
+       qinfo->local_alias->rrset = respip_copy_rrset(r->cname_override, temp);
        if(qinfo->local_alias->rrset == NULL) {
                qinfo->local_alias = NULL;
                return 0; /* out of memory */