Makefile.in: don't remove configure and config.h.in in make distclean.

[tcpdump] / print-pppoe.c

diff --git a/print-pppoe.c b/print-pppoe.c
index e3d6e22287b2672b7cd4d863f9407a70d98c18ab..65518dff16d97adde75b1d7eb537d602354f7998 100644 (file)
--- a/print-pppoe.c
+++ b/print-pppoe.c
@@ -29,6 +29,9 @@
 
 #include "netdissect-stdinc.h"
 
+#include "netdissect-ctype.h"
+
+#define ND_LONGJMP_FROM_TCHECK
 #include "netdissect.h"
 #include "extract.h"
 
@@ -84,11 +87,11 @@ static const struct tok pppoetag2str[] = {
 #define PPPOE_HDRLEN 6
 #define MAXTAGPRINT 80
 
-u_int
+void
 pppoe_if_print(netdissect_options *ndo, const struct pcap_pkthdr *h, const u_char *p)
 {
-       ndo->ndo_protocol = "pppoe_if";
-       return (pppoe_print(ndo, p, h->len));
+       ndo->ndo_protocol = "pppoe";
+       ndo->ndo_ll_hdr_len += pppoe_print(ndo, p, h->len);
 }
 
 u_int
@@ -100,8 +103,8 @@ pppoe_print(netdissect_options *ndo, const u_char *bp, u_int length)
 
        ndo->ndo_protocol = "pppoe";
        if (length < PPPOE_HDRLEN) {
-               ND_PRINT("truncated-pppoe %u", length);
-               return (length);
+               ND_PRINT(" (length %u < %u)", length, PPPOE_HDRLEN);
+               goto invalid;
        }
        length -= PPPOE_HDRLEN;
        pppoe_packet = bp;
@@ -143,7 +146,6 @@ pppoe_print(netdissect_options *ndo, const u_char *bp, u_int length)
                 * tag_type is previous tag or 0xffff for first iteration
                 */
                while (tag_type && p < pppoe_payload + pppoe_length) {
-                       ND_TCHECK_4(p);
                        tag_type = GET_BE_U_2(p);
                        tag_len = GET_BE_U_2(p + 2);
                        p += 4;
@@ -158,7 +160,7 @@ pppoe_print(netdissect_options *ndo, const u_char *bp, u_int length)
                                /* TODO print UTF-8 decoded text */
                                ND_TCHECK_LEN(p, tag_len);
                                for (v = p; v < p + tag_len && tag_str_len < MAXTAGPRINT-1; v++)
-                                       if (ND_ISPRINT(GET_U_1(v))) {
+                                       if (ND_ASCII_ISPRINT(GET_U_1(v))) {
                                                tag_str[tag_str_len++] = GET_U_1(v);
                                                ascii_count++;
                                        } else {
@@ -190,14 +192,15 @@ pppoe_print(netdissect_options *ndo, const u_char *bp, u_int length)
                        p += tag_len;
                        /* p points to next tag */
                }
-               return (0);
+               return PPPOE_HDRLEN;
        } else {
                /* PPPoE data */
                ND_PRINT(" ");
                return (PPPOE_HDRLEN + ppp_print(ndo, pppoe_payload, pppoe_length));
        }
+       /* NOTREACHED */
 
-trunc:
-       nd_print_trunc(ndo);
-       return (PPPOE_HDRLEN);
+invalid:
+       nd_print_invalid(ndo);
+       return 0;
 }