DKIM: fix non-debug build. Bug 1713
authorJasen Betts <jasen@xnet.co.nz>
Sat, 7 Nov 2015 11:53:52 +0000 (11:53 +0000)
committerJeremy Harris <jgh146exb@wizmail.org>
Sat, 7 Nov 2015 11:59:27 +0000 (11:59 +0000)
doc/doc-txt/ChangeLog
src/OS/Makefile-Base
src/src/dkim.c
src/src/pdkim/Makefile

index 2caae22..0468bf2 100644 (file)
@@ -76,6 +76,8 @@ JH/15 Bug 1712: Introduce $prdr_requested flag variable
 JH/16 Bug 1714: Permit an empty string as expansion result for transport
       option transport_filter, meaning no filtering.
 
+JH/17 Bug 1713: Fix non-PDKIM_DEBUG build.  Patch from Jasen Betts.
+
 
 Exim version 4.86
 -----------------
index 95110e6..f666781 100644 (file)
@@ -607,7 +607,7 @@ tod.o:           $(HDRS) tod.c
 transport.o:     $(HDRS) transport.c
 tree.o:          $(HDRS) tree.c
 verify.o:        $(HDRS) verify.c
-dkim.o:          $(HDRS) dkim.c
+dkim.o:          $(HDRS) pdkim/pdkim.h dkim.c
 
 # Dependencies for WITH_CONTENT_SCAN modules
 
@@ -620,19 +620,19 @@ spool_mbox.o:    $(HDRS) spool_mbox.c
 
 # Dependencies for WITH_OLD_DEMIME modules
 
-demime.o:        $(HDRS) demime.c
+demime.o:       $(HDRS) demime.c
 
 
 # Dependencies for EXPERIMENTAL_* modules
 
-bmi_spam.o:            $(HDRS) bmi_spam.c
-dane.o:                        $(HDRS) dane.c dane-gnu.c dane-openssl.c
-dcc.o:                 $(HDRS) dcc.h dcc.c
-dmarc.o:               $(HDRS) dmarc.h dmarc.c
+bmi_spam.o:    $(HDRS) bmi_spam.c
+dane.o:                $(HDRS) dane.c dane-gnu.c dane-openssl.c
+dcc.o:         $(HDRS) dcc.h dcc.c
+dmarc.o:       $(HDRS) pdkim/pdkim.h dmarc.h dmarc.c
 imap_utf7.o:   $(HDRS) imap_utf7.c
-spf.o:                 $(HDRS) spf.h spf.c
-srs.o:                 $(HDRS) srs.h srs.c
-utf8.o:                        $(HDRS) utf8.c
+spf.o:         $(HDRS) spf.h spf.c
+srs.o:         $(HDRS) srs.h srs.c
+utf8.o:                $(HDRS) utf8.c
 
 # The module containing tables of available lookups, routers, auths, and
 # transports must be rebuilt if any of them are. However, because the makefiles
index db2eb49..5045958 100644 (file)
@@ -70,9 +70,11 @@ if (dkim_verify_ctx)
 /* Create new context */
 
 dkim_verify_ctx = pdkim_init_verify(PDKIM_INPUT_SMTP, &dkim_exim_query_dns_txt);
-
-if ((dkim_collect_input = !!dkim_verify_ctx))
+dkim_collect_input = !!dkim_verify_ctx;
+#ifdef PDKIM_DEBUG
+if (dkim_collect_input)
   pdkim_set_debug_stream(dkim_verify_ctx, debug_file);
+#endif
 }
 
 
@@ -565,7 +567,9 @@ while ((dkim_signing_domain = string_nextinlist(&dkim_domain, &sep,
                         (char *) dkim_signing_domain,
                         (char *) dkim_signing_selector,
                         (char *) dkim_private_key_expanded);
+#ifdef PDKIM_DEBUG
   pdkim_set_debug_stream(ctx, debug_file);
+#endif
   pdkim_set_optional(ctx,
                      (char *) dkim_sign_headers_expanded,
                      NULL,
index 610c011..c9576f7 100644 (file)
@@ -14,7 +14,7 @@ pdkim.a:         $(OBJ)
 
 base64.o:           $(HDRS) base64.c
 bignum.o:           $(HDRS) bignum.c
-pdkim.o:            $(HDRS) pdkim.c
+pdkim.o:            $(HDRS) pdkim.h pdkim.c
 rsa.o:              $(HDRS) rsa.c
 sha1.o:             $(HDRS) sha1.c
 sha2.o:             $(HDRS) sha2.c