]> git.ipfire.org Git - thirdparty/bind9.git/commitdiff
projects / thirdparty / bind9.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 96beefd)
4387. [bug] Change 4336 was not complete leading to SERVFAIL
authorMark Andrews <marka@isc.org>
Wed, 22 Jun 2016 05:26:38 +0000 (15:26 +1000)
committerMark Andrews <marka@isc.org>
Wed, 22 Jun 2016 05:26:38 +0000 (15:26 +1000)
                        being return as NS records expired. [RT #42683]

CHANGES patch | blob | blame | history
bin/tests/system/zero/ns1/root.db patch | blob | blame | history
bin/tests/system/zero/ns2/named.conf patch | blob | blame | history
bin/tests/system/zero/ns2/tld.db [new file with mode: 0644] patch | blob
bin/tests/system/zero/ns4/named.conf patch | blob | blame | history
bin/tests/system/zero/ns4/one.tld.db [new file with mode: 0644] patch | blob
bin/tests/system/zero/tests.sh patch | blob | blame | history
lib/dns/rbtdb.c patch | blob | blame | history

diff --git a/CHANGES b/CHANGES
index e374cc66cc6882a845e4e085254f7209eef82765..e2db12abd37a3fa0acdb3aa36ab47c57784d2272 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,3 +1,6 @@
+4387.  [bug]           Change 4336 was not complete leading to SERVFAIL
+                       being return as NS records expired. [RT #42683]
+
 4386.  [bug]           Remove shadowed overmem function/variable. [RT #42706]
 
 4385.  [func]          Add support for allow-query and allow-transfer ACLs
diff --git a/bin/tests/system/zero/ns1/root.db b/bin/tests/system/zero/ns1/root.db
index 7a6c6672e24209a6ced315c6a4b916842936686c..c46bf2a78463592a36052707afeb0399a8d07ec8 100644 (file)
--- a/bin/tests/system/zero/ns1/root.db
+++ b/bin/tests/system/zero/ns1/root.db
@@ -23,4 +23,7 @@ ns2.example. A 10.53.0.2
 example. NS ns4.example.
 ns4.example. A 10.53.0.4
 increment. NS incrementns.
-incrementns A 10.53.0.5
+incrementns. A 10.53.0.5
+tld. NS ns2.tld.
+ns2.tld. A 10.53.0.2
+
diff --git a/bin/tests/system/zero/ns2/named.conf b/bin/tests/system/zero/ns2/named.conf
index 86673b218874d29d8877c6caff2be47602d68d2d..c1d2e6a6471bda24bfc61e1e8c547a39fae0e275 100644 (file)
--- a/bin/tests/system/zero/ns2/named.conf
+++ b/bin/tests/system/zero/ns2/named.conf
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2013  Internet Systems Consortium, Inc. ("ISC")
+ * Copyright (C) 2013, 2016  Internet Systems Consortium, Inc. ("ISC")
  *
  * Permission to use, copy, modify, and/or distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -35,3 +35,7 @@ zone "example" {
        file "example.db";
 };
 
+zone "tld" {
+       type master;
+       file "tld.db";
+};
diff --git a/bin/tests/system/zero/ns2/tld.db b/bin/tests/system/zero/ns2/tld.db
new file mode 100644 (file)
index 0000000..f5e987e
--- /dev/null
+++ b/bin/tests/system/zero/ns2/tld.db
@@ -0,0 +1,23 @@
+; Copyright (C) 2016  Internet Systems Consortium, Inc. ("ISC")
+;
+; Permission to use, copy, modify, and/or distribute this software for any
+; purpose with or without fee is hereby granted, provided that the above
+; copyright notice and this permission notice appear in all copies.
+;
+; THE SOFTWARE IS PROVIDED "AS IS" AND ISC DISCLAIMS ALL WARRANTIES WITH
+; REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY
+; AND FITNESS.  IN NO EVENT SHALL ISC BE LIABLE FOR ANY SPECIAL, DIRECT,
+; INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM
+; LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE
+; OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
+; PERFORMANCE OF THIS SOFTWARE.
+
+$TTL 1
+@      300     SOA     ns2.tld. hostmaster.ns2.tld. 0 1 1 1 1
+@      300     NS      ns2.tld.
+ns2    300     A       10.53.0.2
+;
+;      The TTL of these delegation records needs to 1.
+;
+one    1       NS      ns4.one.tld.
+ns4.one        1       A       10.53.0.4
diff --git a/bin/tests/system/zero/ns4/named.conf b/bin/tests/system/zero/ns4/named.conf
index bceeb231175e104cd71b63262ceb875e7af07441..3cf95a59b253a7b8d1c09fbc65d0af76a0d8859e 100644 (file)
--- a/bin/tests/system/zero/ns4/named.conf
+++ b/bin/tests/system/zero/ns4/named.conf
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2013  Internet Systems Consortium, Inc. ("ISC")
+ * Copyright (C) 2013, 2016  Internet Systems Consortium, Inc. ("ISC")
  *
  * Permission to use, copy, modify, and/or distribute this software for any
  * purpose with or without fee is hereby granted, provided that the above
@@ -36,3 +36,7 @@ zone "example" {
        file "example.bk";
 };
 
+zone "one.tld" {
+       type master;
+       file "one.tld.db";
+};
diff --git a/bin/tests/system/zero/ns4/one.tld.db b/bin/tests/system/zero/ns4/one.tld.db
new file mode 100644 (file)
index 0000000..9442309
--- /dev/null
+++ b/bin/tests/system/zero/ns4/one.tld.db
@@ -0,0 +1,20 @@
+; Copyright (C) 2016  Internet Systems Consortium, Inc. ("ISC")
+;
+; Permission to use, copy, modify, and/or distribute this software for any
+; purpose with or without fee is hereby granted, provided that the above
+; copyright notice and this permission notice appear in all copies.
+;
+; THE SOFTWARE IS PROVIDED "AS IS" AND ISC DISCLAIMS ALL WARRANTIES WITH
+; REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF MERCHANTABILITY
+; AND FITNESS.  IN NO EVENT SHALL ISC BE LIABLE FOR ANY SPECIAL, DIRECT,
+; INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES WHATSOEVER RESULTING FROM
+; LOSS OF USE, DATA OR PROFITS, WHETHER IN AN ACTION OF CONTRACT, NEGLIGENCE
+; OR OTHER TORTIOUS ACTION, ARISING OUT OF OR IN CONNECTION WITH THE USE OR
+; PERFORMANCE OF THIS SOFTWARE.
+
+$TTL 1
+;      The TTL of all these records needs to be 1.
+@      1       SOA     ns4.one.tld. hostmaster.ns4.tld. 0 1 1 1 1
+@      1       NS      ns4.one.tld.
+ns4    1       A       10.53.0.4
+www    1       A       10.53.0.4
diff --git a/bin/tests/system/zero/tests.sh b/bin/tests/system/zero/tests.sh
index b11f7de85391c5895c10c9f53d51b57329475c7a..0c52be2955e629940ffca0d6c059a659f780aa35 100644 (file)
--- a/bin/tests/system/zero/tests.sh
+++ b/bin/tests/system/zero/tests.sh
@@ -16,7 +16,10 @@ SYSTEMTESTTOP=..
 . $SYSTEMTESTTOP/conf.sh
 
 status=0
-echo "I:check lookups against zero TTL records"
+n=0
+
+n=`expr $n + 1`
+echo "I:check lookups against TTL=0 records ($n)"
 i=0
 passes=10
 $DIG -p 5300 @10.53.0.2 axfr example | 
@@ -24,19 +27,19 @@ awk '$2 == "0" { print "-q", $1, $4; print "-q", "zzz"$1, $4;}' > query.list
 while [ $i -lt $passes ]
 do
        ret=0
-       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.1 &
-       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.2 &
-       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.3 &
-       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.4 &
-       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.5 &
-       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.6 &
+       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.1.test$n &
+       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.2.test$n &
+       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.3.test$n &
+       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.4.test$n &
+       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.5.test$n &
+       $DIG -p 5300 @10.53.0.3 -f query.list > dig.out$i.6.test$n &
        wait
-       grep "status: SERVFAIL" dig.out$i.1 && ret=1
-       grep "status: SERVFAIL" dig.out$i.2 && ret=1
-       grep "status: SERVFAIL" dig.out$i.3 && ret=1
-       grep "status: SERVFAIL" dig.out$i.5 && ret=1
-       grep "status: SERVFAIL" dig.out$i.6 && ret=1
-       grep "status: SERVFAIL" dig.out$i.6 && ret=1
+       grep "status: SERVFAIL" dig.out$i.1.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.2.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.3.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.4.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.5.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.6.test$n && ret=1
        [ $ret = 1 ] && break
        i=`expr $i + 1`
        echo "I: successfully completed pass $i of $passes"
@@ -44,7 +47,8 @@ done
 if [ $ret != 0 ]; then echo "I:failed"; fi
 status=`expr $status + $ret`
 
-echo "I:check repeated recursive lookups of non recurring zero ttl responses get new values"
+n=`expr $n + 1`
+echo "I:check repeated recursive lookups of non recurring TTL=0 responses get new values ($n)"
 count=`(
 $DIG +short -p 5300 @10.53.0.3 foo.increment
 $DIG +short -p 5300 @10.53.0.3 foo.increment
@@ -57,5 +61,32 @@ $DIG +short -p 5300 @10.53.0.3 foo.increment
 if [ $count -ne 7 ] ; then echo "I:failed (count=$count)"; ret=1; fi
 status=`expr $status + $ret`
 
+n=`expr $n + 1`
+echo "I:check lookups against TTL=1 records ($n)"
+i=0
+passes=10
+while [ $i -lt $passes ]
+do
+       ret=0
+       $DIG -p 5300 @10.53.0.3 www.one.tld > dig.out$i.1.test$n
+       $DIG -p 5300 @10.53.0.3 www.one.tld > dig.out$i.2.test$n
+       $DIG -p 5300 @10.53.0.3 www.one.tld > dig.out$i.3.test$n
+       $DIG -p 5300 @10.53.0.3 www.one.tld > dig.out$i.4.test$n
+       $DIG -p 5300 @10.53.0.3 www.one.tld > dig.out$i.5.test$n
+       $DIG -p 5300 @10.53.0.3 www.one.tld > dig.out$i.6.test$n
+       grep "status: SERVFAIL" dig.out$i.1.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.2.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.3.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.4.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.5.test$n && ret=1
+       grep "status: SERVFAIL" dig.out$i.6.test$n && ret=1
+       [ $ret = 1 ] && break
+       i=`expr $i + 1`
+       echo "I: successfully completed pass $i of $passes"
+       $PERL -e 'select(undef, undef, undef, 0.3);'
+done
+if [ $ret != 0 ]; then echo "I:failed"; fi
+status=`expr $status + $ret`
+
 echo "I:exit status: $status"
 [ $status -eq 0 ] || exit 1
diff --git a/lib/dns/rbtdb.c b/lib/dns/rbtdb.c
index 0deb37fe72b085ccc698911268377ab5cddc07b7..7b27b09fd2b46e8a3675922f84f5e9385d6d5728 100644 (file)
--- a/lib/dns/rbtdb.c
+++ b/lib/dns/rbtdb.c
@@ -530,6 +530,10 @@ struct acachectl {
 #define ZEROTTL(header) \
        (((header)->attributes & RDATASET_ATTR_ZEROTTL) != 0)
 
+#define ACTIVE(header, now) \
+       (((header)->rdh_ttl > (now)) || \
+        ((header)->rdh_ttl == (now) && ZEROTTL(header)))
+
 #define DEFAULT_NODE_LOCK_COUNT         7       /*%< Should be prime. */
 
 /*%
@@ -4588,8 +4592,7 @@ check_stale_rdataset(dns_rbtnode_t *node, rdatasetheader_t *header,
        UNUSED(lock);
 #endif
 
-       if (header->rdh_ttl < search->now ||
-           (header->rdh_ttl == search->now && !ZEROTTL(header))) {
+       if (!ACTIVE(header, search->now)) {
                /*
                 * This rdataset is stale.  If no one else is using the
                 * node, we can clean it up right now, otherwise we mark
@@ -5795,7 +5798,7 @@ cache_findrdataset(dns_db_t *db, dns_dbnode_t *node, dns_dbversion_t *version,
 
        for (header = rbtnode->data; header != NULL; header = header_next) {
                header_next = header->next;
-               if (header->rdh_ttl < now) {
+               if (!ACTIVE(header, now)) {
                        if ((header->rdh_ttl < now - RBTDB_VIRTUAL) &&
                            (locktype == isc_rwlocktype_write ||
                             NODE_TRYUPGRADE(lock) == ISC_R_SUCCESS)) {
@@ -6119,7 +6122,7 @@ add32(dns_rbtdb_t *rbtdb, dns_rbtnode_t *rbtnode, rbtdb_version_t *rbtversion,
                                        }
                        }
                        if (topheader != NULL && EXISTS(topheader) &&
-                           topheader->rdh_ttl >= now) {
+                           ACTIVE(topheader, now)) {
                                /*
                                 * Found one.
                                 */
@@ -6184,7 +6187,7 @@ add32(dns_rbtdb_t *rbtdb, dns_rbtnode_t *rbtnode, rbtdb_version_t *rbtversion,
                 * has no effect, provided that the cache data isn't stale.
                 */
                if (rbtversion == NULL && trust < header->trust &&
-                   (header->rdh_ttl >= now || header_nx)) {
+                   (ACTIVE(header, now) || header_nx)) {
                        free_rdataset(rbtdb, rbtdb->common.mctx, newheader);
                        if (addedrdataset != NULL)
                                bind_rdataset(rbtdb, rbtnode, header, now,
@@ -6258,7 +6261,7 @@ add32(dns_rbtdb_t *rbtdb, dns_rbtnode_t *rbtnode, rbtdb_version_t *rbtversion,
                 * Don't lower trust of existing record if the
                 * update is forced.
                 */
-               if (IS_CACHE(rbtdb) && header->rdh_ttl >= now &&
+               if (IS_CACHE(rbtdb) && ACTIVE(header, now) &&
                    header->type == dns_rdatatype_ns &&
                    !header_nx && !newheader_nx &&
                    header->trust >= newheader->trust &&
@@ -6294,7 +6297,7 @@ add32(dns_rbtdb_t *rbtdb, dns_rbtnode_t *rbtnode, rbtdb_version_t *rbtversion,
                 * to be no more than the current NS RRset's TTL.  This
                 * ensures the delegations that are withdrawn are honoured.
                 */
-               if (IS_CACHE(rbtdb) && header->rdh_ttl >= now &&
+               if (IS_CACHE(rbtdb) && ACTIVE(header, now) &&
                    header->type == dns_rdatatype_ns &&
                    !header_nx && !newheader_nx &&
                    header->trust <= newheader->trust) {
@@ -6302,7 +6305,7 @@ add32(dns_rbtdb_t *rbtdb, dns_rbtnode_t *rbtnode, rbtdb_version_t *rbtversion,
                                newheader->rdh_ttl = header->rdh_ttl;
                        }
                }
-               if (IS_CACHE(rbtdb) && header->rdh_ttl >= now &&
+               if (IS_CACHE(rbtdb) && ACTIVE(header, now) &&
                    (options & DNS_DBADD_PREFETCH) == 0 &&
                    (header->type == dns_rdatatype_a ||
                     header->type == dns_rdatatype_aaaa ||
Mirror of https://gitlab.isc.org/isc-projects/bind9.git