summaryrefslogtreecommitdiff
path: root/doc
diff options
context:
space:
mode:
authorJeremy Harris <jgh146exb@wizmail.org>2017-12-28 20:09:05 +0000
committerJeremy Harris <jgh146exb@wizmail.org>2017-12-29 13:33:15 +0000
commitce93c6d840d56b3cd7b2ced65c4e96820040c8a7 (patch)
treec04ca2bc77ee5ea20c21d6871c8e74d4cfb73d7f /doc
parentcabf3e9902ba212cb85afb63bde7cd88fa027ef4 (diff)
Fix crash associated with dnsdb lookup done from DKIM ACL. Bug 2215
Broken-by: cc55f4208e
Diffstat (limited to 'doc')
-rw-r--r--doc/doc-txt/ChangeLog12
1 files changed, 12 insertions, 0 deletions
diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog
index 7ec669b1c..7d1d526d7 100644
--- a/doc/doc-txt/ChangeLog
+++ b/doc/doc-txt/ChangeLog
@@ -25,6 +25,18 @@ JH/04 Bug 2217: Tighten up the parsing of DKIM signature headers. Previously
Assumptions at that stage could crash the receive process on malformed
input.
+JH/05 Bug 2215: Fix crash associated with dnsdb lookup done from DKIM ACL.
+ While running the DKIM ACL we operate on the Permanent memory pool so that
+ variables created with "set" persist to the DATA ACL. Also (at any time)
+ DNS lookups that fail create cache records using the Permanent pool. But
+ expansions release any allocations made on the current pool - so a dnsdb
+ lookup expansion done in the DKIM ACL releases the memory used for the
+ DNS negative-cache, and bad things result. Solution is to switch to the
+ Main pool for expansions.
+ While we're in that code, add checks on the DNS cache during store_reset,
+ active in the testsuite.
+ Problem spotted, and debugging aided, by Wolfgang Breyha.
+
Exim version 4.90
-----------------