SPF: support uppercase in v=spf1, permerror-on-multiple-RRs. Bug 2499
authorJeremy Harris <jgh146exb@wizmail.org>
Sun, 22 Dec 2019 19:56:15 +0000 (19:56 +0000)
committerJeremy Harris <jgh146exb@wizmail.org>
Fri, 27 Dec 2019 13:44:11 +0000 (13:44 +0000)
src/src/spf.c
test/dnszones-src/db.example.com
test/scripts/4600-SPF/4601
test/stdout/4601

index b7041d3e3493b2bd0172eaca2842ae3022657233..3ca289656482fb2217811d2321a2cc389bedd5f4 100644 (file)
@@ -42,85 +42,96 @@ const char *domain, ns_type rr_type, int should_cache)
 dns_answer * dnsa = store_get_dns_answer();
 dns_scan dnss;
 SPF_dns_rr_t * spfrr;
+unsigned found = 0;
+
+SPF_dns_rr_t srr = {
+  .domain = CS domain,                 /* query information */
+  .domain_buf_len = 0,
+  .rr_type = rr_type,
+
+  .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
+};
 
 DEBUG(D_receive) debug_printf("SPF_dns_exim_lookup '%s'\n", domain);
 
-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
-       && (rr_type != T_TXT || Ustrncmp(rr->data+1, "v=spf1", 6) == 0))
+if (dns_lookup(dnsa, US domain, rr_type, NULL) == DNS_NOMATCH)
+  {
+  SPF_dns_rr_dup(&spfrr, spf_nxdomain);
+  return spfrr;
+}
+
+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++;
+
+srr.num_rr = found;
+srr.rr = store_malloc(sizeof(SPF_dns_rr_data_t) * found);
+srr.herrno = h_errno,
+
+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)
       {
-      const uschar * s = rr->data;
-      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
-      };
-
-      switch(rr_type)
+      case T_MX:
+       s += 2;                 /* skip the MX precedence field */
+      case T_PTR:
        {
-        case T_MX:
-          s += 2;                      /* skip the MX precedence field */
-        case T_PTR:
-         {
-          uschar * buf = store_malloc(256);
-          (void)dn_expand(dnsa->answer, dnsa->answer + dnsa->answerlen, s,
-            (DN_EXPAND_ARG4_TYPE)buf, 256);
-          s = buf;
-          break;
-         }
+       uschar * buf = store_malloc(256);
+       (void)dn_expand(dnsa->answer, dnsa->answer + dnsa->answerlen, s,
+         (DN_EXPAND_ARG4_TYPE)buf, 256);
+       s = buf;
+       break;
+       }
 
-        case T_TXT:
+      case T_TXT:
+       {
+       gstring * g = NULL;
+       uschar chunk_len;
+
+       if (strncmpic(rr->data+1, US"v=spf1", 6) != 0)
          {
-         gstring * g = NULL;
-         uschar chunk_len;
-          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)
-            {
-            HDEBUG(D_host_lookup) debug_printf("IP address lookup yielded an "
-              "empty name: treated as non-existent host name\n");
-            continue;
-            }
-          gstring_release_unused(g);
-          s = string_copy_malloc(string_from_gstring(g));
-          break;
+         HDEBUG(D_host_lookup) debug_printf("not an spf record\n");
+         continue;
          }
 
-        case T_A:
-        case T_AAAA:
-        default:
+       for (int off = 0; off < rr->size; off += chunk_len)
          {
-          uschar * buf = store_malloc(dnsa->answerlen + 1);
-         s = memcpy(buf, s, dnsa->answerlen + 1);
-          break;
+         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;
        }
-      DEBUG(D_receive) debug_printf("SPF_dns_exim_lookup '%s'\n", s);
-      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);
+/* spfrr->rr must have been malloc()d for this */
+SPF_dns_rr_dup(&spfrr, &srr);
 return spfrr;
 }
 
@@ -129,14 +140,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;
index 683772f773189f11ccfe89933d4b7e21ad9292c2..6b35113e2a24c5056a30e1c2d1103b470b566f22 100644 (file)
@@ -25,6 +25,11 @@ example.com.     NS      exim.example.com.
 
 example.com.   TXT     v=spf1 -all
 
+double         TXT     v=spf1 include:_spf.google.com ~all
+               TXT     v=spf1 +a +mx -all
+
+uppercase      TXT     v=sPf1 +all
+
 ; Alias A record for the local host, under the name "server1"
 
 server1     A       HOSTIPV4
index ab434611c962e52de987495434baf1dea9c440ec..6012cfdebd15b970fa7c78c564d32e507d0eedf2 100644 (file)
@@ -1,12 +1,5 @@
 # lookup string-expansion
 #
-# It is rather difficult to properly test spf.  We use libspf2 to do the work, and it
-# does the DNS lookups, so we cannot intercept them in the testsuite's usual fashion
-# to provide values for testcases.
-#
-# For now just check that what should be working syntax does not cause us to fall over.
-# Be careful with envelope-domains used for testcases, as real DNS lookups will be done.
-#
 exim -bd -DSERVER=server -oX PORT_D:PORT_S
 ****
 client 127.0.0.1 PORT_D
@@ -31,3 +24,9 @@ quit
 ****
 #
 killdaemon
+#
+# A multiple spf-RR return should get permerror
+exim -be
+${lookup {fred@double.example.com} spf {8.8.8.8}}
+${lookup {fred@uppercase.example.com} spf {HOSTIPV4}}
+****
index 66746a4702abe19db0090eb74544a28406e6c2e3..9f69eb29c85142d6388198815b6397cbb9b0d1e8 100644 (file)
@@ -26,3 +26,6 @@ Connecting to 127.0.0.1 port 1224 ... connected
 <<< 250 Accepted
 >>> quit
 End of script
+> permerror
+> pass
+>