Don't insert 2nd space between NSEC3 nexthash and typemap fields

diff --git a/bin/tests/system/xfer/dig1.good b/bin/tests/system/xfer/dig1.good
index 5af1e5928a6d4c97667d151417016e6ed02aa5b9..aba6a43ac363e296eff59d125e26599164cafa1e 100644 (file)
--- a/bin/tests/system/xfer/dig1.good
+++ b/bin/tests/system/xfer/dig1.good
@@ -149,7 +149,7 @@ wks01.example.              3600    IN      WKS     10.0.0.1 6 0 1 2 21 23
 wks02.example.         3600    IN      WKS     10.0.0.1 17 0 1 2 53
 wks03.example.         3600    IN      WKS     10.0.0.2 6 65535
 x2501.example.         3600    IN      X25     "123456789"
-8f1tmio9avcom2k0frp92lgcumak0cad.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C 8FPNS2UCT7FBS643THP2B77PEQ77K6IU  A NS SOA MX AAAA RRSIG DNSKEY NSEC3PARAM
+8f1tmio9avcom2k0frp92lgcumak0cad.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C 8FPNS2UCT7FBS643THP2B77PEQ77K6IU A NS SOA MX AAAA RRSIG DNSKEY NSEC3PARAM
 kcd3juae64f9c5csl1kif1htaui7un0g.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C KD5MN2M20340DGO0BL7NTSB8JP4BSC7E
-mr5ukvsk1l37btu4q7b1dfevft4hkqdk.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C MT38J6VG7S0SN5G17MCUF6IQIKFUAJ05  A AAAA RRSIG
+mr5ukvsk1l37btu4q7b1dfevft4hkqdk.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C MT38J6VG7S0SN5G17MCUF6IQIKFUAJ05 A AAAA RRSIG
 example.               86400   IN      SOA     ns2.example. hostmaster.example. 1397051952 5 5 1814400 3600

diff --git a/bin/tests/system/xfer/dig2.good b/bin/tests/system/xfer/dig2.good
index e12ec118c5d6e0f85d7470b1f157597f6645f18a..40c8c405b41045b013daafb00fd2eb624972c4f7 100644 (file)
--- a/bin/tests/system/xfer/dig2.good
+++ b/bin/tests/system/xfer/dig2.good
@@ -149,7 +149,7 @@ wks01.example.              3600    IN      WKS     10.0.0.1 6 0 1 2 21 23
 wks02.example.         3600    IN      WKS     10.0.0.1 17 0 1 2 53
 wks03.example.         3600    IN      WKS     10.0.0.2 6 65535
 x2501.example.         3600    IN      X25     "123456789"
-8f1tmio9avcom2k0frp92lgcumak0cad.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C 8FPNS2UCT7FBS643THP2B77PEQ77K6IU  A NS SOA MX AAAA RRSIG DNSKEY NSEC3PARAM
+8f1tmio9avcom2k0frp92lgcumak0cad.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C 8FPNS2UCT7FBS643THP2B77PEQ77K6IU A NS SOA MX AAAA RRSIG DNSKEY NSEC3PARAM
 kcd3juae64f9c5csl1kif1htaui7un0g.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C KD5MN2M20340DGO0BL7NTSB8JP4BSC7E
-mr5ukvsk1l37btu4q7b1dfevft4hkqdk.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C MT38J6VG7S0SN5G17MCUF6IQIKFUAJ05  A AAAA RRSIG
+mr5ukvsk1l37btu4q7b1dfevft4hkqdk.example. 3600 IN NSEC3 1 0 10 D2CF0294C020CE6C MT38J6VG7S0SN5G17MCUF6IQIKFUAJ05 A AAAA RRSIG
 example.               86400   IN      SOA     ns2.example. hostmaster.example. 1397051953 5 5 1814400 3600

diff --git a/lib/dns/rdata.c b/lib/dns/rdata.c
index aebf254292088ce3e2cc14e1310c6e826f677153..0594bc2f01a3aded2bcaf5c3f1ac9e1d82400002 100644 (file)
--- a/lib/dns/rdata.c
+++ b/lib/dns/rdata.c
@@ -469,7 +469,7 @@ typemap_totext(isc_region_t *sr, dns_rdata_textctx_t *tctx,
 {
        unsigned int i, j, k;
        unsigned int window, len;
-       isc_boolean_t first = ISC_FALSE;
+       isc_boolean_t first = ISC_TRUE;
 
        for (i = 0; i < sr->length; i += len) {
                if (tctx != NULL &&

diff --git a/lib/dns/rdata/generic/csync_62.c b/lib/dns/rdata/generic/csync_62.c
index eff2ef5e41088ba4a1bc94b036bc4c1d6c60ae9d..35c840ba1c257d24fdac2a28c29d8454c0e245b9 100644 (file)
--- a/lib/dns/rdata/generic/csync_62.c
+++ b/lib/dns/rdata/generic/csync_62.c
@@ -69,6 +69,12 @@ totext_csync(ARGS_TOTEXT) {
        snprintf(buf, sizeof(buf), "%lu", num);
        RETERR(str_totext(buf, target));
 
+       /*
+        * Don't leave a trailing space when there's no typemap present.
+        */
+       if (sr.length > 0) {
+               RETERR(str_totext(" ", target));
+       }
        return (typemap_totext(&sr, NULL, target));
 }
 

diff --git a/lib/dns/rdata/generic/nsec3_50.c b/lib/dns/rdata/generic/nsec3_50.c
index c58e575463378e9b64b28914cab7c7f011329434..294e856fc481d47cfcea6f7d17ddedd39bec6fde 100644 (file)
--- a/lib/dns/rdata/generic/nsec3_50.c
+++ b/lib/dns/rdata/generic/nsec3_50.c
@@ -162,9 +162,9 @@ totext_nsec3(ARGS_TOTEXT) {
        /*
         * Don't leave a trailing space when there's no typemap present.
         */
-       if (((tctx->flags & DNS_STYLEFLAG_MULTILINE) == 0) && (sr.length > 0))
+       if (((tctx->flags & DNS_STYLEFLAG_MULTILINE) == 0) && (sr.length > 0)) {
                RETERR(str_totext(" ", target));
-
+       }
        RETERR(typemap_totext(&sr, tctx, target));
 
        if ((tctx->flags & DNS_STYLEFLAG_MULTILINE) != 0)

diff --git a/lib/dns/rdata/generic/nsec_47.c b/lib/dns/rdata/generic/nsec_47.c
index e6647bbc9c0c9e03a174cc10a40b17ad24801282..67f9967fd397d56718047ecbe85f69ab5546ac07 100644 (file)
--- a/lib/dns/rdata/generic/nsec_47.c
+++ b/lib/dns/rdata/generic/nsec_47.c
@@ -61,6 +61,12 @@ totext_nsec(ARGS_TOTEXT) {
        dns_name_fromregion(&name, &sr);
        isc_region_consume(&sr, name_length(&name));
        RETERR(dns_name_totext(&name, ISC_FALSE, target));
+       /*
+        * Don't leave a trailing space when there's no typemap present.
+        */
+       if (sr.length > 0) {
+               RETERR(str_totext(" ", target));
+       }
        return (typemap_totext(&sr, NULL, target));
 }