diff --git a/CHANGES b/CHANGES index 9a033e1573..3af1d90076 100644 --- a/CHANGES +++ b/CHANGES @@ -1,3 +1,7 @@ +6308. [bug] Prevent crashes caused by the zone journal getting + destroyed before all changes from an incoming IXFR are + written to it. [GL #4496] + 6307. [bug] Obtain a client->handle reference when calling async_restart. [GL #4439] diff --git a/lib/dns/probes.d b/lib/dns/probes.d index 2d9f2c3d62..f5abec43ca 100644 --- a/lib/dns/probes.d +++ b/lib/dns/probes.d @@ -17,8 +17,6 @@ provider libdns { probe xfrin_connected(void *, char *, int); probe xfrin_done_callback_begin(void *, char *, int); probe xfrin_done_callback_end(void *, char *, int); - probe xfrin_journal_destroy_begin(void *, char *, int); - probe xfrin_journal_destroy_end(void *, char *, int); probe xfrin_read(void *, char *, int); probe xfrin_recv_answer(void *, char *, void *); probe xfrin_recv_done(void *, char *, int); diff --git a/lib/dns/xfrin.c b/lib/dns/xfrin.c index 697e8507c9..9ac923f5e7 100644 --- a/lib/dns/xfrin.c +++ b/lib/dns/xfrin.c @@ -1656,13 +1656,6 @@ get_edns_expire(dns_xfrin_t *xfr, dns_message_t *msg) { static void xfrin_end(dns_xfrin_t *xfr, isc_result_t result) { - /* Close the journal. */ - if (xfr->ixfr.journal != NULL) { - LIBDNS_XFRIN_JOURNAL_DESTROY_BEGIN(xfr, xfr->info, result); - dns_journal_destroy(&xfr->ixfr.journal); - LIBDNS_XFRIN_JOURNAL_DESTROY_END(xfr, xfr->info, result); - } - /* Inform the caller. */ if (xfr->done != NULL) { LIBDNS_XFRIN_DONE_CALLBACK_BEGIN(xfr, xfr->info, result);