From: Jeremy Harris <jgh146exb@wizmail.org>
Date: Thu, 20 Nov 2014 20:17:32 +0000 (+0000)
Subject: When following a CNAME chain, if any lookup is insecure the whole must be too
X-Git-Tag: exim-4_91_RC1~148^2~42^2~66
X-Git-Url: https://git.exim.org/exim.git/commitdiff_plain/734076a52c94c290826224e524f49be07d4476e1

When following a CNAME chain, if any lookup is insecure the whole must be too
---

diff --git a/src/src/dns.c b/src/src/dns.c
index ac52e5621..542354db3 100644
--- a/src/src/dns.c
+++ b/src/src/dns.c
@@ -455,6 +455,14 @@ return h->ad ? TRUE : FALSE;
 #endif
 }
 
+static void
+dns_set_insecure(dns_answer * dnsa)
+{
+HEADER * h = (HEADER *)dnsa->answer;
+h->ad = 0;
+}
+
+
 
 
 
@@ -752,7 +760,8 @@ int
 dns_lookup(dns_answer *dnsa, uschar *name, int type, uschar **fully_qualified_name)
 {
 int i;
-uschar *orig_name = name;
+const uschar *orig_name = name;
+BOOL secure_so_far = TRUE;
 
 /* Loop to follow CNAME chains so far, but no further... */
 
@@ -825,6 +834,9 @@ for (i = 0; i < 10; i++)
   if (datalen < 0) return DNS_FAIL;
   name = data;
 
+  if (!dns_is_secure(dnsa))
+    secure_so_far = FALSE;
+
   DEBUG(D_dns) debug_printf("CNAME found: change to %s\n", name);
   }       /* Loop back to do another lookup */