Check dnsdb PTR key for IP address before playing the reversing game.

diff --git a/doc/doc-txt/ChangeLog b/doc/doc-txt/ChangeLog
index 021b61bcc2b479ba27239894249116ceb6fd6ada..2b75b4656adc2f81255f6567b4ea4ab851920498 100644 (file)
--- a/doc/doc-txt/ChangeLog
+++ b/doc/doc-txt/ChangeLog
@@ -1,4 +1,4 @@
-$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.52 2004/12/21 12:00:59 ph10 Exp $
+$Cambridge: exim/doc/doc-txt/ChangeLog,v 1.53 2004/12/21 12:21:46 ph10 Exp $

 
 Change log file for Exim from version 4.21
 -------------------------------------------
 
 Change log file for Exim from version 4.21
 -------------------------------------------


@@ -234,6 +234,11 @@ Exim version 4.50
       But simultaneously result of request was absolutely normal ldap result,
       so I produce this patch..."
 
       But simultaneously result of request was absolutely normal ldap result,
       so I produce this patch..."
 

+55. Some experimental protocols are using DNS PTR records for new purposes. The
+    keys for these records are domain names, not reversed IP addresses. The
+    dnsdb lookup now tests whether it's key is an IP address. If not, it leaves
+    it alone. Component reversal etc. now happens only for IP addresses.
+

 
 Exim version 4.43
 -----------------
 
 Exim version 4.43
 -----------------

diff --git a/doc/doc-txt/NewStuff b/doc/doc-txt/NewStuff
index 3109ea96faab3cc02de663b835ce48d16058727a..11d7b3cc31a2ff0cf4355beb7c5aa97d4d3633e8 100644 (file)
--- a/doc/doc-txt/NewStuff
+++ b/doc/doc-txt/NewStuff
@@ -1,4 +1,4 @@
-$Cambridge: exim/doc/doc-txt/NewStuff,v 1.21 2004/12/20 15:24:27 ph10 Exp $
+$Cambridge: exim/doc/doc-txt/NewStuff,v 1.22 2004/12/21 12:21:46 ph10 Exp $

 
 New Features in Exim
 --------------------
 
 New Features in Exim
 --------------------


@@ -234,6 +234,10 @@ Version 4.50
 19. The Exiscan patch is now merged into the main source. See src/EDITME for
     parameters for the build.
 
 19. The Exiscan patch is now merged into the main source. See src/EDITME for
     parameters for the build.
 

+20. If the key for a dnsdb lookup is not an IP address, it is used verbatim,
+    without component reversal and without the addition of in-addr.arpa or
+    ip6.arpa.
+

 
 Version 4.43
 ------------
 
 Version 4.43
 ------------

diff --git a/src/src/lookups/dnsdb.c b/src/src/lookups/dnsdb.c
index 9cd8612b10c35fef6bb16c1ab6374f90720cc8fb..756247d35185fc3861b62f676413a8aa84e1c949 100644 (file)
--- a/src/src/lookups/dnsdb.c
+++ b/src/src/lookups/dnsdb.c
@@ -1,4 +1,4 @@
-/* $Cambridge: exim/src/src/lookups/dnsdb.c,v 1.6 2004/12/20 15:24:28 ph10 Exp $ */
+/* $Cambridge: exim/src/src/lookups/dnsdb.c,v 1.7 2004/12/21 12:21:46 ph10 Exp $ */

 
 /*************************************************
 *     Exim - an Internet mail transport agent    *
 
 /*************************************************
 *     Exim - an Internet mail transport agent    *


@@ -231,10 +231,12 @@ while ((domain = string_nextinlist(&keystring, &sep, buffer, sizeof(buffer)))
   int searchtype = (type == T_ZNS)? T_NS :          /* record type we want */
                    (type == T_MXH)? T_MX : type; 
 
   int searchtype = (type == T_ZNS)? T_NS :          /* record type we want */
                    (type == T_MXH)? T_MX : type; 
 

-  /* If the type is PTR, we have to construct the relevant magic lookup
-  key. This code is now in a separate function. */
+  /* If the type is PTR, we have to construct the relevant magic lookup key if
+  the original is an IP address (some experimental protocols are using PTR
+  records for different purposes where the key string is a host name). This
+  code for doing the reversal is now in a separate function. */

   
   

-  if (type == T_PTR)
+  if (type == T_PTR && string_is_ip_address(domain, NULL)) 

     {
     dns_build_reverse(domain, rbuffer);
     domain = rbuffer;
     {
     dns_build_reverse(domain, rbuffer);
     domain = rbuffer;