fix chaosnet address comparison

(cherry picked from commit b4dca44ad268a1f3ecca097e4076160957779048)

diff --git a/CHANGES b/CHANGES
index df0e1c074d7eb8325996979e6c566fa97db10a26..607e772929bfc8ffe4458711e4d0d06405c91c5d 100644 (file)
--- a/CHANGES
+++ b/CHANGES
@@ -1,3 +1,6 @@
+5038.  [bug]           Chaosnet addresses were compared incorrectly.
+                       [GL #562]
+
 5035.  [test]          Fixed errors that prevented the DNSRPS subtests
                        from running in the rpz and rpzrecurse system
                        tests. [GL #503]

diff --git a/bin/tests/system/auth/ns1/chaos.db b/bin/tests/system/auth/ns1/chaos.db
new file mode 100644 (file)
index 0000000..758f4f6
--- /dev/null
+++ b/bin/tests/system/auth/ns1/chaos.db
@@ -0,0 +1,21 @@
+; Copyright (C) Internet Systems Consortium, Inc. ("ISC")
+;
+; This Source Code Form is subject to the terms of the Mozilla Public
+; License, v. 2.0. If a copy of the MPL was not distributed with this
+; file, You can obtain one at http://mozilla.org/MPL/2.0/.
+;
+; See the COPYRIGHT file distributed with this work for additional
+; information regarding copyright ownership.
+
+$TTL 300       ; 5 minutes
+@                      CH SOA  ns root (
+                               2018010100 ; serial
+                               1800       ; refresh (30 minutes)
+                               1800       ; retry (30 minutes)
+                               1814400    ; expire (3 weeks)
+                               3600       ; minimum (1 hour)
+                               )
+                       NS      ns
+ns                     A       ch-addr.net. 1001
+test                   A       ch-addr.net. 1002
+                       A       ch-addr.net. 1003

diff --git a/bin/tests/system/auth/ns1/named.conf.in b/bin/tests/system/auth/ns1/named.conf.in
index f239e15a736331d3616ef1eb043d37dfa75ccd72..1cddc494034493296c84ce6c4f7be0034df5227d 100644 (file)
--- a/bin/tests/system/auth/ns1/named.conf.in
+++ b/bin/tests/system/auth/ns1/named.conf.in
@@ -21,12 +21,21 @@ options {
        notify yes;
 };
 
-zone example.net {
-       type master;
-       file "example.net.db";
+view main in {
+       zone example.net {
+               type master;
+               file "example.net.db";
+       };
+
+       zone example.com {
+               type master;
+               file "example.com.db";
+       };
 };
 
-zone example.com {
-       type master;
-       file "example.com.db";
+view alt chaos {
+       zone example.chaos chaos {
+               type master;
+               file "chaos.db";
+       };
 };

diff --git a/bin/tests/system/auth/tests.sh b/bin/tests/system/auth/tests.sh
index 348fcf8975d75260415772413df8f1efe3a69c74..cc47c39b92dd0cf9db05221d742e88b249ea63a7 100644 (file)
--- a/bin/tests/system/auth/tests.sh
+++ b/bin/tests/system/auth/tests.sh
@@ -128,5 +128,14 @@ grep "a.example.com.*A.*10.53.0.1" dig.out.test$n > /dev/null || ret=1
 [ $ret -eq 0 ] || echo_i "failed"
 status=`expr $status + $ret`
 
+n=`expr $n + 1`
+echo_i "check that CHAOS addresses are compared correctly ($n)"
+ret=0
+$DIG $DIGOPTS @10.53.0.1 +noall +answer ch test.example.chaos > dig.out.test$n
+lines=`wc -l < dig.out.test$n`
+[ ${lines:-0} -eq 2 ] || ret=1
+[ $ret -eq 0 ] || echo_i "failed"
+status=`expr $status + $ret`
+
 echo_i "exit status: $status"
 [ $status -eq 0 ] || exit 1

diff --git a/lib/dns/rdata/ch_3/a_1.c b/lib/dns/rdata/ch_3/a_1.c
index a1b571fa14582bd207b7aa598e2ed2f496686cd4..94cae92c3c0a69e30e49df76d971050c91ff7128 100644 (file)
--- a/lib/dns/rdata/ch_3/a_1.c
+++ b/lib/dns/rdata/ch_3/a_1.c
@@ -179,7 +179,7 @@ compare_ch_a(ARGS_COMPARE) {
        if (order != 0)
                return (order);
 
-       order = memcmp(rdata1->data, rdata2->data, 2);
+       order = memcmp(region1.base, region2.base, 2);
        if (order != 0)
                order = (order < 0) ? -1 : 1;
        return (order);

diff --git a/util/copyrights b/util/copyrights
index dcc7e3b7acf27ef49f337091bf189a9c48f0a13e..717c32379a8e6b15eec36d0646745c0a03e522c9 100644 (file)
--- a/util/copyrights
+++ b/util/copyrights
@@ -485,6 +485,7 @@
 ./bin/tests/system/allow-query/tests.sh                SH      2018
 ./bin/tests/system/ans.pl                      PERL    2011,2012,2014,2016,2017,2018
 ./bin/tests/system/auth/clean.sh               SH      2018
+./bin/tests/system/auth/ns1/chaos.db           ZONE    2018
 ./bin/tests/system/auth/ns1/example.com.db     ZONE    2018
 ./bin/tests/system/auth/ns1/example.net.db     ZONE    2018
 ./bin/tests/system/auth/ns1/named.conf.in      CONF-C  2018