]> The Tcpdump Group git mirrors - tcpdump/blobdiff - print-fr.c
projects / tcpdump / blobdiff
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
ICMPv6: Add the IPv6-Only flag
[tcpdump] / print-fr.c
diff --git a/print-fr.c b/print-fr.c
index 8c8bf781787870f022ef563b350c3c0a4f4ede65..8565dae5ff273b7feaf789b3e081e7b69c68ded6 100644 (file)
--- a/print-fr.c
+++ b/print-fr.c
@@ -22,10 +22,10 @@
 /* \summary: Frame Relay printer */
 
 #ifdef HAVE_CONFIG_H
 /* \summary: Frame Relay printer */
 
 #ifdef HAVE_CONFIG_H
-#include "config.h"
+#include <config.h>
 #endif
 
 #endif
 
-#include <netdissect-stdinc.h>
+#include "netdissect-stdinc.h"
 
 #include <stdio.h>
 #include <string.h>
 
 #include <stdio.h>
 #include <string.h>
@@ -98,9 +98,9 @@ static const struct tok frf_flag_values[] = {
  * 0 on invalid address, -1 on truncated packet
  * save the flags dep. on address length
  */
  * 0 on invalid address, -1 on truncated packet
  * save the flags dep. on address length
  */
-static int parse_q922_addr(netdissect_options *ndo,
+static int parse_q922_header(netdissect_options *ndo,
                            const u_char *p, u_int *dlci,
                            const u_char *p, u_int *dlci,
-                           u_int *addr_len, uint8_t *flags, u_int length)
+                           u_int *addr_len, uint32_t *flags, u_int length)
 {
        if (!ND_TTEST_1(p) || length < 1)
                return -1;
 {
        if (!ND_TTEST_1(p) || length < 1)
                return -1;
@@ -112,10 +112,8 @@ static int parse_q922_addr(netdissect_options *ndo,
        *addr_len = 2;
        *dlci = ((EXTRACT_U_1(p) & 0xFC) << 2) | ((EXTRACT_U_1(p + 1) & 0xF0) >> 4);
 
        *addr_len = 2;
        *dlci = ((EXTRACT_U_1(p) & 0xFC) << 2) | ((EXTRACT_U_1(p + 1) & 0xF0) >> 4);
 
-        flags[0] = EXTRACT_U_1(p) & 0x02; /* populate the first flag fields */
-        flags[1] = EXTRACT_U_1(p + 1) & 0x0c;
-        flags[2] = 0;           /* clear the rest of the flags */
-        flags[3] = 0;
+       *flags = ((EXTRACT_U_1(p) & 0x02) << 24) |      /* CR flag */
+                ((EXTRACT_U_1(p + 1) & 0x0e) << 16);   /* FECN,BECN,DE flags */
 
        if (EXTRACT_U_1(p + 1) & FR_EA_BIT)
                return 1;       /* 2-byte Q.922 address */
 
        if (EXTRACT_U_1(p + 1) & FR_EA_BIT)
                return 1;       /* 2-byte Q.922 address */
@@ -137,24 +135,24 @@ static int parse_q922_addr(netdissect_options *ndo,
        if ((EXTRACT_U_1(p) & FR_EA_BIT) == 0)
                return 0; /* more than 4 bytes of Q.922 address? */
 
        if ((EXTRACT_U_1(p) & FR_EA_BIT) == 0)
                return 0; /* more than 4 bytes of Q.922 address? */
 
-        flags[3] = EXTRACT_U_1(p) & 0x02;
+       *flags = *flags | (EXTRACT_U_1(p) & 0x02);      /* SDLC flag */
 
         *dlci = (*dlci << 6) | (EXTRACT_U_1(p) >> 2);
 
        return 1;
 }
 
 
         *dlci = (*dlci << 6) | (EXTRACT_U_1(p) >> 2);
 
        return 1;
 }
 
-char *
+const char *
 q922_string(netdissect_options *ndo, const u_char *p, u_int length)
 {
 
     static u_int dlci, addr_len;
 q922_string(netdissect_options *ndo, const u_char *p, u_int length)
 {
 
     static u_int dlci, addr_len;
-    static uint8_t flags[4];
+    static uint32_t flags;
     static char buffer[sizeof("DLCI xxxxxxxxxx")];
     memset(buffer, 0, sizeof(buffer));
 
     static char buffer[sizeof("DLCI xxxxxxxxxx")];
     memset(buffer, 0, sizeof(buffer));
 
-    if (parse_q922_addr(ndo, p, &dlci, &addr_len, flags, length) == 1){
-        snprintf(buffer, sizeof(buffer), "DLCI %u", dlci);
+    if (parse_q922_header(ndo, p, &dlci, &addr_len, &flags, length) == 1){
+        nd_snprintf(buffer, sizeof(buffer), "DLCI %u", dlci);
     }
 
     return buffer;
     }
 
     return buffer;
@@ -188,54 +186,55 @@ q922_string(netdissect_options *ndo, const u_char *p, u_int length)
 */
 
 static void
 */
 
 static void
-fr_hdr_print(netdissect_options *ndo,
-             int length, u_int addr_len, u_int dlci, uint8_t *flags, uint16_t nlpid)
+fr_hdr_print(netdissect_options *ndo, int length, u_int addr_len,
+            u_int dlci, uint32_t flags, uint16_t nlpid)
 {
     if (ndo->ndo_qflag) {
 {
     if (ndo->ndo_qflag) {
-        ND_PRINT((ndo, "Q.922, DLCI %u, length %u: ",
+        ND_PRINT("Q.922, DLCI %u, length %u: ",
                      dlci,
                      dlci,
-                     length));
+                     length);
     } else {
         if (nlpid <= 0xff) /* if its smaller than 256 then its a NLPID */
     } else {
         if (nlpid <= 0xff) /* if its smaller than 256 then its a NLPID */
-            ND_PRINT((ndo, "Q.922, hdr-len %u, DLCI %u, Flags [%s], NLPID %s (0x%02x), length %u: ",
+            ND_PRINT("Q.922, hdr-len %u, DLCI %u, Flags [%s], NLPID %s (0x%02x), length %u: ",
                          addr_len,
                          dlci,
                          addr_len,
                          dlci,
-                         bittok2str(fr_header_flag_values, "none", EXTRACT_BE_U_4(flags)),
+                         bittok2str(fr_header_flag_values, "none", flags),
                          tok2str(nlpid_values,"unknown", nlpid),
                          nlpid,
                          tok2str(nlpid_values,"unknown", nlpid),
                          nlpid,
-                         length));
+                         length);
         else /* must be an ethertype */
         else /* must be an ethertype */
-            ND_PRINT((ndo, "Q.922, hdr-len %u, DLCI %u, Flags [%s], cisco-ethertype %s (0x%04x), length %u: ",
+            ND_PRINT("Q.922, hdr-len %u, DLCI %u, Flags [%s], cisco-ethertype %s (0x%04x), length %u: ",
                          addr_len,
                          dlci,
                          addr_len,
                          dlci,
-                         bittok2str(fr_header_flag_values, "none", EXTRACT_BE_U_4(flags)),
+                         bittok2str(fr_header_flag_values, "none", flags),
                          tok2str(ethertype_values, "unknown", nlpid),
                          nlpid,
                          tok2str(ethertype_values, "unknown", nlpid),
                          nlpid,
-                         length));
+                         length);
     }
 }
 
 u_int
 fr_if_print(netdissect_options *ndo,
     }
 }
 
 u_int
 fr_if_print(netdissect_options *ndo,
-            const struct pcap_pkthdr *h, register const u_char *p)
+            const struct pcap_pkthdr *h, const u_char *p)
 {
 {
-       register u_int length = h->len;
-       register u_int caplen = h->caplen;
+       u_int length = h->len;
+       u_int caplen = h->caplen;
 
 
+       ndo->ndo_protocol = "fr_if";
         ND_TCHECK_4(p); /* minimum frame header length */
 
         if ((length = fr_print(ndo, p, length)) == 0)
             return (0);
         else
             return length;
         ND_TCHECK_4(p); /* minimum frame header length */
 
         if ((length = fr_print(ndo, p, length)) == 0)
             return (0);
         else
             return length;
- trunc:
-        ND_PRINT((ndo, "[|fr]"));
+trunc:
+        nd_print_trunc(ndo);
         return caplen;
 }
 
 u_int
 fr_print(netdissect_options *ndo,
         return caplen;
 }
 
 u_int
 fr_print(netdissect_options *ndo,
-         register const u_char *p, u_int length)
+         const u_char *p, u_int length)
 {
        int ret;
        uint16_t extracted_ethertype;
 {
        int ret;
        uint16_t extracted_ethertype;
@@ -243,13 +242,14 @@ fr_print(netdissect_options *ndo,
        u_int addr_len;
        uint16_t nlpid;
        u_int hdr_len;
        u_int addr_len;
        uint16_t nlpid;
        u_int hdr_len;
-       uint8_t flags[4];
+       uint32_t flags;
 
 
-       ret = parse_q922_addr(ndo, p, &dlci, &addr_len, flags, length);
+       ndo->ndo_protocol = "fr";
+       ret = parse_q922_header(ndo, p, &dlci, &addr_len, &flags, length);
        if (ret == -1)
                goto trunc;
        if (ret == 0) {
        if (ret == -1)
                goto trunc;
        if (ret == 0) {
-               ND_PRINT((ndo, "Q.922, invalid address"));
+               ND_PRINT("Q.922, invalid address");
                return 0;
        }
 
                return 0;
        }
 
@@ -265,7 +265,7 @@ fr_print(netdissect_options *ndo,
                  */
                if (!ND_TTEST_2(p + addr_len) || length < addr_len + 2) {
                         /* no Ethertype */
                  */
                if (!ND_TTEST_2(p + addr_len) || length < addr_len + 2) {
                         /* no Ethertype */
-                        ND_PRINT((ndo, "UI %02x! ", EXTRACT_U_1(p + addr_len)));
+                        ND_PRINT("UI %02x! ", EXTRACT_U_1(p + addr_len));
                 } else {
                         extracted_ethertype = EXTRACT_BE_U_2(p + addr_len);
 
                 } else {
                         extracted_ethertype = EXTRACT_BE_U_2(p + addr_len);
 
@@ -279,7 +279,7 @@ fr_print(netdissect_options *ndo,
                                             ndo->ndo_snapend-p-addr_len-ETHERTYPE_LEN,
                                             NULL, NULL) == 0)
                                 /* ether_type not known, probably it wasn't one */
                                             ndo->ndo_snapend-p-addr_len-ETHERTYPE_LEN,
                                             NULL, NULL) == 0)
                                 /* ether_type not known, probably it wasn't one */
-                                ND_PRINT((ndo, "UI %02x! ", EXTRACT_U_1(p + addr_len)));
+                                ND_PRINT("UI %02x! ", EXTRACT_U_1(p + addr_len));
                         else
                                 return addr_len + 2;
                 }
                         else
                                 return addr_len + 2;
                 }
@@ -295,7 +295,7 @@ fr_print(netdissect_options *ndo,
                 * A pad byte should only be used with 3-byte Q.922.
                 */
                if (addr_len != 3)
                 * A pad byte should only be used with 3-byte Q.922.
                 */
                if (addr_len != 3)
-                       ND_PRINT((ndo, "Pad! "));
+                       ND_PRINT("Pad! ");
                hdr_len = addr_len + 1 /* UI */ + 1 /* pad */ + 1 /* NLPID */;
        } else {
                /*
                hdr_len = addr_len + 1 /* UI */ + 1 /* pad */ + 1 /* NLPID */;
        } else {
                /*
@@ -303,7 +303,7 @@ fr_print(netdissect_options *ndo,
                 * A pad byte should be used with 3-byte Q.922.
                 */
                if (addr_len == 3)
                 * A pad byte should be used with 3-byte Q.922.
                 */
                if (addr_len == 3)
-                       ND_PRINT((ndo, "No pad! "));
+                       ND_PRINT("No pad! ");
                hdr_len = addr_len + 1 /* UI */ + 1 /* NLPID */;
        }
 
                hdr_len = addr_len + 1 /* UI */ + 1 /* NLPID */;
        }
 
@@ -365,27 +365,28 @@ fr_print(netdissect_options *ndo,
 
        return hdr_len;
 
 
        return hdr_len;
 
- trunc:
-        ND_PRINT((ndo, "[|fr]"));
+trunc:
+        nd_print_trunc(ndo);
         return 0;
 
 }
 
 u_int
 mfr_if_print(netdissect_options *ndo,
         return 0;
 
 }
 
 u_int
 mfr_if_print(netdissect_options *ndo,
-             const struct pcap_pkthdr *h, register const u_char *p)
+             const struct pcap_pkthdr *h, const u_char *p)
 {
 {
-       register u_int length = h->len;
-       register u_int caplen = h->caplen;
+       u_int length = h->len;
+       u_int caplen = h->caplen;
 
 
+       ndo->ndo_protocol = "mfr_if";
         ND_TCHECK_2(p); /* minimum frame header length */
 
         if ((length = mfr_print(ndo, p, length)) == 0)
             return (0);
         else
             return length;
         ND_TCHECK_2(p); /* minimum frame header length */
 
         if ((length = mfr_print(ndo, p, length)) == 0)
             return (0);
         else
             return length;
- trunc:
-        ND_PRINT((ndo, "[|mfr]"));
+trunc:
+        nd_print_trunc(ndo);
         return caplen;
 }
 
         return caplen;
 }
 
@@ -435,7 +436,7 @@ struct ie_tlv_header_t {
 
 u_int
 mfr_print(netdissect_options *ndo,
 
 u_int
 mfr_print(netdissect_options *ndo,
-          register const u_char *p, u_int length)
+          const u_char *p, u_int length)
 {
     u_int tlen,idx,hdr_len = 0;
     uint16_t sequence_num;
 {
     u_int tlen,idx,hdr_len = 0;
     uint16_t sequence_num;
@@ -456,13 +457,14 @@ mfr_print(netdissect_options *ndo,
  *    +----+----+----+----+----+----+----+----+
  */
 
  *    +----+----+----+----+----+----+----+----+
  */
 
+    ndo->ndo_protocol = "mfr";
     ND_TCHECK_4(p); /* minimum frame header length */
 
     if ((EXTRACT_U_1(p) & MFR_BEC_MASK) == MFR_CTRL_FRAME && EXTRACT_U_1(p + 1) == 0) {
     ND_TCHECK_4(p); /* minimum frame header length */
 
     if ((EXTRACT_U_1(p) & MFR_BEC_MASK) == MFR_CTRL_FRAME && EXTRACT_U_1(p + 1) == 0) {
-        ND_PRINT((ndo, "FRF.16 Control, Flags [%s], %s, length %u",
+        ND_PRINT("FRF.16 Control, Flags [%s], %s, length %u",
                bittok2str(frf_flag_values,"none",(EXTRACT_U_1(p) & MFR_BEC_MASK)),
                tok2str(mfr_ctrl_msg_values,"Unknown Message (0x%02x)",EXTRACT_U_1(p + 2)),
                bittok2str(frf_flag_values,"none",(EXTRACT_U_1(p) & MFR_BEC_MASK)),
                tok2str(mfr_ctrl_msg_values,"Unknown Message (0x%02x)",EXTRACT_U_1(p + 2)),
-               length));
+               length);
         tptr = p + 3;
         tlen = length -3;
         hdr_len = 3;
         tptr = p + 3;
         tlen = length -3;
         hdr_len = 3;
@@ -471,20 +473,20 @@ mfr_print(netdissect_options *ndo,
             return hdr_len;
 
         while (tlen>sizeof(struct ie_tlv_header_t)) {
             return hdr_len;
 
         while (tlen>sizeof(struct ie_tlv_header_t)) {
-            ND_TCHECK2(*tptr, sizeof(struct ie_tlv_header_t));
-            ie_type=tptr[0];
-            ie_len=tptr[1];
+            ND_TCHECK_LEN(tptr, sizeof(struct ie_tlv_header_t));
+            ie_type=EXTRACT_U_1(tptr);
+            ie_len=EXTRACT_U_1(tptr + 1);
 
 
-            ND_PRINT((ndo, "\n\tIE %s (%u), length %u: ",
+            ND_PRINT("\n\tIE %s (%u), length %u: ",
                    tok2str(mfr_ctrl_ie_values,"Unknown",ie_type),
                    ie_type,
                    tok2str(mfr_ctrl_ie_values,"Unknown",ie_type),
                    ie_type,
-                   ie_len));
+                   ie_len);
 
             /* infinite loop check */
             if (ie_type == 0 || ie_len <= sizeof(struct ie_tlv_header_t))
                 return hdr_len;
 
 
             /* infinite loop check */
             if (ie_type == 0 || ie_len <= sizeof(struct ie_tlv_header_t))
                 return hdr_len;
 
-            ND_TCHECK2(*tptr, ie_len);
+            ND_TCHECK_LEN(tptr, ie_len);
             tptr+=sizeof(struct ie_tlv_header_t);
             /* tlv len includes header */
             ie_len-=sizeof(struct ie_tlv_header_t);
             tptr+=sizeof(struct ie_tlv_header_t);
             /* tlv len includes header */
             ie_len-=sizeof(struct ie_tlv_header_t);
@@ -493,14 +495,14 @@ mfr_print(netdissect_options *ndo,
             switch (ie_type) {
 
             case MFR_CTRL_IE_MAGIC_NUM:
             switch (ie_type) {
 
             case MFR_CTRL_IE_MAGIC_NUM:
-                ND_PRINT((ndo, "0x%08x", EXTRACT_BE_U_4(tptr)));
+                ND_PRINT("0x%08x", EXTRACT_BE_U_4(tptr));
                 break;
 
             case MFR_CTRL_IE_BUNDLE_ID: /* same message format */
             case MFR_CTRL_IE_LINK_ID:
                 for (idx = 0; idx < ie_len && idx < MFR_ID_STRING_MAXLEN; idx++) {
                     if (EXTRACT_U_1(tptr + idx) != 0) /* don't print null termination */
                 break;
 
             case MFR_CTRL_IE_BUNDLE_ID: /* same message format */
             case MFR_CTRL_IE_LINK_ID:
                 for (idx = 0; idx < ie_len && idx < MFR_ID_STRING_MAXLEN; idx++) {
                     if (EXTRACT_U_1(tptr + idx) != 0) /* don't print null termination */
-                        safeputchar(ndo, EXTRACT_U_1(tptr + idx));
+                        fn_print_char(ndo, EXTRACT_U_1(tptr + idx));
                     else
                         break;
                 }
                     else
                         break;
                 }
@@ -542,11 +544,11 @@ mfr_print(netdissect_options *ndo,
  *
  *      7    6    5    4    3    2    1    0
  *    +----+----+----+----+----+----+----+----+
  *
  *      7    6    5    4    3    2    1    0
  *    +----+----+----+----+----+----+----+----+
- *    | B  | E  | C=0|seq. (high 4 bits) | EA  |
+ *    | B  | E  | C=0|seq. (high 4 bits) | EA |
  *    +----+----+----+----+----+----+----+----+
  *    |        sequence  (low 8 bits)         |
  *    +----+----+----+----+----+----+----+----+
  *    +----+----+----+----+----+----+----+----+
  *    |        sequence  (low 8 bits)         |
  *    +----+----+----+----+----+----+----+----+
- *    |        DLCI (6 bits)        | CR | EA  |
+ *    |        DLCI (6 bits)        | CR | EA |
  *    +----+----+----+----+----+----+----+----+
  *    |   DLCI (4 bits)   |FECN|BECN| DE | EA |
  *    +----+----+----+----+----+----+----+----+
  *    +----+----+----+----+----+----+----+----+
  *    |   DLCI (4 bits)   |FECN|BECN| DE | EA |
  *    +----+----+----+----+----+----+----+----+
@@ -556,24 +558,24 @@ mfr_print(netdissect_options *ndo,
     /* whole packet or first fragment ? */
     if ((EXTRACT_U_1(p) & MFR_BEC_MASK) == MFR_FRAG_FRAME ||
         (EXTRACT_U_1(p) & MFR_BEC_MASK) == MFR_B_BIT) {
     /* whole packet or first fragment ? */
     if ((EXTRACT_U_1(p) & MFR_BEC_MASK) == MFR_FRAG_FRAME ||
         (EXTRACT_U_1(p) & MFR_BEC_MASK) == MFR_B_BIT) {
-        ND_PRINT((ndo, "FRF.16 Frag, seq %u, Flags [%s], ",
+        ND_PRINT("FRF.16 Frag, seq %u, Flags [%s], ",
                sequence_num,
                sequence_num,
-               bittok2str(frf_flag_values,"none",(EXTRACT_U_1(p) & MFR_BEC_MASK))));
+               bittok2str(frf_flag_values,"none",(EXTRACT_U_1(p) & MFR_BEC_MASK)));
         hdr_len = 2;
         fr_print(ndo, p+hdr_len,length-hdr_len);
         return hdr_len;
     }
 
     /* must be a middle or the last fragment */
         hdr_len = 2;
         fr_print(ndo, p+hdr_len,length-hdr_len);
         return hdr_len;
     }
 
     /* must be a middle or the last fragment */
-    ND_PRINT((ndo, "FRF.16 Frag, seq %u, Flags [%s]",
+    ND_PRINT("FRF.16 Frag, seq %u, Flags [%s]",
            sequence_num,
            sequence_num,
-           bittok2str(frf_flag_values,"none",(EXTRACT_U_1(p) & MFR_BEC_MASK))));
+           bittok2str(frf_flag_values,"none",(EXTRACT_U_1(p) & MFR_BEC_MASK)));
     print_unknown_data(ndo, p, "\n\t", length);
 
     return hdr_len;
 
     print_unknown_data(ndo, p, "\n\t", length);
 
     return hdr_len;
 
- trunc:
-    ND_PRINT((ndo, "[|mfr]"));
+trunc:
+    nd_print_trunc(ndo);
     return length;
 }
 
     return length;
 }
 
@@ -607,11 +609,11 @@ frf15_print(netdissect_options *ndo,
     flags = EXTRACT_U_1(p)&MFR_BEC_MASK;
     sequence_num = (EXTRACT_U_1(p)&0x1e)<<7 | EXTRACT_U_1(p + 1);
 
     flags = EXTRACT_U_1(p)&MFR_BEC_MASK;
     sequence_num = (EXTRACT_U_1(p)&0x1e)<<7 | EXTRACT_U_1(p + 1);
 
-    ND_PRINT((ndo, "FRF.15, seq 0x%03x, Flags [%s],%s Fragmentation, length %u",
+    ND_PRINT("FRF.15, seq 0x%03x, Flags [%s],%s Fragmentation, length %u",
            sequence_num,
            bittok2str(frf_flag_values,"none",flags),
            EXTRACT_U_1(p)&FR_FRF15_FRAGTYPE ? "Interface" : "End-to-End",
            sequence_num,
            bittok2str(frf_flag_values,"none",flags),
            EXTRACT_U_1(p)&FR_FRF15_FRAGTYPE ? "Interface" : "End-to-End",
-           length));
+           length);
 
 /* TODO:
  * depending on all permutations of the B, E and C bit
 
 /* TODO:
  * depending on all permutations of the B, E and C bit
@@ -624,7 +626,7 @@ frf15_print(netdissect_options *ndo,
     return;
 
 trunc:
     return;
 
 trunc:
-    ND_PRINT((ndo, "[|frf.15]"));
+    nd_print_trunc(ndo);
 }
 
 /*
 }
 
 /*
@@ -800,12 +802,13 @@ q933_print(netdissect_options *ndo,
        u_int non_locking_shift;
        u_int unshift_codeset;
 
        u_int non_locking_shift;
        u_int unshift_codeset;
 
-       ND_PRINT((ndo, "%s", ndo->ndo_eflag ? "" : "Q.933"));
+       ndo->ndo_protocol = "q.933";
+       ND_PRINT("%s", ndo->ndo_eflag ? "" : "Q.933");
 
 
-       if (length == 0 || !ND_TTEST(*p)) {
+       if (length == 0 || !ND_TTEST_1(p)) {
                if (!ndo->ndo_eflag)
                if (!ndo->ndo_eflag)
-                       ND_PRINT((ndo, ", "));
-               ND_PRINT((ndo, "length %u", length));
+                       ND_PRINT(", ");
+               ND_PRINT("length %u", length);
                goto trunc;
        }
 
                goto trunc;
        }
 
@@ -821,10 +824,10 @@ q933_print(netdissect_options *ndo,
         * Get the call reference value.
         */
        for (i = 0; i < call_ref_length; i++) {
         * Get the call reference value.
         */
        for (i = 0; i < call_ref_length; i++) {
-               if (length == 0 || !ND_TTEST(*p)) {
+               if (length == 0 || !ND_TTEST_1(p)) {
                        if (!ndo->ndo_eflag)
                        if (!ndo->ndo_eflag)
-                               ND_PRINT((ndo, ", "));
-                       ND_PRINT((ndo, "length %u", olen));
+                               ND_PRINT(", ");
+                       ND_PRINT("length %u", olen);
                        goto trunc;
                }
                call_ref[i] = EXTRACT_U_1(p);
                        goto trunc;
                }
                call_ref[i] = EXTRACT_U_1(p);
@@ -835,10 +838,10 @@ q933_print(netdissect_options *ndo,
        /*
         * Get the message type.
         */
        /*
         * Get the message type.
         */
-       if (length == 0 || !ND_TTEST(*p)) {
+       if (length == 0 || !ND_TTEST_1(p)) {
                if (!ndo->ndo_eflag)
                if (!ndo->ndo_eflag)
-                       ND_PRINT((ndo, ", "));
-               ND_PRINT((ndo, "length %u", olen));
+                       ND_PRINT(", ");
+               ND_PRINT("length %u", olen);
                goto trunc;
        }
        msgtype = EXTRACT_U_1(p);
                goto trunc;
        }
        msgtype = EXTRACT_U_1(p);
@@ -851,10 +854,10 @@ q933_print(netdissect_options *ndo,
        non_locking_shift = 0;
        unshift_codeset = codeset;
        if (length != 0) {
        non_locking_shift = 0;
        unshift_codeset = codeset;
        if (length != 0) {
-               if (!ND_TTEST(*p)) {
+               if (!ND_TTEST_1(p)) {
                        if (!ndo->ndo_eflag)
                        if (!ndo->ndo_eflag)
-                               ND_PRINT((ndo, ", "));
-                       ND_PRINT((ndo, "length %u", olen));
+                               ND_PRINT(", ");
+                       ND_PRINT("length %u", olen);
                        goto trunc;
                }
                iecode = EXTRACT_U_1(p);
                        goto trunc;
                }
                iecode = EXTRACT_U_1(p);
@@ -903,8 +906,8 @@ q933_print(netdissect_options *ndo,
 
        /* printing out header part */
        if (!ndo->ndo_eflag)
 
        /* printing out header part */
        if (!ndo->ndo_eflag)
-               ND_PRINT((ndo, ", "));
-       ND_PRINT((ndo, "%s, codeset %u", is_ansi ? "ANSI" : "CCITT", codeset));
+               ND_PRINT(", ");
+       ND_PRINT("%s, codeset %u", is_ansi ? "ANSI" : "CCITT", codeset);
 
        if (call_ref_length != 0) {
                ND_TCHECK_1(p);
 
        if (call_ref_length != 0) {
                ND_TCHECK_1(p);
@@ -912,21 +915,21 @@ q933_print(netdissect_options *ndo,
                        /*
                         * Not a dummy call reference.
                         */
                        /*
                         * Not a dummy call reference.
                         */
-                       ND_PRINT((ndo, ", Call Ref: 0x"));
+                       ND_PRINT(", Call Ref: 0x");
                        for (i = 0; i < call_ref_length; i++)
                        for (i = 0; i < call_ref_length; i++)
-                               ND_PRINT((ndo, "%02x", call_ref[i]));
+                               ND_PRINT("%02x", call_ref[i]);
                }
        }
        if (ndo->ndo_vflag) {
                }
        }
        if (ndo->ndo_vflag) {
-               ND_PRINT((ndo, ", %s (0x%02x), length %u",
+               ND_PRINT(", %s (0x%02x), length %u",
                   tok2str(fr_q933_msg_values,
                        "unknown message", msgtype),
                   msgtype,
                   tok2str(fr_q933_msg_values,
                        "unknown message", msgtype),
                   msgtype,
-                  olen));
+                  olen);
        } else {
        } else {
-               ND_PRINT((ndo, ", %s",
+               ND_PRINT(", %s",
                       tok2str(fr_q933_msg_values,
                       tok2str(fr_q933_msg_values,
-                              "unknown message 0x%02x", msgtype)));
+                              "unknown message 0x%02x", msgtype));
        }
 
        /* Loop through the rest of the IEs */
        }
 
        /* Loop through the rest of the IEs */
@@ -952,9 +955,9 @@ q933_print(netdissect_options *ndo,
                /*
                 * Get the first octet of the IE.
                 */
                /*
                 * Get the first octet of the IE.
                 */
-               if (!ND_TTEST(*p)) {
+               if (!ND_TTEST_1(p)) {
                        if (!ndo->ndo_vflag) {
                        if (!ndo->ndo_vflag) {
-                               ND_PRINT((ndo, ", length %u", olen));
+                               ND_PRINT(", length %u", olen);
                        }
                        goto trunc;
                }
                        }
                        goto trunc;
                }
@@ -995,9 +998,9 @@ q933_print(netdissect_options *ndo,
                        /*
                         * No.  Get the IE length.
                         */
                        /*
                         * No.  Get the IE length.
                         */
-                       if (length == 0 || !ND_TTEST(*p)) {
+                       if (length == 0 || !ND_TTEST_1(p)) {
                                if (!ndo->ndo_vflag) {
                                if (!ndo->ndo_vflag) {
-                                       ND_PRINT((ndo, ", length %u", olen));
+                                       ND_PRINT(", length %u", olen);
                                }
                                goto trunc;
                        }
                                }
                                goto trunc;
                        }
@@ -1009,20 +1012,20 @@ q933_print(netdissect_options *ndo,
                         * however some IEs (DLCI Status, Link Verify)
                         * are also interesting in non-verbose mode */
                        if (ndo->ndo_vflag) {
                         * however some IEs (DLCI Status, Link Verify)
                         * are also interesting in non-verbose mode */
                        if (ndo->ndo_vflag) {
-                               ND_PRINT((ndo, "\n\t%s IE (0x%02x), length %u: ",
+                               ND_PRINT("\n\t%s IE (0x%02x), length %u: ",
                                    tok2str(fr_q933_ie_codesets[codeset],
                                        "unknown", iecode),
                                    iecode,
                                    tok2str(fr_q933_ie_codesets[codeset],
                                        "unknown", iecode),
                                    iecode,
-                                   ielength));
+                                   ielength);
                        }
 
                        /* sanity checks */
                        if (iecode == 0 || ielength == 0) {
                                return;
                        }
                        }
 
                        /* sanity checks */
                        if (iecode == 0 || ielength == 0) {
                                return;
                        }
-                       if (length < ielength || !ND_TTEST2(*p, ielength)) {
+                       if (length < ielength || !ND_TTEST_LEN(p, ielength)) {
                                if (!ndo->ndo_vflag) {
                                if (!ndo->ndo_vflag) {
-                                       ND_PRINT((ndo, ", length %u", olen));
+                                       ND_PRINT(", length %u", olen);
                                }
                                goto trunc;
                        }
                                }
                                goto trunc;
                        }
@@ -1056,12 +1059,12 @@ q933_print(netdissect_options *ndo,
                }
        }
        if (!ndo->ndo_vflag) {
                }
        }
        if (!ndo->ndo_vflag) {
-           ND_PRINT((ndo, ", length %u", olen));
+           ND_PRINT(", length %u", olen);
        }
        return;
 
 trunc:
        }
        return;
 
 trunc:
-       ND_PRINT((ndo, "[|q.933]"));
+       nd_print_trunc(ndo);
 }
 
 static int
 }
 
 static int
@@ -1076,15 +1079,15 @@ fr_q933_print_ie_codeset_0_5(netdissect_options *ndo, u_int iecode,
         case FR_LMI_CCITT_REPORT_TYPE_IE:
             if (ielength < 1) {
                 if (!ndo->ndo_vflag) {
         case FR_LMI_CCITT_REPORT_TYPE_IE:
             if (ielength < 1) {
                 if (!ndo->ndo_vflag) {
-                    ND_PRINT((ndo, ", "));
+                    ND_PRINT(", ");
                }
                }
-                ND_PRINT((ndo, "Invalid REPORT TYPE IE"));
+                ND_PRINT("Invalid REPORT TYPE IE");
                 return 1;
             }
             if (ndo->ndo_vflag) {
                 return 1;
             }
             if (ndo->ndo_vflag) {
-                ND_PRINT((ndo, "%s (%u)",
+                ND_PRINT("%s (%u)",
                        tok2str(fr_lmi_report_type_ie_values,"unknown",EXTRACT_U_1(p)),
                        tok2str(fr_lmi_report_type_ie_values,"unknown",EXTRACT_U_1(p)),
-                       EXTRACT_U_1(p)));
+                       EXTRACT_U_1(p));
            }
             return 1;
 
            }
             return 1;
 
@@ -1092,19 +1095,19 @@ fr_q933_print_ie_codeset_0_5(netdissect_options *ndo, u_int iecode,
         case FR_LMI_CCITT_LINK_VERIFY_IE:
         case FR_LMI_ANSI_LINK_VERIFY_IE_91:
             if (!ndo->ndo_vflag) {
         case FR_LMI_CCITT_LINK_VERIFY_IE:
         case FR_LMI_ANSI_LINK_VERIFY_IE_91:
             if (!ndo->ndo_vflag) {
-                ND_PRINT((ndo, ", "));
+                ND_PRINT(", ");
            }
             if (ielength < 2) {
            }
             if (ielength < 2) {
-                ND_PRINT((ndo, "Invalid LINK VERIFY IE"));
+                ND_PRINT("Invalid LINK VERIFY IE");
                 return 1;
             }
                 return 1;
             }
-            ND_PRINT((ndo, "TX Seq: %3d, RX Seq: %3d", EXTRACT_U_1(p), EXTRACT_U_1(p + 1)));
+            ND_PRINT("TX Seq: %3d, RX Seq: %3d", EXTRACT_U_1(p), EXTRACT_U_1(p + 1));
             return 1;
 
         case FR_LMI_ANSI_PVC_STATUS_IE: /* fall through */
         case FR_LMI_CCITT_PVC_STATUS_IE:
             if (!ndo->ndo_vflag) {
             return 1;
 
         case FR_LMI_ANSI_PVC_STATUS_IE: /* fall through */
         case FR_LMI_CCITT_PVC_STATUS_IE:
             if (!ndo->ndo_vflag) {
-                ND_PRINT((ndo, ", "));
+                ND_PRINT(", ");
            }
             /* now parse the DLCI information element. */
             if ((ielength < 3) ||
            }
             /* now parse the DLCI information element. */
             if ((ielength < 3) ||
@@ -1117,7 +1120,7 @@ fr_q933_print_ie_codeset_0_5(netdissect_options *ndo, u_int iecode,
                    !(EXTRACT_U_1(p + 3) & 0x80))) ||
                 (ielength > 5) ||
                 !(EXTRACT_U_1(p + ielength - 1) & 0x80)) {
                    !(EXTRACT_U_1(p + 3) & 0x80))) ||
                 (ielength > 5) ||
                 !(EXTRACT_U_1(p + ielength - 1) & 0x80)) {
-                ND_PRINT((ndo, "Invalid DLCI in PVC STATUS IE"));
+                ND_PRINT("Invalid DLCI in PVC STATUS IE");
                 return 1;
            }
 
                 return 1;
            }
 
@@ -1129,17 +1132,11 @@ fr_q933_print_ie_codeset_0_5(netdissect_options *ndo, u_int iecode,
                 dlci = (dlci << 13) | (EXTRACT_U_1(p + 2) & 0x7F) | ((EXTRACT_U_1(p + 3) & 0x7E) >> 1);
            }
 
                 dlci = (dlci << 13) | (EXTRACT_U_1(p + 2) & 0x7F) | ((EXTRACT_U_1(p + 3) & 0x7E) >> 1);
            }
 
-            ND_PRINT((ndo, "DLCI %u: status %s%s", dlci,
+            ND_PRINT("DLCI %u: status %s%s", dlci,
                     EXTRACT_U_1(p + ielength - 1) & 0x8 ? "New, " : "",
                     EXTRACT_U_1(p + ielength - 1) & 0x8 ? "New, " : "",
-                    EXTRACT_U_1(p + ielength - 1) & 0x2 ? "Active" : "Inactive"));
+                    EXTRACT_U_1(p + ielength - 1) & 0x2 ? "Active" : "Inactive");
             return 1;
        }
 
         return 0;
 }
             return 1;
        }
 
         return 0;
 }
-/*
- * Local Variables:
- * c-style: whitesmith
- * c-basic-offset: 8
- * End:
- */
[mirror] the TCPdump network dissector