diff --git a/CHANGES b/CHANGES index 792531d6f1..79e101de9d 100644 --- a/CHANGES +++ b/CHANGES @@ -1,3 +1,6 @@ +4098. [bug] Address use-after-free issue when using a + predecessor key with dnssec-settime. [RT #39272] + 4097. [func] Add additional logging about xfrin transfer status. [RT #39170] @@ -92,7 +95,7 @@ 4061. [bug] Handle timeout in legacy system test. [RT #38573] 4060. [bug] dns_rdata_freestruct could be called on a - uninitialised structure when handling a error. + uninitialized structure when handling a error. [RT #38568] 4059. [bug] Addressed valgrind warnings. [RT #38549] diff --git a/bin/dnssec/dnssec-settime.c b/bin/dnssec/dnssec-settime.c index fce63efcc1..e26356c788 100644 --- a/bin/dnssec/dnssec-settime.c +++ b/bin/dnssec/dnssec-settime.c @@ -399,7 +399,6 @@ main(int argc, char **argv) { "inactive.\n", program); changed = setpub = setact = ISC_TRUE; - dst_key_free(&prevkey); } else { if (prepub < 0) prepub = 0; @@ -590,6 +589,8 @@ main(int argc, char **argv) { printf("%s\n", newname); } + if (prevkey != NULL) + dst_key_free(&prevkey); dst_key_free(&key); dst_lib_destroy(); isc_hash_destroy();