Handle IPv6 addresses with SPF.
authorPhil Pennock <pdp@exim.org>
Sun, 28 Aug 2011 20:35:03 +0000 (16:35 -0400)
committerPhil Pennock <pdp@exim.org>
Sun, 28 Aug 2011 20:35:03 +0000 (16:35 -0400)
Patch from Wolfgang Breyha.

Also, slight improvements to some debug statements.

fixes bug 860

doc/doc-txt/ChangeLog
src/src/spf.c

index 23a18cf2f9b93f65d4ecf0d29c118308fcc2e667..b0d6b06afb46ff12972dd05f437707ca9b247e38 100644 (file)
@@ -109,6 +109,9 @@ PP/07 Make maildir_use_size_file an _expandable_ boolean.
 PP/08 Handle ${run} returning more data than OS pipe buffer size.
       Bugzilla 1131.  Patch from Holger Weiß.
 
 PP/08 Handle ${run} returning more data than OS pipe buffer size.
       Bugzilla 1131.  Patch from Holger Weiß.
 
+PP/09 Handle IPv6 addresses with SPF.
+      Bugzilla 860.  Patch from Wolfgang Breyha.
+
 
 Exim version 4.76
 -----------------
 
 Exim version 4.76
 -----------------
index 6be339390a4abc127785f361b4449cf4a2fa9c3c..4bbabbf4c7ad387c0455ec0ad2a1c86d67f28a8c 100644 (file)
@@ -42,22 +42,23 @@ int spf_init(uschar *spf_helo_domain, uschar *spf_remote_addr) {
   }
 
   if (SPF_server_set_rec_dom(spf_server, CS primary_hostname)) {
   }
 
   if (SPF_server_set_rec_dom(spf_server, CS primary_hostname)) {
-    debug_printf("spf: SPF_server_set_rec_dom() failed.\n");
+    debug_printf("spf: SPF_server_set_rec_dom(\"%s\") failed.\n", primary_hostname);
     spf_server = NULL;
     return 0;
   }
 
   spf_request = SPF_request_new(spf_server);
 
     spf_server = NULL;
     return 0;
   }
 
   spf_request = SPF_request_new(spf_server);
 
-  if (SPF_request_set_ipv4_str(spf_request, CS spf_remote_addr)) {
-    debug_printf("spf: SPF_request_set_ipv4_str() failed.\n");
+  if (SPF_request_set_ipv4_str(spf_request, CS spf_remote_addr)
+      && SPF_request_set_ipv6_str(spf_request, CS spf_remote_addr)) {
+    debug_printf("spf: SPF_request_set_ipv4_str() and SPF_request_set_ipv6_str() failed [%s]\n", spf_remote_addr);
     spf_server = NULL;
     spf_request = NULL;
     return 0;
   }
 
   if (SPF_request_set_helo_dom(spf_request, CS spf_helo_domain)) {
     spf_server = NULL;
     spf_request = NULL;
     return 0;
   }
 
   if (SPF_request_set_helo_dom(spf_request, CS spf_helo_domain)) {
-    debug_printf("spf: SPF_set_helo_dom() failed.\n");
+    debug_printf("spf: SPF_set_helo_dom(\"%s\") failed.\n", spf_helo_domain);
     spf_server = NULL;
     spf_request = NULL;
     return 0;
     spf_server = NULL;
     spf_request = NULL;
     return 0;