Update ND_PRINT() as a variadic macro

[tcpdump] / print-ascii.c

diff --git a/print-ascii.c b/print-ascii.c
index 4ef38a17a50dba22e246d1f3a712665e0864fa4c..27ecc2e2e811a128cb0c5bbabe8222efa6c79d53 100644 (file)
--- a/print-ascii.c
+++ b/print-ascii.c
@@ -46,6 +46,7 @@
 #include <stdio.h>
 
 #include "netdissect.h"
+#include "extract.h"
 
 #define ASCII_LINELENGTH 300
 #define HEXDUMP_BYTES_PER_LINE 16
@@ -59,14 +60,15 @@ ascii_print(netdissect_options *ndo,
             const u_char *cp, u_int length)
 {
        u_int caplength;
-       register u_char s;
+       u_char s;
 
        caplength = (ndo->ndo_snapend >= cp) ? ndo->ndo_snapend - cp : 0;
        if (length > caplength)
                length = caplength;
-       ND_PRINT((ndo, "\n"));
+       ND_PRINT("\n");
        while (length > 0) {
-               s = *cp++;
+               s = EXTRACT_U_1(cp);
+               cp++;
                length--;
                if (s == '\r') {
                        /*
@@ -78,26 +80,26 @@ ascii_print(netdissect_options *ndo,
                         *
                         * In the middle of a line, just print a '.'.
                         */
-                       if (length > 1 && *cp != '\n')
-                               ND_PRINT((ndo, "."));
+                       if (length > 1 && EXTRACT_U_1(cp) != '\n')
+                               ND_PRINT(".");
                } else {
                        if (!ND_ISGRAPH(s) &&
                            (s != '\t' && s != ' ' && s != '\n'))
-                               ND_PRINT((ndo, "."));
+                               ND_PRINT(".");
                        else
-                               ND_PRINT((ndo, "%c", s));
+                               ND_PRINT("%c", s);
                }
        }
 }
 
 void
-hex_and_ascii_print_with_offset(netdissect_options *ndo, register const char *ident,
-    register const u_char *cp, register u_int length, register u_int oset)
+hex_and_ascii_print_with_offset(netdissect_options *ndo, const char *ident,
+    const u_char *cp, u_int length, u_int oset)
 {
        u_int caplength;
-       register u_int i;
-       register int s1, s2;
-       register int nshorts;
+       u_int i;
+       int s1, s2;
+       int nshorts;
        char hexstuff[HEXDUMP_SHORTS_PER_LINE*HEXDUMP_HEXSTUFF_PER_SHORT+1], *hsp;
        char asciistuff[ASCII_LINELENGTH+1], *asp;
 
@@ -118,9 +120,9 @@ hex_and_ascii_print_with_offset(netdissect_options *ndo, register const char *id
                i++;
                if (i >= HEXDUMP_SHORTS_PER_LINE) {
                        *hsp = *asp = '\0';
-                       ND_PRINT((ndo, "%s0x%04x: %-*s  %s",
+                       ND_PRINT("%s0x%04x: %-*s  %s",
                            ident, oset, HEXDUMP_HEXSTUFF_PER_LINE,
-                           hexstuff, asciistuff));
+                           hexstuff, asciistuff);
                        i = 0; hsp = hexstuff; asp = asciistuff;
                        oset += HEXDUMP_BYTES_PER_LINE;
                }
@@ -135,15 +137,15 @@ hex_and_ascii_print_with_offset(netdissect_options *ndo, register const char *id
        }
        if (i > 0) {
                *hsp = *asp = '\0';
-               ND_PRINT((ndo, "%s0x%04x: %-*s  %s",
+               ND_PRINT("%s0x%04x: %-*s  %s",
                     ident, oset, HEXDUMP_HEXSTUFF_PER_LINE,
-                    hexstuff, asciistuff));
+                    hexstuff, asciistuff);
        }
 }
 
 void
-hex_and_ascii_print(netdissect_options *ndo, register const char *ident,
-    register const u_char *cp, register u_int length)
+hex_and_ascii_print(netdissect_options *ndo, const char *ident,
+    const u_char *cp, u_int length)
 {
        hex_and_ascii_print_with_offset(ndo, ident, cp, length, 0);
 }
@@ -157,8 +159,8 @@ hex_print_with_offset(netdissect_options *ndo,
                      u_int oset)
 {
        u_int caplength;
-       register u_int i, s;
-       register int nshorts;
+       u_int i, s;
+       int nshorts;
 
        caplength = (ndo->ndo_snapend >= cp) ? ndo->ndo_snapend - cp : 0;
        if (length > caplength)
@@ -167,16 +169,16 @@ hex_print_with_offset(netdissect_options *ndo,
        i = 0;
        while (--nshorts >= 0) {
                if ((i++ % 8) == 0) {
-                       ND_PRINT((ndo,"%s0x%04x: ", ident, oset));
+                       ND_PRINT("%s0x%04x: ", ident, oset);
                        oset += HEXDUMP_BYTES_PER_LINE;
                }
                s = *cp++;
-               ND_PRINT((ndo," %02x%02x", s, *cp++));
+               ND_PRINT(" %02x%02x", s, *cp++);
        }
        if (length & 1) {
                if ((i % 8) == 0)
-                       ND_PRINT((ndo,"%s0x%04x: ", ident, oset));
-               ND_PRINT((ndo," %02x", *cp));
+                       ND_PRINT("%s0x%04x: ", ident, oset);
+               ND_PRINT(" %02x", *cp);
        }
 }