]> The Tcpdump Group git mirrors - tcpdump/blobdiff - print-domain.c
Revert partially the commit 21b1273
[tcpdump] / print-domain.c
index c3066f7d7d30e75a7ce05ccb0e7feedd768b67bd..ad08073f129c51d0468f4d8cfd85dbb38c399dcd 100644 (file)
  * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 
+/* \summary: Domain Name System (DNS) printer */
+
 #ifdef HAVE_CONFIG_H
-#include "config.h"
+#include <config.h>
 #endif
 
-#include <netdissect-stdinc.h>
-
-#include "nameser.h"
+#include "netdissect-stdinc.h"
 
 #include <string.h>
 
@@ -34,6 +34,8 @@
 #include "addrtostr.h"
 #include "extract.h"
 
+#include "nameser.h"
+
 static const char *ns_ops[] = {
        "", " inv_q", " stat", " op3", " notify", " update", " op6", " op7",
        " op8", " updateA", " updateD", " updateDA",
@@ -45,18 +47,32 @@ static const char *ns_resp[] = {
        " NotImp", " Refused", " YXDomain", " YXRRSet",
        " NXRRSet", " NotAuth", " NotZone", " Resp11",
        " Resp12", " Resp13", " Resp14", " NoChange",
+       " BadVers", "Resp17", " Resp18", " Resp19",
+       " Resp20", "Resp21", " Resp22", " BadCookie",
 };
 
+static const char *
+ns_rcode(u_int rcode) {
+       static char buf[sizeof(" Resp4095")];
+
+       if (rcode < sizeof(ns_resp)/sizeof(ns_resp[0])) {
+               return (ns_resp[rcode]);
+       }
+       snprintf(buf, sizeof(buf), " Resp%u", rcode & 0xfff);
+       return (buf);
+}
+
 /* skip over a domain name */
 static const u_char *
 ns_nskip(netdissect_options *ndo,
-         register const u_char *cp)
+         const u_char *cp)
 {
-       register u_char i;
+       u_char i;
 
-       if (!ND_TTEST2(*cp, 1))
+       if (!ND_TTEST_1(cp))
                return (NULL);
-       i = *cp++;
+       i = GET_U_1(cp);
+       cp++;
        while (i) {
                if ((i & INDIR_MASK) == INDIR_MASK)
                        return (cp + 1);
@@ -65,100 +81,99 @@ ns_nskip(netdissect_options *ndo,
 
                        if ((i & ~INDIR_MASK) != EDNS0_ELT_BITLABEL)
                                return(NULL); /* unknown ELT */
-                       if (!ND_TTEST2(*cp, 1))
+                       if (!ND_TTEST_1(cp))
                                return (NULL);
-                       if ((bitlen = *cp++) == 0)
+                       if ((bitlen = GET_U_1(cp)) == 0)
                                bitlen = 256;
+                       cp++;
                        bytelen = (bitlen + 7) / 8;
                        cp += bytelen;
                } else
                        cp += i;
-               if (!ND_TTEST2(*cp, 1))
+               if (!ND_TTEST_1(cp))
                        return (NULL);
-               i = *cp++;
+               i = GET_U_1(cp);
+               cp++;
        }
        return (cp);
 }
 
-/* print a <domain-name> */
 static const u_char *
 blabel_print(netdissect_options *ndo,
              const u_char *cp)
 {
-       int bitlen, slen, b;
+       u_int bitlen, slen, b;
        const u_char *bitp, *lim;
-       char tc;
+       uint8_t tc;
 
-       if (!ND_TTEST2(*cp, 1))
+       if (!ND_TTEST_1(cp))
                return(NULL);
-       if ((bitlen = *cp) == 0)
+       if ((bitlen = GET_U_1(cp)) == 0)
                bitlen = 256;
        slen = (bitlen + 3) / 4;
        lim = cp + 1 + slen;
 
        /* print the bit string as a hex string */
-       ND_PRINT((ndo, "\\[x"));
+       ND_PRINT("\\[x");
        for (bitp = cp + 1, b = bitlen; bitp < lim && b > 7; b -= 8, bitp++) {
-               ND_TCHECK(*bitp);
-               ND_PRINT((ndo, "%02x", *bitp));
+               ND_PRINT("%02x", GET_U_1(bitp));
        }
        if (b > 4) {
-               ND_TCHECK(*bitp);
-               tc = *bitp++;
-               ND_PRINT((ndo, "%02x", tc & (0xff << (8 - b))));
+               tc = GET_U_1(bitp);
+               bitp++;
+               ND_PRINT("%02x", tc & (0xff << (8 - b)));
        } else if (b > 0) {
-               ND_TCHECK(*bitp);
-               tc = *bitp++;
-               ND_PRINT((ndo, "%1x", ((tc >> 4) & 0x0f) & (0x0f << (4 - b))));
+               tc = GET_U_1(bitp);
+               bitp++;
+               ND_PRINT("%1x", ((tc >> 4) & 0x0f) & (0x0f << (4 - b)));
        }
-       ND_PRINT((ndo, "/%d]", bitlen));
+       ND_PRINT("/%u]", bitlen);
        return lim;
-trunc:
-       ND_PRINT((ndo, ".../%d]", bitlen));
-       return NULL;
 }
 
 static int
 labellen(netdissect_options *ndo,
          const u_char *cp)
 {
-       register u_int i;
+       u_int i;
 
-       if (!ND_TTEST2(*cp, 1))
+       if (!ND_TTEST_1(cp))
                return(-1);
-       i = *cp;
+       i = GET_U_1(cp);
        if ((i & INDIR_MASK) == EDNS0_MASK) {
-               int bitlen, elt;
+               u_int bitlen, elt;
                if ((elt = (i & ~INDIR_MASK)) != EDNS0_ELT_BITLABEL) {
-                       ND_PRINT((ndo, "<ELT %d>", elt));
+                       ND_PRINT("<ELT %d>", elt);
                        return(-1);
                }
-               if (!ND_TTEST2(*(cp + 1), 1))
+               if (!ND_TTEST_1(cp + 1))
                        return(-1);
-               if ((bitlen = *(cp + 1)) == 0)
+               if ((bitlen = GET_U_1(cp + 1)) == 0)
                        bitlen = 256;
                return(((bitlen + 7) / 8) + 1);
        } else
                return(i);
 }
 
+/* print a <domain-name> */
 const u_char *
-ns_nprint(netdissect_options *ndo,
-          register const u_char *cp, register const u_char *bp)
+fqdn_print(netdissect_options *ndo,
+          const u_char *cp, const u_char *bp)
 {
-       register u_int i, l;
-       register const u_char *rp = NULL;
-       register int compress = 0;
-       int chars_processed;
-       int elt;
-       int data_size = ndo->ndo_snapend - bp;
+       u_int i, l;
+       const u_char *rp = NULL;
+       int compress = 0;
+       u_int elt;
+       u_int offset, max_offset;
 
        if ((l = labellen(ndo, cp)) == (u_int)-1)
                return(NULL);
-       if (!ND_TTEST2(*cp, 1))
+       if (!ND_TTEST_1(cp))
                return(NULL);
-       chars_processed = 1;
-       if (((i = *cp++) & INDIR_MASK) != INDIR_MASK) {
+       max_offset = (u_int)(cp - bp);
+       i = GET_U_1(cp);
+       cp++;
+       if ((i & INDIR_MASK) != INDIR_MASK) {
                compress = 0;
                rp = cp + l;
        }
@@ -170,26 +185,31 @@ ns_nprint(netdissect_options *ndo,
                                        rp = cp + 1;
                                        compress = 1;
                                }
-                               if (!ND_TTEST2(*cp, 1))
-                                       return(NULL);
-                               cp = bp + (((i << 8) | *cp) & 0x3fff);
-                               if ((l = labellen(ndo, cp)) == (u_int)-1)
-                                       return(NULL);
-                               if (!ND_TTEST2(*cp, 1))
+                               if (!ND_TTEST_1(cp))
                                        return(NULL);
-                               i = *cp++;
-                               chars_processed++;
-
+                               offset = (((i << 8) | GET_U_1(cp)) & 0x3fff);
                                /*
-                                * If we've looked at every character in
-                                * the message, this pointer will make
-                                * us look at some character again,
-                                * which means we're looping.
+                                * This must move backwards in the packet.
+                                * No RFC explicitly says that, but BIND's
+                                * name decompression code requires it,
+                                * as a way of preventing infinite loops
+                                * and other bad behavior, and it's probably
+                                * what was intended (compress by pointing
+                                * to domain name suffixes already seen in
+                                * the packet).
                                 */
-                               if (chars_processed >= data_size) {
-                                       ND_PRINT((ndo, "<LOOP>"));
-                                       return (NULL);
+                               if (offset >= max_offset) {
+                                       ND_PRINT("<BAD PTR>");
+                                       return(NULL);
                                }
+                               max_offset = offset;
+                               cp = bp + offset;
+                               if ((l = labellen(ndo, cp)) == (u_int)-1)
+                                       return(NULL);
+                               if (!ND_TTEST_1(cp))
+                                       return(NULL);
+                               i = GET_U_1(cp);
+                               cp++;
                                continue;
                        }
                        if ((i & INDIR_MASK) == EDNS0_MASK) {
@@ -201,47 +221,216 @@ ns_nprint(netdissect_options *ndo,
                                        break;
                                default:
                                        /* unknown ELT */
-                                       ND_PRINT((ndo, "<ELT %d>", elt));
+                                       ND_PRINT("<ELT %u>", elt);
                                        return(NULL);
                                }
                        } else {
-                               if (fn_printn(ndo, cp, l, ndo->ndo_snapend))
+                               if (nd_printn(ndo, cp, l, ndo->ndo_snapend))
                                        return(NULL);
                        }
 
                        cp += l;
-                       chars_processed += l;
-                       ND_PRINT((ndo, "."));
+                       ND_PRINT(".");
                        if ((l = labellen(ndo, cp)) == (u_int)-1)
                                return(NULL);
-                       if (!ND_TTEST2(*cp, 1))
+                       if (!ND_TTEST_1(cp))
                                return(NULL);
-                       i = *cp++;
-                       chars_processed++;
+                       i = GET_U_1(cp);
+                       cp++;
                        if (!compress)
                                rp += l + 1;
                }
        else
-               ND_PRINT((ndo, "."));
+               ND_PRINT(".");
        return (rp);
 }
 
 /* print a <character-string> */
 static const u_char *
 ns_cprint(netdissect_options *ndo,
-          register const u_char *cp)
+          const u_char *cp)
 {
-       register u_int i;
+       u_int i;
 
-       if (!ND_TTEST2(*cp, 1))
+       if (!ND_TTEST_1(cp))
                return (NULL);
-       i = *cp++;
-       if (fn_printn(ndo, cp, i, ndo->ndo_snapend))
+       i = GET_U_1(cp);
+       cp++;
+       if (nd_printn(ndo, cp, i, ndo->ndo_snapend))
                return (NULL);
        return (cp + i);
 }
 
-/* http://www.iana.org/assignments/dns-parameters */
+static void
+print_eopt_ecs(netdissect_options *ndo, const u_char *cp,
+               u_int data_len)
+{
+    u_int family, addr_bits, src_len, scope_len;
+
+    u_char padded[32];
+    char addr[INET6_ADDRSTRLEN];
+
+    /* ecs option must at least contain family, src len, and scope len */
+    if (data_len < 4) {
+        nd_print_invalid(ndo);
+        return;
+    }
+
+    family = GET_BE_U_2(cp);
+    cp += 2;
+    src_len = GET_U_1(cp);
+    cp += 1;
+    scope_len = GET_U_1(cp);
+    cp += 1;
+
+    if (family == 1)
+        addr_bits = 32;
+    else if (family == 2)
+        addr_bits = 128;
+    else {
+        nd_print_invalid(ndo);
+        return;
+    }
+
+    if (data_len - 4 > (addr_bits / 8)) {
+        nd_print_invalid(ndo);
+        return;
+    }
+    /* checks for invalid ecs scope or source length */
+    if (src_len > addr_bits || scope_len > addr_bits || ((src_len + 7) / 8) != (data_len - 4)) {
+        nd_print_invalid(ndo);
+        return;
+    }
+
+    /* pad the truncated address from ecs with zeros */
+    memset(padded, 0, sizeof(padded));
+    memcpy(padded, cp, data_len - 4);
+
+
+    if (family == 1)
+        ND_PRINT("%s/%d/%d", addrtostr(padded, addr, INET_ADDRSTRLEN),
+                src_len, scope_len);
+    else
+        ND_PRINT("%s/%d/%d", addrtostr6(padded, addr, INET6_ADDRSTRLEN),
+                src_len, scope_len);
+
+}
+
+extern const struct tok edns_opt2str[];
+extern const struct tok dau_alg2str[];
+extern const struct tok dhu_alg2str[];
+extern const struct tok n3u_alg2str[];
+
+
+/* print an <EDNS-option> */
+static const u_char *
+eopt_print(netdissect_options *ndo,
+          const u_char *cp)
+{
+    u_int opt, data_len, i;
+
+    if (!ND_TTEST_2(cp))
+        return (NULL);
+    opt = GET_BE_U_2(cp);
+    cp += 2;
+    ND_PRINT("%s", tok2str(edns_opt2str, "Opt%u", opt));
+    if (!ND_TTEST_2(cp))
+        return (NULL);
+    data_len = GET_BE_U_2(cp);
+    cp += 2;
+
+    ND_TCHECK_LEN(cp, data_len);
+
+    if (data_len > 0) {
+        ND_PRINT(" ");
+        switch (opt) {
+
+        case E_ECS:
+            print_eopt_ecs(ndo, cp, data_len);
+            break;
+        case E_COOKIE:
+            if (data_len < 8 || (data_len > 8 && data_len < 16) || data_len > 40)
+                nd_print_invalid(ndo);
+            else {
+                for (i = 0; i < data_len; ++i) {
+                    /* split client and server cookie */
+                    if (i == 8)
+                        ND_PRINT(" ");
+                    ND_PRINT("%02x", GET_U_1(cp + i));
+                }
+            }
+            break;
+        case E_KEEPALIVE:
+            if (data_len != 2)
+                nd_print_invalid(ndo);
+            else
+                /* keepalive is in increments of 100ms. Convert to seconds */
+                ND_PRINT("%0.1f sec", (GET_BE_U_2(cp) / 10.0));
+            break;
+        case E_EXPIRE:
+            if (data_len != 4)
+                nd_print_invalid(ndo);
+            else
+                ND_PRINT("%u sec", GET_BE_U_4(cp));
+            break;
+        case E_PADDING:
+            /* ignore contents and just print length */
+            ND_PRINT("(%u)", data_len);
+            break;
+        case E_KEYTAG:
+            if (data_len % 2 != 0)
+                nd_print_invalid(ndo);
+            else
+                for (i = 0; i < data_len; i += 2) {
+                    if (i > 0)
+                        ND_PRINT(" ");
+                    ND_PRINT("%u", GET_BE_U_2(cp + i));
+                }
+            break;
+        case E_DAU:
+            for (i = 0; i < data_len; ++i) {
+                if (i > 0)
+                    ND_PRINT(" ");
+                ND_PRINT("%s", tok2str(dau_alg2str, "Alg_%u", GET_U_1(cp + i)));
+            }
+            break;
+        case E_DHU:
+            for (i = 0; i < data_len; ++i) {
+                if (i > 0)
+                    ND_PRINT(" ");
+                ND_PRINT("%s", tok2str(dhu_alg2str, "Alg_%u", GET_U_1(cp + i)));
+            }
+            break;
+        case E_N3U:
+            for (i = 0; i < data_len; ++i) {
+                if (i > 0)
+                    ND_PRINT(" ");
+                ND_PRINT("%s", tok2str(n3u_alg2str, "Alg_%u", GET_U_1(cp + i)));
+            }
+            break;
+        case E_CHAIN:
+            fqdn_print(ndo, cp, cp + data_len);
+            break;
+        case E_NSID:
+            /* intentional fall-through. NSID is an undefined byte string */
+        default:
+            for (i = 0; i < data_len; ++i)
+                ND_PRINT("%02x", GET_U_1(cp + i));
+            break;
+        }
+    }
+    return (cp + data_len);
+
+  trunc:
+    return (NULL);
+
+}
+
+
+
+extern const struct tok ns_type2str[];
+
+/* https://www.iana.org/assignments/dns-parameters */
 const struct tok ns_type2str[] = {
        { T_A,          "A" },                  /* RFC 1035 */
        { T_NS,         "NS" },                 /* RFC 1035 */
@@ -304,9 +493,12 @@ const struct tok ns_type2str[] = {
        { T_MAILB,      "MAILB" },              /* RFC 1035 */
        { T_MAILA,      "MAILA" },              /* RFC 1035 */
        { T_ANY,        "ANY" },
+       { T_URI,        "URI" },                /* RFC 7553 */
        { 0,            NULL }
 };
 
+extern const struct tok ns_class2str[];
+
 const struct tok ns_class2str[] = {
        { C_IN,         "IN" },         /* Not used */
        { C_CHAOS,      "CHAOS" },
@@ -315,108 +507,161 @@ const struct tok ns_class2str[] = {
        { 0,            NULL }
 };
 
+const struct tok edns_opt2str[] = {
+    { E_LLQ,        "LLQ" },
+    { E_UL,         "UL" },
+    { E_NSID,       "NSID" },
+    { E_DAU,        "DAU" },
+    { E_DHU,        "DHU" },
+    { E_N3U,        "N3U" },
+    { E_ECS,        "ECS" },
+    { E_EXPIRE,     "EXPIRE" },
+    { E_COOKIE,     "COOKIE" },
+    { E_KEEPALIVE,  "KEEPALIVE" },
+    { E_PADDING,    "PADDING" },
+    { E_CHAIN,      "CHAIN" },
+    { E_KEYTAG,     "KEY-TAG" },
+    { E_CLIENTTAG,  "CLIENT-TAG" },
+    { E_SERVERTAG,  "SERVER-TAG" },
+    { 0,            NULL }
+};
+
+const struct tok dau_alg2str[] = {
+    { A_DELETE,             "DELETE" },
+    { A_RSAMD5,             "RSAMD5" },
+    { A_DH,                 "DH" },
+    { A_DSA,                "DS" },
+    { A_RSASHA1,            "RSASHA1" },
+    { A_DSA_NSEC3_SHA1,     "DSA-NSEC3-SHA1" },
+    { A_RSASHA1_NSEC3_SHA1, "RSASHA1-NSEC3-SHA1" },
+    { A_RSASHA256,          "RSASHA256" },
+    { A_RSASHA512,          "RSASHA512" },
+    { A_ECC_GOST,           "ECC-GOST" },
+    { A_ECDSAP256SHA256,    "ECDSAP256SHA256" },
+    { A_ECDSAP384SHA384,    "ECDSAP384SHA384" },
+    { A_ED25519,            "ED25519" },
+    { A_ED448,              "ED448" },
+    { A_INDIRECT,           "INDIRECT" },
+    { A_PRIVATEDNS,         "PRIVATEDNS" },
+    { A_PRIVATEOID,         "PRIVATEOID" },
+    { 0,                NULL }
+};
+
+const struct tok dhu_alg2str[] = {
+    { DS_SHA1,  "SHA-1" },
+    { DS_SHA256,"SHA-256" },
+    { DS_GOST,  "GOST_R_34.11-94" },
+    { DS_SHA384,"SHA-384" },
+    { 0,    NULL }
+};
+
+const struct tok n3u_alg2str[] = {
+    { NSEC_SHA1,"SHA-1" },
+    { 0,    NULL }
+};
+
 /* print a query */
 static const u_char *
 ns_qprint(netdissect_options *ndo,
-          register const u_char *cp, register const u_char *bp, int is_mdns)
+          const u_char *cp, const u_char *bp, int is_mdns)
 {
-       register const u_char *np = cp;
-       register u_int i, class;
+       const u_char *np = cp;
+       u_int i, class;
 
        cp = ns_nskip(ndo, cp);
 
-       if (cp == NULL || !ND_TTEST2(*cp, 4))
+       if (cp == NULL || !ND_TTEST_4(cp))
                return(NULL);
 
        /* print the qtype */
-       i = EXTRACT_16BITS(cp);
+       i = GET_BE_U_2(cp);
        cp += 2;
-       ND_PRINT((ndo, " %s", tok2str(ns_type2str, "Type%d", i)));
+       ND_PRINT(" %s", tok2str(ns_type2str, "Type%u", i));
        /* print the qclass (if it's not IN) */
-       i = EXTRACT_16BITS(cp);
+       i = GET_BE_U_2(cp);
        cp += 2;
        if (is_mdns)
                class = (i & ~C_QU);
        else
                class = i;
        if (class != C_IN)
-               ND_PRINT((ndo, " %s", tok2str(ns_class2str, "(Class %d)", class)));
+               ND_PRINT(" %s", tok2str(ns_class2str, "(Class %u)", class));
        if (is_mdns) {
-               ND_PRINT((ndo, i & C_QU ? " (QU)" : " (QM)"));
+               ND_PRINT(i & C_QU ? " (QU)" : " (QM)");
        }
 
-       ND_PRINT((ndo, "? "));
-       cp = ns_nprint(ndo, np, bp);
+       ND_PRINT("? ");
+       cp = fqdn_print(ndo, np, bp);
        return(cp ? cp + 4 : NULL);
 }
 
 /* print a reply */
 static const u_char *
 ns_rprint(netdissect_options *ndo,
-          register const u_char *cp, register const u_char *bp, int is_mdns)
+          const u_char *cp, const u_char *bp, int is_mdns)
 {
-       register u_int i, class, opt_flags = 0;
-       register u_short typ, len;
-       register const u_char *rp;
+       u_int i, class, opt_flags = 0;
+       u_short typ, len;
+       const u_char *rp;
 
        if (ndo->ndo_vflag) {
-               ND_PRINT((ndo, " "));
-               if ((cp = ns_nprint(ndo, cp, bp)) == NULL)
+               ND_PRINT(" ");
+               if ((cp = fqdn_print(ndo, cp, bp)) == NULL)
                        return NULL;
        } else
                cp = ns_nskip(ndo, cp);
 
-       if (cp == NULL || !ND_TTEST2(*cp, 10))
+       if (cp == NULL || !ND_TTEST_LEN(cp, 10))
                return (ndo->ndo_snapend);
 
        /* print the type/qtype */
-       typ = EXTRACT_16BITS(cp);
+       typ = GET_BE_U_2(cp);
        cp += 2;
        /* print the class (if it's not IN and the type isn't OPT) */
-       i = EXTRACT_16BITS(cp);
+       i = GET_BE_U_2(cp);
        cp += 2;
        if (is_mdns)
                class = (i & ~C_CACHE_FLUSH);
        else
                class = i;
        if (class != C_IN && typ != T_OPT)
-               ND_PRINT((ndo, " %s", tok2str(ns_class2str, "(Class %d)", class)));
+               ND_PRINT(" %s", tok2str(ns_class2str, "(Class %u)", class));
        if (is_mdns) {
                if (i & C_CACHE_FLUSH)
-                       ND_PRINT((ndo, " (Cache flush)"));
+                       ND_PRINT(" (Cache flush)");
        }
 
        if (typ == T_OPT) {
                /* get opt flags */
                cp += 2;
-               opt_flags = EXTRACT_16BITS(cp);
+               opt_flags = GET_BE_U_2(cp);
                /* ignore rest of ttl field */
                cp += 2;
        } else if (ndo->ndo_vflag > 2) {
                /* print ttl */
-               ND_PRINT((ndo, " ["));
-               relts_print(ndo, EXTRACT_32BITS(cp));
-               ND_PRINT((ndo, "]"));
+               ND_PRINT(" [");
+               unsigned_relts_print(ndo, GET_BE_U_4(cp));
+               ND_PRINT("]");
                cp += 4;
        } else {
                /* ignore ttl */
                cp += 4;
        }
 
-       len = EXTRACT_16BITS(cp);
+       len = GET_BE_U_2(cp);
        cp += 2;
 
        rp = cp + len;
 
-       ND_PRINT((ndo, " %s", tok2str(ns_type2str, "Type%d", typ)));
+       ND_PRINT(" %s", tok2str(ns_type2str, "Type%u", typ));
        if (rp > ndo->ndo_snapend)
                return(NULL);
 
        switch (typ) {
        case T_A:
-               if (!ND_TTEST2(*cp, sizeof(struct in_addr)))
+               if (!ND_TTEST_LEN(cp, sizeof(nd_ipv4)))
                        return(NULL);
-               ND_PRINT((ndo, " %s", intoa(htonl(EXTRACT_32BITS(cp)))));
+               ND_PRINT(" %s", intoa(GET_IPV4_TO_NETWORK_ORDER(cp)));
                break;
 
        case T_NS:
@@ -425,113 +670,132 @@ ns_rprint(netdissect_options *ndo,
 #ifdef T_DNAME
        case T_DNAME:
 #endif
-               ND_PRINT((ndo, " "));
-               if (ns_nprint(ndo, cp, bp) == NULL)
+               ND_PRINT(" ");
+               if (fqdn_print(ndo, cp, bp) == NULL)
                        return(NULL);
                break;
 
        case T_SOA:
                if (!ndo->ndo_vflag)
                        break;
-               ND_PRINT((ndo, " "));
-               if ((cp = ns_nprint(ndo, cp, bp)) == NULL)
+               ND_PRINT(" ");
+               if ((cp = fqdn_print(ndo, cp, bp)) == NULL)
                        return(NULL);
-               ND_PRINT((ndo, " "));
-               if ((cp = ns_nprint(ndo, cp, bp)) == NULL)
+               ND_PRINT(" ");
+               if ((cp = fqdn_print(ndo, cp, bp)) == NULL)
                        return(NULL);
-               if (!ND_TTEST2(*cp, 5 * 4))
+               if (!ND_TTEST_LEN(cp, 5 * 4))
                        return(NULL);
-               ND_PRINT((ndo, " %u", EXTRACT_32BITS(cp)));
+               ND_PRINT(" %u", GET_BE_U_4(cp));
                cp += 4;
-               ND_PRINT((ndo, " %u", EXTRACT_32BITS(cp)));
+               ND_PRINT(" %u", GET_BE_U_4(cp));
                cp += 4;
-               ND_PRINT((ndo, " %u", EXTRACT_32BITS(cp)));
+               ND_PRINT(" %u", GET_BE_U_4(cp));
                cp += 4;
-               ND_PRINT((ndo, " %u", EXTRACT_32BITS(cp)));
+               ND_PRINT(" %u", GET_BE_U_4(cp));
                cp += 4;
-               ND_PRINT((ndo, " %u", EXTRACT_32BITS(cp)));
+               ND_PRINT(" %u", GET_BE_U_4(cp));
                cp += 4;
                break;
        case T_MX:
-               ND_PRINT((ndo, " "));
-               if (!ND_TTEST2(*cp, 2))
+               ND_PRINT(" ");
+               if (!ND_TTEST_2(cp))
                        return(NULL);
-               if (ns_nprint(ndo, cp + 2, bp) == NULL)
+               if (fqdn_print(ndo, cp + 2, bp) == NULL)
                        return(NULL);
-               ND_PRINT((ndo, " %d", EXTRACT_16BITS(cp)));
+               ND_PRINT(" %u", GET_BE_U_2(cp));
                break;
 
        case T_TXT:
                while (cp < rp) {
-                       ND_PRINT((ndo, " \""));
+                       ND_PRINT(" \"");
                        cp = ns_cprint(ndo, cp);
                        if (cp == NULL)
                                return(NULL);
-                       ND_PRINT((ndo, "\""));
+                       ND_PRINT("\"");
                }
                break;
 
        case T_SRV:
-               ND_PRINT((ndo, " "));
-               if (!ND_TTEST2(*cp, 6))
+               ND_PRINT(" ");
+               if (!ND_TTEST_6(cp))
                        return(NULL);
-               if (ns_nprint(ndo, cp + 6, bp) == NULL)
+               if (fqdn_print(ndo, cp + 6, bp) == NULL)
                        return(NULL);
-               ND_PRINT((ndo, ":%d %d %d", EXTRACT_16BITS(cp + 4),
-                       EXTRACT_16BITS(cp), EXTRACT_16BITS(cp + 2)));
+               ND_PRINT(":%u %u %u", GET_BE_U_2(cp + 4),
+                         GET_BE_U_2(cp), GET_BE_U_2(cp + 2));
                break;
 
        case T_AAAA:
            {
                char ntop_buf[INET6_ADDRSTRLEN];
 
-               if (!ND_TTEST2(*cp, sizeof(struct in6_addr)))
+               if (!ND_TTEST_LEN(cp, sizeof(nd_ipv6)))
                        return(NULL);
-               ND_PRINT((ndo, " %s",
-                   addrtostr6(cp, ntop_buf, sizeof(ntop_buf))));
+               ND_PRINT(" %s",
+                   addrtostr6(cp, ntop_buf, sizeof(ntop_buf)));
 
                break;
            }
 
        case T_A6:
            {
-               struct in6_addr a;
+               nd_ipv6 a;
                int pbit, pbyte;
                char ntop_buf[INET6_ADDRSTRLEN];
 
-               if (!ND_TTEST2(*cp, 1))
+               if (!ND_TTEST_1(cp))
                        return(NULL);
-               pbit = *cp;
+               pbit = GET_U_1(cp);
                pbyte = (pbit & ~7) / 8;
                if (pbit > 128) {
-                       ND_PRINT((ndo, " %u(bad plen)", pbit));
+                       ND_PRINT(" %u(bad plen)", pbit);
                        break;
                } else if (pbit < 128) {
-                       if (!ND_TTEST2(*(cp + 1), sizeof(a) - pbyte))
+                       if (!ND_TTEST_LEN(cp + 1, sizeof(a) - pbyte))
                                return(NULL);
-                       memset(&a, 0, sizeof(a));
-                       memcpy(&a.s6_addr[pbyte], cp + 1, sizeof(a) - pbyte);
-                       ND_PRINT((ndo, " %u %s", pbit,
-                           addrtostr6(&a, ntop_buf, sizeof(ntop_buf))));
+                       memset(a, 0, sizeof(a));
+                       memcpy(a + pbyte, cp + 1, sizeof(a) - pbyte);
+                       ND_PRINT(" %u %s", pbit,
+                           addrtostr6(&a, ntop_buf, sizeof(ntop_buf)));
                }
                if (pbit > 0) {
-                       ND_PRINT((ndo, " "));
-                       if (ns_nprint(ndo, cp + 1 + sizeof(a) - pbyte, bp) == NULL)
+                       ND_PRINT(" ");
+                       if (fqdn_print(ndo, cp + 1 + sizeof(a) - pbyte, bp) == NULL)
                                return(NULL);
                }
                break;
            }
 
+       case T_URI:
+               if (!ND_TTEST_LEN(cp, len))
+                       return(NULL);
+               ND_PRINT(" %u %u ", GET_BE_U_2(cp), GET_BE_U_2(cp + 2));
+               if (nd_printn(ndo, cp + 4, len - 4, ndo->ndo_snapend))
+                       return(NULL);
+               break;
+
        case T_OPT:
-               ND_PRINT((ndo, " UDPsize=%u", class));
+               ND_PRINT(" UDPsize=%u", class);
                if (opt_flags & 0x8000)
-                       ND_PRINT((ndo, " DO"));
+                       ND_PRINT(" DO");
+        if (cp < rp) {
+            ND_PRINT(" [");
+            while (cp < rp) {
+                cp = eopt_print(ndo, cp);
+                if (cp == NULL)
+                    return(NULL);
+                if (cp < rp)
+                    ND_PRINT(",");
+            }
+            ND_PRINT("]");
+        }
                break;
 
        case T_UNSPECA:         /* One long string */
-               if (!ND_TTEST2(*cp, len))
+               if (!ND_TTEST_LEN(cp, len))
                        return(NULL);
-               if (fn_printn(ndo, cp, len, ndo->ndo_snapend))
+               if (nd_printn(ndo, cp, len, ndo->ndo_snapend))
                        return(NULL);
                break;
 
@@ -541,29 +805,29 @@ ns_rprint(netdissect_options *ndo,
                        return(NULL);
                if (!ndo->ndo_vflag)
                        break;
-               ND_PRINT((ndo, " "));
-               if ((cp = ns_nprint(ndo, cp, bp)) == NULL)
+               ND_PRINT(" ");
+               if ((cp = fqdn_print(ndo, cp, bp)) == NULL)
                        return(NULL);
                cp += 6;
-               if (!ND_TTEST2(*cp, 2))
+               if (!ND_TTEST_2(cp))
                        return(NULL);
-               ND_PRINT((ndo, " fudge=%u", EXTRACT_16BITS(cp)));
+               ND_PRINT(" fudge=%u", GET_BE_U_2(cp));
                cp += 2;
-               if (!ND_TTEST2(*cp, 2))
+               if (!ND_TTEST_2(cp))
                        return(NULL);
-               ND_PRINT((ndo, " maclen=%u", EXTRACT_16BITS(cp)));
-               cp += 2 + EXTRACT_16BITS(cp);
-               if (!ND_TTEST2(*cp, 2))
+               ND_PRINT(" maclen=%u", GET_BE_U_2(cp));
+               cp += 2 + GET_BE_U_2(cp);
+               if (!ND_TTEST_2(cp))
                        return(NULL);
-               ND_PRINT((ndo, " origid=%u", EXTRACT_16BITS(cp)));
+               ND_PRINT(" origid=%u", GET_BE_U_2(cp));
                cp += 2;
-               if (!ND_TTEST2(*cp, 2))
+               if (!ND_TTEST_2(cp))
                        return(NULL);
-               ND_PRINT((ndo, " error=%u", EXTRACT_16BITS(cp)));
+               ND_PRINT(" error=%u", GET_BE_U_2(cp));
                cp += 2;
-               if (!ND_TTEST2(*cp, 2))
+               if (!ND_TTEST_2(cp))
                        return(NULL);
-               ND_PRINT((ndo, " otherlen=%u", EXTRACT_16BITS(cp)));
+               ND_PRINT(" otherlen=%u", GET_BE_U_2(cp));
                cp += 2;
            }
        }
@@ -571,42 +835,126 @@ ns_rprint(netdissect_options *ndo,
 }
 
 void
-ns_print(netdissect_options *ndo,
-         register const u_char *bp, u_int length, int is_mdns)
+domain_print(netdissect_options *ndo,
+             const u_char *bp, u_int length, int over_tcp, int is_mdns)
 {
-       register const HEADER *np;
-       register int qdcount, ancount, nscount, arcount;
-       register const u_char *cp;
+       const dns_header_t *np;
+       uint16_t flags, rcode, rdlen, type;
+       u_int qdcount, ancount, nscount, arcount;
+       u_int i;
+       const u_char *cp;
        uint16_t b2;
 
-       np = (const HEADER *)bp;
-       ND_TCHECK(*np);
+       ndo->ndo_protocol = "domain";
+
+       if (over_tcp) {
+               /*
+                * The message is prefixed with a two byte length field
+                * which gives the message length, excluding the two byte
+                * length field. (RFC 1035 - 4.2.2. TCP usage)
+                */
+               if (length < 2) {
+                       ND_PRINT(" [DNS over TCP: length %u < 2]", length);
+                       nd_print_invalid(ndo);
+                       return;
+               } else {
+                       length -= 2; /* excluding the two byte length field */
+                       if (GET_BE_U_2(bp) != length) {
+                               ND_PRINT(" [prefix length(%u) != length(%u)]",
+                                        GET_BE_U_2(bp), length);
+                               nd_print_invalid(ndo);
+                               return;
+                       } else {
+                               bp += 2;
+                               /* in over TCP case, we need to prepend a space
+                                * (not needed in over UDP case)
+                                */
+                               ND_PRINT(" ");
+                       }
+               }
+       }
+
+       np = (const dns_header_t *)bp;
+
+       if(length < sizeof(*np)) {
+               nd_print_protocol(ndo);
+               ND_PRINT(" [length %u < %zu]", length, sizeof(*np));
+               nd_print_invalid(ndo);
+               return;
+       }
+
+       ND_TCHECK_SIZE(np);
+       flags = GET_BE_U_2(np->flags);
        /* get the byte-order right */
-       qdcount = EXTRACT_16BITS(&np->qdcount);
-       ancount = EXTRACT_16BITS(&np->ancount);
-       nscount = EXTRACT_16BITS(&np->nscount);
-       arcount = EXTRACT_16BITS(&np->arcount);
+       qdcount = GET_BE_U_2(np->qdcount);
+       ancount = GET_BE_U_2(np->ancount);
+       nscount = GET_BE_U_2(np->nscount);
+       arcount = GET_BE_U_2(np->arcount);
 
-       if (DNS_QR(np)) {
+       /* find the opt record to extract extended rcode */
+       cp = (const u_char *)(np + 1);
+       rcode = DNS_RCODE(flags);
+       for (i = 0; i < qdcount; i++) {
+               if ((cp = ns_nskip(ndo, cp)) == NULL)
+                       goto print;
+               cp += 4;        /* skip QTYPE and QCLASS */
+               if (cp >= ndo->ndo_snapend)
+                       goto print;
+       }
+       for (i = 0; i < ancount + nscount; i++) {
+               if ((cp = ns_nskip(ndo, cp)) == NULL)
+                       goto print;
+               cp += 8;        /* skip TYPE, CLASS and TTL */
+               if (cp + 2 > ndo->ndo_snapend)
+                       goto print;
+               rdlen = GET_BE_U_2(cp);
+               cp += 2 + rdlen;
+               if (cp >= ndo->ndo_snapend)
+                       goto print;
+       }
+       for (i = 0; i < arcount; i++) {
+               if ((cp = ns_nskip(ndo, cp)) == NULL)
+                       goto print;
+               if (cp + 2 > ndo->ndo_snapend)
+                       goto print;
+               type = GET_BE_U_2(cp);
+               cp += 4;        /* skip TYPE and CLASS */
+               if (cp + 1 > ndo->ndo_snapend)
+                       goto print;
+               if (type == T_OPT) {
+                       rcode |= (GET_U_1(cp) << 4);
+                       goto print;
+               }
+               cp += 4;
+               if (cp + 2 > ndo->ndo_snapend)
+                       goto print;
+               rdlen = GET_BE_U_2(cp);
+               cp += 2 + rdlen;
+               if (cp >= ndo->ndo_snapend)
+                       goto print;
+       }
+
+ print:
+       if (DNS_QR(flags)) {
                /* this is a response */
-               ND_PRINT((ndo, "%d%s%s%s%s%s%s",
-                       EXTRACT_16BITS(&np->id),
-                       ns_ops[DNS_OPCODE(np)],
-                       ns_resp[DNS_RCODE(np)],
-                       DNS_AA(np)? "*" : "",
-                       DNS_RA(np)? "" : "-",
-                       DNS_TC(np)? "|" : "",
-                       DNS_AD(np)? "$" : ""));
+               ND_PRINT("%u%s%s%s%s%s%s",
+                       GET_BE_U_2(np->id),
+                       ns_ops[DNS_OPCODE(flags)],
+                       ns_rcode(rcode),
+                       DNS_AA(flags)? "*" : "",
+                       DNS_RA(flags)? "" : "-",
+                       DNS_TC(flags)? "|" : "",
+                       DNS_AD(flags)? "$" : "");
 
                if (qdcount != 1)
-                       ND_PRINT((ndo, " [%dq]", qdcount));
+                       ND_PRINT(" [%uq]", qdcount);
                /* Print QUESTION section on -vv */
                cp = (const u_char *)(np + 1);
-               while (qdcount--) {
-                       if (qdcount < EXTRACT_16BITS(&np->qdcount) - 1)
-                               ND_PRINT((ndo, ","));
+               for (i = 0; i < qdcount; i++) {
+                       if (i != 0)
+                               ND_PRINT(",");
                        if (ndo->ndo_vflag > 1) {
-                               ND_PRINT((ndo, " q:"));
+                               ND_PRINT(" q:");
                                if ((cp = ns_qprint(ndo, cp, bp, is_mdns)) == NULL)
                                        goto trunc;
                        } else {
@@ -615,132 +963,147 @@ ns_print(netdissect_options *ndo,
                                cp += 4;        /* skip QTYPE and QCLASS */
                        }
                }
-               ND_PRINT((ndo, " %d/%d/%d", ancount, nscount, arcount));
-               if (ancount--) {
+               ND_PRINT(" %u/%u/%u", ancount, nscount, arcount);
+               if (ancount) {
                        if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                goto trunc;
-                       while (cp < ndo->ndo_snapend && ancount--) {
-                               ND_PRINT((ndo, ","));
+                       ancount--;
+                       while (cp < ndo->ndo_snapend && ancount) {
+                               ND_PRINT(",");
                                if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                        goto trunc;
+                               ancount--;
                        }
                }
-               if (ancount > 0)
+               if (ancount)
                        goto trunc;
                /* Print NS and AR sections on -vv */
                if (ndo->ndo_vflag > 1) {
-                       if (cp < ndo->ndo_snapend && nscount--) {
-                               ND_PRINT((ndo, " ns:"));
+                       if (cp < ndo->ndo_snapend && nscount) {
+                               ND_PRINT(" ns:");
                                if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                        goto trunc;
-                               while (cp < ndo->ndo_snapend && nscount--) {
-                                       ND_PRINT((ndo, ","));
+                               nscount--;
+                               while (cp < ndo->ndo_snapend && nscount) {
+                                       ND_PRINT(",");
                                        if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                                goto trunc;
+                                       nscount--;
                                }
                        }
-                       if (nscount > 0)
+                       if (nscount)
                                goto trunc;
-                       if (cp < ndo->ndo_snapend && arcount--) {
-                               ND_PRINT((ndo, " ar:"));
+                       if (cp < ndo->ndo_snapend && arcount) {
+                               ND_PRINT(" ar:");
                                if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                        goto trunc;
-                               while (cp < ndo->ndo_snapend && arcount--) {
-                                       ND_PRINT((ndo, ","));
+                               arcount--;
+                               while (cp < ndo->ndo_snapend && arcount) {
+                                       ND_PRINT(",");
                                        if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                                goto trunc;
+                                       arcount--;
                                }
                        }
-                       if (arcount > 0)
+                       if (arcount)
                                goto trunc;
                }
        }
        else {
                /* this is a request */
-               ND_PRINT((ndo, "%d%s%s%s", EXTRACT_16BITS(&np->id), ns_ops[DNS_OPCODE(np)],
-                   DNS_RD(np) ? "+" : "",
-                   DNS_CD(np) ? "%" : ""));
+               ND_PRINT("%u%s%s%s", GET_BE_U_2(np->id),
+                         ns_ops[DNS_OPCODE(flags)],
+                         DNS_RD(flags) ? "+" : "",
+                         DNS_CD(flags) ? "%" : "");
 
                /* any weirdness? */
-               b2 = EXTRACT_16BITS(((const u_short *)np)+1);
+               b2 = GET_BE_U_2(((const u_short *)np) + 1);
                if (b2 & 0x6cf)
-                       ND_PRINT((ndo, " [b2&3=0x%x]", b2));
+                       ND_PRINT(" [b2&3=0x%x]", b2);
 
-               if (DNS_OPCODE(np) == IQUERY) {
+               if (DNS_OPCODE(flags) == IQUERY) {
                        if (qdcount)
-                               ND_PRINT((ndo, " [%dq]", qdcount));
+                               ND_PRINT(" [%uq]", qdcount);
                        if (ancount != 1)
-                               ND_PRINT((ndo, " [%da]", ancount));
+                               ND_PRINT(" [%ua]", ancount);
                }
                else {
                        if (ancount)
-                               ND_PRINT((ndo, " [%da]", ancount));
+                               ND_PRINT(" [%ua]", ancount);
                        if (qdcount != 1)
-                               ND_PRINT((ndo, " [%dq]", qdcount));
+                               ND_PRINT(" [%uq]", qdcount);
                }
                if (nscount)
-                       ND_PRINT((ndo, " [%dn]", nscount));
+                       ND_PRINT(" [%un]", nscount);
                if (arcount)
-                       ND_PRINT((ndo, " [%dau]", arcount));
+                       ND_PRINT(" [%uau]", arcount);
 
                cp = (const u_char *)(np + 1);
-               if (qdcount--) {
+               if (qdcount) {
                        cp = ns_qprint(ndo, cp, (const u_char *)np, is_mdns);
                        if (!cp)
                                goto trunc;
-                       while (cp < ndo->ndo_snapend && qdcount--) {
+                       qdcount--;
+                       while (cp < ndo->ndo_snapend && qdcount) {
                                cp = ns_qprint(ndo, (const u_char *)cp,
                                               (const u_char *)np,
                                               is_mdns);
                                if (!cp)
                                        goto trunc;
+                               qdcount--;
                        }
                }
-               if (qdcount > 0)
+               if (qdcount)
                        goto trunc;
 
                /* Print remaining sections on -vv */
                if (ndo->ndo_vflag > 1) {
-                       if (ancount--) {
+                       if (ancount) {
                                if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                        goto trunc;
-                               while (cp < ndo->ndo_snapend && ancount--) {
-                                       ND_PRINT((ndo, ","));
+                               ancount--;
+                               while (cp < ndo->ndo_snapend && ancount) {
+                                       ND_PRINT(",");
                                        if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                                goto trunc;
+                                       ancount--;
                                }
                        }
-                       if (ancount > 0)
+                       if (ancount)
                                goto trunc;
-                       if (cp < ndo->ndo_snapend && nscount--) {
-                               ND_PRINT((ndo, " ns:"));
+                       if (cp < ndo->ndo_snapend && nscount) {
+                               ND_PRINT(" ns:");
                                if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                        goto trunc;
-                               while (nscount-- && cp < ndo->ndo_snapend) {
-                                       ND_PRINT((ndo, ","));
+                               nscount--;
+                               while (cp < ndo->ndo_snapend && nscount) {
+                                       ND_PRINT(",");
                                        if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                                goto trunc;
+                                       nscount--;
                                }
                        }
                        if (nscount > 0)
                                goto trunc;
-                       if (cp < ndo->ndo_snapend && arcount--) {
-                               ND_PRINT((ndo, " ar:"));
+                       if (cp < ndo->ndo_snapend && arcount) {
+                               ND_PRINT(" ar:");
                                if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                        goto trunc;
-                               while (cp < ndo->ndo_snapend && arcount--) {
-                                       ND_PRINT((ndo, ","));
+                               arcount--;
+                               while (cp < ndo->ndo_snapend && arcount) {
+                                       ND_PRINT(",");
                                        if ((cp = ns_rprint(ndo, cp, bp, is_mdns)) == NULL)
                                                goto trunc;
+                                       arcount--;
                                }
                        }
-                       if (arcount > 0)
+                       if (arcount)
                                goto trunc;
                }
        }
-       ND_PRINT((ndo, " (%d)", length));
+       ND_PRINT(" (%u)", length);
        return;
 
   trunc:
-       ND_PRINT((ndo, "[|domain]"));
+       nd_print_trunc(ndo);
 }