BIND 10 trac505, updated. 32ca390dad48d4d5c8bea98248bdfc62894f87ed [trac505] No authority section on long DNAME

BIND 10 source code commits bind10-changes at lists.isc.org
Fri Feb 4 16:51:32 UTC 2011


The branch, trac505 has been updated
       via  32ca390dad48d4d5c8bea98248bdfc62894f87ed (commit)
      from  f998d613872373c2fbf7088088d7464e79cc10c3 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
commit 32ca390dad48d4d5c8bea98248bdfc62894f87ed
Author: Michal 'vorner' Vaner <michal.vaner at nic.cz>
Date:   Fri Feb 4 17:50:22 2011 +0100

    [trac505] No authority section on long DNAME
    
    It seems the RFC 2672 isn't too clear about it, but all checked
    authoritative servers don't provide the authority section in case of
    YXDOMAIN caused by too long synthesized name.

-----------------------------------------------------------------------

Summary of changes:
 src/bin/auth/query.cc                |    1 -
 src/bin/auth/tests/query_unittest.cc |    4 ++--
 2 files changed, 2 insertions(+), 3 deletions(-)

-----------------------------------------------------------------------
diff --git a/src/bin/auth/query.cc b/src/bin/auth/query.cc
index 38c8180..d7018b1 100644
--- a/src/bin/auth/query.cc
+++ b/src/bin/auth/query.cc
@@ -196,7 +196,6 @@ Query::process() const {
                  */
                 catch (const isc::dns::TooLongName&) {
                     response_.setRcode(Rcode::YXDOMAIN());
-                    getAuthAdditional(*result.zone);
                     return;
                 }
                 // No break; here, fall trough.
diff --git a/src/bin/auth/tests/query_unittest.cc b/src/bin/auth/tests/query_unittest.cc
index 60cc290..90ddbda 100644
--- a/src/bin/auth/tests/query_unittest.cc
+++ b/src/bin/auth/tests/query_unittest.cc
@@ -641,8 +641,8 @@ TEST_F(QueryTest, LongDNAME) {
         "somethingveryveryverylong.longdname.example.com"), RRType::A(),
         response).process());
 
-    responseCheck(response, Rcode::YXDOMAIN(), AA_FLAG, 1, 3, 3,
-        dname_long_txt, zone_ns_txt, ns_addrs_txt);
+    responseCheck(response, Rcode::YXDOMAIN(), AA_FLAG, 1, 0, 0,
+        dname_long_txt, NULL, NULL);
 }
 
 }




More information about the bind10-changes mailing list