Mark variables that are unused before release of store in the receive message loop
authorJeremy Harris <jgh146exb@wizmail.org>
Thu, 15 Mar 2018 14:23:04 +0000 (14:23 +0000)
committerJeremy Harris <jgh146exb@wizmail.org>
Thu, 15 Mar 2018 15:42:50 +0000 (15:42 +0000)
doc/doc-txt/ChangeLog
src/src/acl.c
src/src/smtp_in.c

index f3f849a..6fb4402 100644 (file)
@@ -146,7 +146,8 @@ JH/28 Ensure that variables possibly set during message acceptance are marked
       about them when the debug_store option is enabled.  Discovered specifically
       for sender_rate_period, but applies to a whole set of variables.
       Do the same for the queue-runner loop, for variables set from spool
-      message files.
+      message files.  Do the same for the SMTP per-message loop, for certain
+      variables indirectly set in ACL operations.
 
 
 Exim version 4.90
index f614116..35d955d 100644 (file)
@@ -4477,7 +4477,7 @@ switch (where)
   }
 
 deliver_domain = deliver_localpart = deliver_address_data =
-  sender_address_data = NULL;
+  deliver_domain_data = sender_address_data = NULL;
 
 /* A DISCARD response is permitted only for message ACLs, excluding the PREDATA
 ACL, which is really in the middle of an SMTP command. */
index 6cac7d2..c9fe792 100644 (file)
@@ -1980,8 +1980,8 @@ active_local_sender_retain = local_sender_retain;    /* Can be set by ACL */
 sending_ip_address = NULL;
 return_path = sender_address = NULL;
 sender_data = NULL;                                 /* Can be set by ACL */
-deliver_localpart_orig = NULL;
-deliver_domain_orig = NULL;
+deliver_localpart_parent = deliver_localpart_orig = NULL;
+deliver_domain_parent = deliver_domain_orig = NULL;
 callout_address = NULL;
 submission_name = NULL;                              /* Can be set by ACL */
 raw_sender = NULL;                  /* After SMTP rewrite, before qualifying */