From: Philip Hazel <ph10@hermes.cam.ac.uk>
Date: Tue, 26 Jun 2007 13:50:41 +0000 (+0000)
Subject: Fix bug introduced with +ignore_defer and +include_defer.
X-Git-Tag: exim-4_68~30
X-Git-Url: https://git.exim.org/exim.git/commitdiff_plain/321f3596d3d1a331feae4c5b4e520f47ce7d8890

Fix bug introduced with +ignore_defer and +include_defer.
---

diff --git a/src/src/match.c b/src/src/match.c
index ae1d90149..8b25d2e13 100644
--- a/src/src/match.c
+++ b/src/src/match.c
@@ -1,4 +1,4 @@
-/* $Cambridge: exim/src/src/match.c,v 1.18 2007/06/26 09:23:34 ph10 Exp $ */
+/* $Cambridge: exim/src/src/match.c,v 1.19 2007/06/26 13:50:41 ph10 Exp $ */
 
 /*************************************************
 *     Exim - an Internet mail transport agent    *
@@ -730,7 +730,7 @@ while ((sss = string_nextinlist(&list, &sep, buffer, sizeof(buffer))) != NULL)
 
     else
       {
-      uschar *error;
+      uschar *error = NULL;
       switch ((func)(arg, ss, valueptr, &error))
         {
         case OK:
@@ -747,8 +747,6 @@ while ((sss = string_nextinlist(&list, &sep, buffer, sizeof(buffer))) != NULL)
             error);
           break;
           }
-        HDEBUG(D_lists) debug_printf("%s %s (%s)\n", ot,
-          include_defer? "yes":"no", error);
         if (include_defer)
           {
           log_write(0, LOG_MAIN, "%s: accepted by +include_defer", error);
@@ -862,8 +860,6 @@ while ((sss = string_nextinlist(&list, &sep, buffer, sizeof(buffer))) != NULL)
           break;
           }
         (void)fclose(f);
-        HDEBUG(D_lists) debug_printf("%s %s (%s)\n", ot,
-          include_defer? "yes":"no", error);
         if (include_defer)
           {
           log_write(0, LOG_MAIN, "%s: accepted by +include_defer", error);
diff --git a/test/stderr/0069 b/test/stderr/0069
index 72ef492ba..bd1621da1 100644
--- a/test/stderr/0069
+++ b/test/stderr/0069
@@ -133,7 +133,6 @@ LOG: failed to find host name for V4NET.0.0.13: accepted by +include_unknown
 >>> processing "deny"
 >>> check hosts = +include_defer : test.again.dns
 >>> test.again.dns in dns_again_means_nonexist? no (option unset)
->>> host in "+include_defer : test.again.dns"? yes (DNS lookup of test.again.dns deferred)
 LOG: DNS lookup of test.again.dns deferred: accepted by +include_defer
 >>> deny: condition test succeeded
 LOG: H=[V4NET.0.0.13] F=<userx@myhost.test.ex> rejected RCPT <a@test.ex>
@@ -153,7 +152,6 @@ LOG: failed to find host name for V4NET.0.0.13: accepted by +include_unknown
 >>> processing "accept"
 >>> check hosts = test.again.dns : V4NET.0.0.13
 >>> test.again.dns in dns_again_means_nonexist? no (option unset)
->>> host in "test.again.dns : V4NET.0.0.13"? no (DNS lookup of test.again.dns deferred)
 >>> host in "test.again.dns : V4NET.0.0.13"? list match deferred for test.again.dns
 >>> accept: condition test deferred
 LOG: H=[V4NET.0.0.13] F=<userx@test.ex> temporarily rejected RCPT <b@test.ex>