Use nd_ types, add EXTRACT_ calls, clean up signed vs. unsigned.

[tcpdump] / print-vjc.c

diff --git a/print-vjc.c b/print-vjc.c
index 597ba060456750dda87f4c91d377ea8f9a6e3df6..0743cfa0d28f46d745642106b14a70c5329fe892 100644 (file)
--- a/print-vjc.c
+++ b/print-vjc.c
@@ -30,6 +30,7 @@
 #include <netdissect-stdinc.h>
 
 #include "netdissect.h"
+#include "extract.h"
 #include "slcompress.h"
 #include "ppp.h"
 
@@ -83,11 +84,11 @@
  * to the packet capture mechanism.
  */
 int
-vjc_print(netdissect_options *ndo, register const u_char *bp, u_short proto _U_)
+vjc_print(netdissect_options *ndo, const u_char *bp, u_short proto _U_)
 {
        int i;
 
-       switch (bp[0] & 0xf0) {
+       switch (EXTRACT_U_1(bp) & 0xf0) {
        case TYPE_IP:
                if (ndo->ndo_eflag)
                        ND_PRINT((ndo, "(vjc type=IP) "));
@@ -100,12 +101,12 @@ vjc_print(netdissect_options *ndo, register const u_char *bp, u_short proto _U_)
                if (ndo->ndo_eflag)
                        ND_PRINT((ndo, "(vjc type=compressed TCP) "));
                for (i = 0; i < 8; i++) {
-                       if (bp[1] & (0x80 >> i))
+                       if (EXTRACT_U_1(bp + 1) & (0x80 >> i))
                                ND_PRINT((ndo, "%c", "?CI?SAWU"[i]));
                }
-               if (bp[1])
+               if (EXTRACT_U_1(bp + 1))
                        ND_PRINT((ndo, " "));
-               ND_PRINT((ndo, "C=0x%02x ", bp[2]));
+               ND_PRINT((ndo, "C=0x%02x ", EXTRACT_U_1(bp + 2)));
                ND_PRINT((ndo, "sum=0x%04x ", *(const u_short *)(bp + 3)));
                return -1;
        case TYPE_ERROR:
@@ -114,7 +115,7 @@ vjc_print(netdissect_options *ndo, register const u_char *bp, u_short proto _U_)
                return -1;
        default:
                if (ndo->ndo_eflag)
-                       ND_PRINT((ndo, "(vjc type=0x%02x) ", bp[0] & 0xf0));
+                       ND_PRINT((ndo, "(vjc type=0x%02x) ", EXTRACT_U_1(bp) & 0xf0));
                return -1;
        }
 }