SPF: fix handling mix of spf and other txt records. Bug 2499
[exim.git] / src / src / spf.c
index cf8176e8e755ac5ba88336bd7277919aad56ea1a..8ead817b95674455c7559f73a4f3622c2a8bde76 100644 (file)
@@ -37,62 +37,111 @@ SPF_dns_rr_t  * spf_nxdomain = NULL;
 
 static SPF_dns_rr_t *
 SPF_dns_exim_lookup(SPF_dns_server_t *spf_dns_server,
-const char *domain, ns_type rr_type, int should_cache)
+  const char *domain, ns_type rr_type, int should_cache)
 {
-dns_answer dnsa;
+dns_answer * dnsa = store_get_dns_answer();
 dns_scan dnss;
 SPF_dns_rr_t * spfrr;
+unsigned found = 0;
 
-DEBUG(D_receive) debug_printf("SPF_dns_exim_lookup\n");
+SPF_dns_rr_t srr = {
+  .domain = CS domain,                 /* query information */
+  .domain_buf_len = 0,
+  .rr_type = rr_type,
 
-if (dns_lookup(&dnsa, US domain, rr_type, NULL) == DNS_SUCCEED)
-  for (dns_record * rr = dns_next_rr(&dnsa, &dnss, RESET_ANSWERS); rr;
-       rr = dns_next_rr(&dnsa, &dnss, RESET_NEXT))
-    if (  rr->type == rr_type
-       && Ustrncmp(rr->data+1, "v=spf1", 6) == 0)
+  .rr_buf_len = 0,                     /* answer information */
+  .rr_buf_num = 0, /* no free of s */
+  .utc_ttl = 0,
+
+  .hook = NULL,                                /* misc information */
+  .source = spf_dns_server
+};
+int dns_rc;
+
+DEBUG(D_receive) debug_printf("SPF_dns_exim_lookup '%s'\n", domain);
+
+switch (dns_rc = dns_lookup(dnsa, US domain, rr_type, NULL))
+  {
+  case DNS_SUCCEED:    srr.herrno = NETDB_SUCCESS;     break;
+  case DNS_AGAIN:      srr.herrno = TRY_AGAIN;         break;
+  case DNS_NOMATCH:    srr.herrno = HOST_NOT_FOUND;    break;
+  case DNS_NODATA:     srr.herrno = NO_DATA;           break;
+  case DNS_FAIL:
+  default:             srr.herrno = NO_RECOVERY;       break;
+  } 
+
+for (dns_record * rr = dns_next_rr(dnsa, &dnss, RESET_ANSWERS); rr;
+     rr = dns_next_rr(dnsa, &dnss, RESET_NEXT))
+  if (rr->type == rr_type) found++;
+
+if (found == 0)
+  {
+  SPF_dns_rr_dup(&spfrr, &srr);
+  return spfrr;
+  }
+
+srr.rr = store_malloc(sizeof(SPF_dns_rr_data_t) * found);
+
+found = 0;
+for (dns_record * rr = dns_next_rr(dnsa, &dnss, RESET_ANSWERS); rr;
+   rr = dns_next_rr(dnsa, &dnss, RESET_NEXT))
+  if (rr->type == rr_type)
+    {
+    const uschar * s = rr->data;
+
+    srr.ttl = rr->ttl;
+    switch(rr_type)
       {
-      gstring * g = NULL;
-      uschar chunk_len;
-      uschar * s;
-      SPF_dns_rr_t srr = {
-       .domain = CS rr->name,                  /* query information */
-       .domain_buf_len = DNS_MAXNAME,
-       .rr_type = rr->type,
-
-       .num_rr = 1,                            /* answer information */
-       .rr = NULL,
-       .rr_buf_len = 0,
-       .rr_buf_num = 0,
-       .ttl = rr->ttl,
-       .utc_ttl = 0,
-       .herrno = NETDB_SUCCESS,
-
-       .hook = NULL,                           /* misc information */
-       .source = spf_dns_server
-      };
-
-      for (int off = 0; off < rr->size; off += chunk_len)
+      case T_MX:
+       s += 2; /* skip the MX precedence field */
+      case T_PTR:
        {
-       chunk_len = (rr->data)[off++];
-       g = string_catn(g, US ((rr->data)+off), chunk_len);
+       uschar * buf = store_malloc(256);
+       (void)dn_expand(dnsa->answer, dnsa->answer + dnsa->answerlen, s,
+         (DN_EXPAND_ARG4_TYPE)buf, 256);
+       s = buf;
+       break;
        }
-      if (!g)
+
+      case T_TXT:
        {
-       HDEBUG(D_host_lookup) debug_printf("IP address lookup yielded an "
-         "empty name: treated as non-existent host name\n");
-       continue;
+       gstring * g = NULL;
+       uschar chunk_len;
+
+       if (strncmpic(rr->data+1, US"v=spf1", 6) != 0)
+         {
+         HDEBUG(D_host_lookup) debug_printf("not an spf record\n");
+         continue;
+         }
+
+       for (int off = 0; off < rr->size; off += chunk_len)
+         {
+         if (!(chunk_len = s[off++])) break;
+         g = string_catn(g, s+off, chunk_len);
+         }
+       if (!g)
+         continue;
+       gstring_release_unused(g);
+       s = string_copy_malloc(string_from_gstring(g));
+       break;
        }
-      gstring_release_unused(g);
-      s = string_copy_malloc(string_from_gstring(g));
-      srr.rr = (void *) &s;
-
-      /* spfrr->rr must have been malloc()d for this */
-      SPF_dns_rr_dup(&spfrr, &srr);
 
-      return spfrr;
+      case T_A:
+      case T_AAAA:
+      default:
+       {
+       uschar * buf = store_malloc(dnsa->answerlen + 1);
+       s = memcpy(buf, s, dnsa->answerlen + 1);
+       break;
+       }
       }
+    DEBUG(D_receive) debug_printf("SPF_dns_exim_lookup '%s'\n", s);
+    srr.rr[found++] = (void *) s;
+    }
 
-SPF_dns_rr_dup(&spfrr, spf_nxdomain);
+srr.num_rr = found;
+/* spfrr->rr must have been malloc()d for this */
+SPF_dns_rr_dup(&spfrr, &srr);
 return spfrr;
 }
 
@@ -101,14 +150,11 @@ return spfrr;
 SPF_dns_server_t *
 SPF_dns_exim_new(int debug)
 {
-SPF_dns_server_t *spf_dns_server;
+SPF_dns_server_t * spf_dns_server = store_malloc(sizeof(SPF_dns_server_t));
 
 DEBUG(D_receive) debug_printf("SPF_dns_exim_new\n");
 
-if (!(spf_dns_server = malloc(sizeof(SPF_dns_server_t))))
-  return NULL;
 memset(spf_dns_server, 0, sizeof(SPF_dns_server_t));
-
 spf_dns_server->destroy      = NULL;
 spf_dns_server->lookup       = SPF_dns_exim_lookup;
 spf_dns_server->get_spf      = NULL;
@@ -133,25 +179,18 @@ return spf_dns_server;
 
 
 
-/* spf_init sets up a context that can be re-used for several
-   messages on the same SMTP connection (that come from the
-   same host with the same HELO string).
-XXX the spf_server layer could usefully be separately init'd
-given that it sets up a dns cache.
 
-Return: Boolean success */
+/* Construct the SPF library stack.
+   Return: Boolean success.
+*/
 
 BOOL
-spf_init(uschar *spf_helo_domain, uschar *spf_remote_addr)
+spf_init(void)
 {
-int debug = 0;
 SPF_dns_server_t * dc;
+int debug = 0;
 
-DEBUG(D_receive)
-  {
-  debug_printf("spf_init: %s %s\n", spf_helo_domain, spf_remote_addr);
-  debug = 1;
-  }
+DEBUG(D_receive) debug = 1;
 
 /* We insert our own DNS access layer rather than letting the spf library
 do it, so that our dns access path is used for debug tracing and for the
@@ -172,6 +211,30 @@ if (!(spf_server = SPF_server_new_dns(dc, debug)))
   DEBUG(D_receive) debug_printf("spf: SPF_server_new() failed.\n");
   return FALSE;
   }
+  /* Quick hack to override the outdated explanation URL.
+  See https://www.mail-archive.com/mailop@mailop.org/msg08019.html */
+  SPF_server_set_explanation(spf_server, "Please%_see%_http://www.open-spf.org/Why?id=%{S}&ip=%{C}&receiver=%{R}", &spf_response);
+  if (SPF_response_errcode(spf_response) != SPF_E_SUCCESS)
+    log_write(0, LOG_MAIN|LOG_PANIC_DIE, "%s", SPF_strerror(SPF_response_errcode(spf_response)));
+
+return TRUE;
+}
+
+
+/* Set up a context that can be re-used for several
+   messages on the same SMTP connection (that come from the
+   same host with the same HELO string).
+
+Return: Boolean success
+*/
+
+BOOL
+spf_conn_init(uschar * spf_helo_domain, uschar * spf_remote_addr)
+{
+DEBUG(D_receive)
+  debug_printf("spf_conn_init: %s %s\n", spf_helo_domain, spf_remote_addr);
+
+if (!spf_server && !spf_init()) return FALSE;
 
 if (SPF_server_set_rec_dom(spf_server, CS primary_hostname))
   {