ICMP: Fix a typo in a comment.

[tcpdump] / print-domain.c

diff --git a/print-domain.c b/print-domain.c
index 4523d67f4aebb7e6b9a1d3cf33ac8b504b7ab924..c3c6fd8f3d542650f89ade1fa5d114031f56af2a 100644 (file)
--- a/print-domain.c
+++ b/print-domain.c
@@ -74,12 +74,15 @@ ns_nskip(netdissect_options *ndo,
        i = GET_U_1(cp);
        cp++;
        while (i) {
        i = GET_U_1(cp);
        cp++;
        while (i) {

-               if ((i & INDIR_MASK) == INDIR_MASK)
+               switch (i & TYPE_MASK) {
+
+               case TYPE_INDIR:

                        return (cp + 1);
                        return (cp + 1);

-               if ((i & INDIR_MASK) == EDNS0_MASK) {
+
+               case TYPE_EDNS0: {

                        int bitlen, bytelen;
 
                        int bitlen, bytelen;
 

-                       if ((i & ~INDIR_MASK) != EDNS0_ELT_BITLABEL)
+                       if ((i & ~TYPE_MASK) != EDNS0_ELT_BITLABEL)

                                return(NULL); /* unknown ELT */
                        if (!ND_TTEST_1(cp))
                                return (NULL);
                                return(NULL); /* unknown ELT */
                        if (!ND_TTEST_1(cp))
                                return (NULL);


@@ -88,8 +91,16 @@ ns_nskip(netdissect_options *ndo,
                        cp++;
                        bytelen = (bitlen + 7) / 8;
                        cp += bytelen;
                        cp++;
                        bytelen = (bitlen + 7) / 8;
                        cp += bytelen;

-               } else
+               }
+               break;
+
+               case TYPE_RESERVED:
+                       return (NULL);
+
+               case TYPE_LABEL:

                        cp += i;
                        cp += i;

+                       break;
+               }

                if (!ND_TTEST_1(cp))
                        return (NULL);
                i = GET_U_1(cp);
                if (!ND_TTEST_1(cp))
                        return (NULL);
                i = GET_U_1(cp);


@@ -116,25 +127,19 @@ blabel_print(netdissect_options *ndo,
        /* print the bit string as a hex string */
        ND_PRINT("\\[x");
        for (bitp = cp + 1, b = bitlen; bitp < lim && b > 7; b -= 8, bitp++) {
        /* print the bit string as a hex string */
        ND_PRINT("\\[x");
        for (bitp = cp + 1, b = bitlen; bitp < lim && b > 7; b -= 8, bitp++) {

-               ND_TCHECK_1(bitp);

                ND_PRINT("%02x", GET_U_1(bitp));
        }
        if (b > 4) {
                ND_PRINT("%02x", GET_U_1(bitp));
        }
        if (b > 4) {

-               ND_TCHECK_1(bitp);

                tc = GET_U_1(bitp);
                bitp++;
                ND_PRINT("%02x", tc & (0xff << (8 - b)));
        } else if (b > 0) {
                tc = GET_U_1(bitp);
                bitp++;
                ND_PRINT("%02x", tc & (0xff << (8 - b)));
        } else if (b > 0) {

-               ND_TCHECK_1(bitp);

                tc = GET_U_1(bitp);
                bitp++;
                ND_PRINT("%1x", ((tc >> 4) & 0x0f) & (0x0f << (4 - b)));
        }
        ND_PRINT("/%u]", bitlen);
        return lim;
                tc = GET_U_1(bitp);
                bitp++;
                ND_PRINT("%1x", ((tc >> 4) & 0x0f) & (0x0f << (4 - b)));
        }
        ND_PRINT("/%u]", bitlen);
        return lim;

-trunc:
-       ND_PRINT(".../%u]", bitlen);
-       return NULL;

 }
 
 static int
 }
 
 static int


@@ -146,9 +151,11 @@ labellen(netdissect_options *ndo,
        if (!ND_TTEST_1(cp))
                return(-1);
        i = GET_U_1(cp);
        if (!ND_TTEST_1(cp))
                return(-1);
        i = GET_U_1(cp);

-       if ((i & INDIR_MASK) == EDNS0_MASK) {
+       switch (i & TYPE_MASK) {
+
+       case TYPE_EDNS0: {

                u_int bitlen, elt;
                u_int bitlen, elt;

-               if ((elt = (i & ~INDIR_MASK)) != EDNS0_ELT_BITLABEL) {
+               if ((elt = (i & ~TYPE_MASK)) != EDNS0_ELT_BITLABEL) {

                        ND_PRINT("<ELT %d>", elt);
                        return(-1);
                }
                        ND_PRINT("<ELT %d>", elt);
                        return(-1);
                }


@@ -157,8 +164,20 @@ labellen(netdissect_options *ndo,
                if ((bitlen = GET_U_1(cp + 1)) == 0)
                        bitlen = 256;
                return(((bitlen + 7) / 8) + 1);
                if ((bitlen = GET_U_1(cp + 1)) == 0)
                        bitlen = 256;
                return(((bitlen + 7) / 8) + 1);

-       } else
+       }
+
+       case TYPE_INDIR:
+       case TYPE_LABEL:

                return(i);
                return(i);

+
+       default:
+               /*
+                * TYPE_RESERVED, but we use default to suppress compiler
+                * warnings about falling out of the switch statement.
+                */
+               ND_PRINT("<BAD LABEL TYPE>");
+               return(-1);
+       }

 }
 
 /* print a <domain-name> */
 }
 
 /* print a <domain-name> */


@@ -171,6 +190,7 @@ fqdn_print(netdissect_options *ndo,
        int compress = 0;
        u_int elt;
        u_int offset, max_offset;
        int compress = 0;
        u_int elt;
        u_int offset, max_offset;

+       u_int name_chars = 0;

 
        if ((l = labellen(ndo, cp)) == (u_int)-1)
                return(NULL);
 
        if ((l = labellen(ndo, cp)) == (u_int)-1)
                return(NULL);


@@ -179,14 +199,16 @@ fqdn_print(netdissect_options *ndo,
        max_offset = (u_int)(cp - bp);
        i = GET_U_1(cp);
        cp++;
        max_offset = (u_int)(cp - bp);
        i = GET_U_1(cp);
        cp++;

-       if ((i & INDIR_MASK) != INDIR_MASK) {
+       if ((i & TYPE_MASK) != TYPE_INDIR) {

                compress = 0;
                rp = cp + l;
        }
 
                compress = 0;
                rp = cp + l;
        }
 

-       if (i != 0)
+       if (i != 0) {

                while (i && cp < ndo->ndo_snapend) {
                while (i && cp < ndo->ndo_snapend) {

-                       if ((i & INDIR_MASK) == INDIR_MASK) {
+                       switch (i & TYPE_MASK) {
+
+                       case TYPE_INDIR:

                                if (!compress) {
                                        rp = cp + 1;
                                        compress = 1;
                                if (!compress) {
                                        rp = cp + 1;
                                        compress = 1;


@@ -210,16 +232,16 @@ fqdn_print(netdissect_options *ndo,
                                }
                                max_offset = offset;
                                cp = bp + offset;
                                }
                                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);
                                if (!ND_TTEST_1(cp))
                                        return(NULL);
                                i = GET_U_1(cp);

+                               if ((l = labellen(ndo, cp)) == (u_int)-1)
+                                       return(NULL);

                                cp++;
                                continue;
                                cp++;
                                continue;

-                       }
-                       if ((i & INDIR_MASK) == EDNS0_MASK) {
-                               elt = (i & ~INDIR_MASK);
+
+                       case TYPE_EDNS0:
+                               elt = (i & ~TYPE_MASK);

                                switch(elt) {
                                case EDNS0_ELT_BITLABEL:
                                        if (blabel_print(ndo, cp) == NULL)
                                switch(elt) {
                                case EDNS0_ELT_BITLABEL:
                                        if (blabel_print(ndo, cp) == NULL)


@@ -230,23 +252,41 @@ fqdn_print(netdissect_options *ndo,
                                        ND_PRINT("<ELT %u>", elt);
                                        return(NULL);
                                }
                                        ND_PRINT("<ELT %u>", elt);
                                        return(NULL);
                                }

-                       } else {
-                               if (nd_printn(ndo, cp, l, ndo->ndo_snapend))
-                                       return(NULL);
+                               break;
+
+                       case TYPE_RESERVED:
+                               ND_PRINT("<BAD LABEL TYPE>");
+                               return(NULL);
+
+                       case TYPE_LABEL:
+                               if (name_chars + l <= MAXCDNAME) {
+                                       if (nd_printn(ndo, cp, l, ndo->ndo_snapend))
+                                               return(NULL);
+                               } else if (name_chars < MAXCDNAME) {
+                                       if (nd_printn(ndo, cp,
+                                           MAXCDNAME - name_chars, ndo->ndo_snapend))
+                                               return(NULL);
+                               }
+                               name_chars += l;
+                               break;

                        }
 
                        cp += l;
                        }
 
                        cp += l;

-                       ND_PRINT(".");
-                       if ((l = labellen(ndo, cp)) == (u_int)-1)
-                               return(NULL);
+                       if (name_chars <= MAXCDNAME)
+                               ND_PRINT(".");
+                       name_chars++;

                        if (!ND_TTEST_1(cp))
                                return(NULL);
                        i = GET_U_1(cp);
                        if (!ND_TTEST_1(cp))
                                return(NULL);
                        i = GET_U_1(cp);

+                       if ((l = labellen(ndo, cp)) == (u_int)-1)
+                               return(NULL);

                        cp++;
                        if (!compress)
                                rp += l + 1;
                }
                        cp++;
                        if (!compress)
                                rp += l + 1;
                }

-       else
+               if (name_chars > MAXCDNAME)
+                       ND_PRINT("<DOMAIN NAME TOO LONG>");
+       } else

                ND_PRINT(".");
        return (rp);
 }
                ND_PRINT(".");
        return (rp);
 }


@@ -673,9 +713,7 @@ ns_rprint(netdissect_options *ndo,
        case T_NS:
        case T_CNAME:
        case T_PTR:
        case T_NS:
        case T_CNAME:
        case T_PTR:

-#ifdef T_DNAME

        case T_DNAME:
        case T_DNAME:

-#endif

                ND_PRINT(" ");
                if (fqdn_print(ndo, cp, bp) == NULL)
                        return(NULL);
                ND_PRINT(" ");
                if (fqdn_print(ndo, cp, bp) == NULL)
                        return(NULL);


@@ -758,10 +796,8 @@ ns_rprint(netdissect_options *ndo,
                        ND_PRINT(" %u(bad plen)", pbit);
                        break;
                } else if (pbit < 128) {
                        ND_PRINT(" %u(bad plen)", pbit);
                        break;
                } else if (pbit < 128) {

-                       if (!ND_TTEST_LEN(cp + 1, sizeof(a) - pbyte))
-                               return(NULL);

                        memset(a, 0, sizeof(a));
                        memset(a, 0, sizeof(a));

-                       memcpy(a + pbyte, cp + 1, sizeof(a) - pbyte);
+                       GET_CPY_BYTES(a + pbyte, cp + 1, sizeof(a) - pbyte);

                        ND_PRINT(" %u %s", pbit,
                            addrtostr6(&a, ntop_buf, sizeof(ntop_buf)));
                }
                        ND_PRINT(" %u %s", pbit,
                            addrtostr6(&a, ntop_buf, sizeof(ntop_buf)));
                }