]> The Tcpdump Group git mirrors - tcpdump/blobdiff - print-decnet.c
projects / tcpdump / blobdiff
summary | shortlog | log | commit | commitdiff | tree
raw | inline | side by side
arista: update test pcap with hwinfo values
[tcpdump] / print-decnet.c
diff --git a/print-decnet.c b/print-decnet.c
index 9746d962c6f56912e0f59d77b44ea855bf965768..6a6dd25982f9376d2b1bcdf3f1c34b1a9aa2156f 100644 (file)
--- a/print-decnet.c
+++ b/print-decnet.c
@@ -19,53 +19,45 @@
  * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 
  * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 
+/* \summary: DECnet printer */
+
 #ifdef HAVE_CONFIG_H
 #ifdef HAVE_CONFIG_H
-#include "config.h"
+#include <config.h>
 #endif
 
 #endif
 
-#include <tcpdump-stdinc.h>
-
-struct mbuf;
-struct rtentry;
-
-#ifdef HAVE_NETDNET_DNETDB_H
-#include <netdnet/dnetdb.h>
-#endif
+#include "netdissect-stdinc.h"
 
 #include <stdio.h>
 #include <stdlib.h>
 
 #include <stdio.h>
 #include <stdlib.h>
-#include <string.h>
 
 
+#define ND_LONGJMP_FROM_TCHECK
+#include "netdissect.h"
 #include "extract.h"
 #include "extract.h"
-#include "interface.h"
 #include "addrtoname.h"
 
 #include "addrtoname.h"
 
-static const char tstr[] = "[|decnet]";
 
 
-#ifndef WIN32
-typedef u_int8_t byte[1];              /* single byte field */
+#ifndef _WIN32
+typedef nd_uint8_t byte;               /* single byte field */
 #else
 /*
  * the keyword 'byte' generates conflicts in Windows
  */
 #else
 /*
  * the keyword 'byte' generates conflicts in Windows
  */
-typedef unsigned char Byte[1];         /* single byte field */
+typedef nd_uint8_t Byte;               /* single byte field */
 #define byte Byte
 #define byte Byte
-#endif /* WIN32 */
-typedef u_int8_t word[2];              /* 2 byte field */
-typedef u_int8_t longword[4];          /* 4 bytes field */
+#endif /* _WIN32 */
+typedef nd_uint16_t word;              /* 2 byte field */
+typedef nd_uint32_t longword;          /* 4 bytes field */
 
 /*
  * Definitions for DECNET Phase IV protocol headers
  */
 
 /*
  * Definitions for DECNET Phase IV protocol headers
  */
-union etheraddress {
-       u_int8_t   dne_addr[6];         /* full ethernet address */
+typedef union {
+       nd_mac_addr dne_addr;   /* full Ethernet address */
        struct {
        struct {
-               u_int8_t dne_hiord[4];  /* DECnet HIORD prefix */
-               u_int8_t dne_nodeaddr[2]; /* DECnet node address */
+               nd_byte dne_hiord[4];   /* DECnet HIORD prefix */
+               nd_byte dne_nodeaddr[2]; /* DECnet node address */
        } dne_remote;
        } dne_remote;
-};
-
-typedef union etheraddress etheraddr;  /* Ethernet address */
+} etheraddr;   /* Ethernet address */
 
 #define HIORD 0x000400aa               /* high 32-bits of address (swapped) */
 
 
 #define HIORD 0x000400aa               /* high 32-bits of address (swapped) */
 
@@ -73,12 +65,6 @@ typedef union etheraddress etheraddr;        /* Ethernet address */
 #define        AREASHIFT       10              /* bit-offset for area field */
 #define NODEMASK       01777           /* mask for node address field */
 
 #define        AREASHIFT       10              /* bit-offset for area field */
 #define NODEMASK       01777           /* mask for node address field */
 
-#define DN_MAXADDL     20              /* max size of DECnet address */
-struct dn_naddr {
-       u_int16_t       a_len;          /* length of address */
-       u_int8_t a_addr[DN_MAXADDL]; /* address as bytes */
-};
-
 /*
  * Define long and short header formats.
  */
 /*
  * Define long and short header formats.
  */
@@ -171,18 +157,6 @@ union routehdr
 /*
  * Define control message formats.
  */
 /*
  * Define control message formats.
  */
-struct initmsgIII                      /* phase III initialization message */
-  {
-    byte       inIII_flags;            /* route flags */
-    word       inIII_src;              /* source node address */
-    byte       inIII_info;             /* routing layer information */
-    word       inIII_blksize;          /* maximum data link block size */
-    byte       inIII_vers;             /* version number */
-    byte       inIII_eco;              /* ECO number */
-    byte       inIII_ueco;             /* user ECO number */
-    byte       inIII_rsvd;             /* reserved image field */
-  };
-
 struct initmsg                         /* initialization message */
   {
     byte       in_flags;               /* route flags */
 struct initmsg                         /* initialization message */
   {
     byte       in_flags;               /* route flags */
@@ -270,7 +244,7 @@ union controlmsg
 /* Macros for decoding routing-info fields */
 #define        RI_COST(x)      ((x)&0777)
 #define        RI_HOPS(x)      (((x)>>10)&037)
 /* Macros for decoding routing-info fields */
 #define        RI_COST(x)      ((x)&0777)
 #define        RI_HOPS(x)      (((x)>>10)&037)
-\f
+
 /*
  * NSP protocol fields and values.
  */
 /*
  * NSP protocol fields and values.
  */
@@ -324,7 +298,6 @@ union controlmsg
 #define COS_NONE 0                     /* no flow control */
 #define COS_SEGMENT 04                 /* segment flow control */
 #define COS_MESSAGE 010                        /* message flow control */
 #define COS_NONE 0                     /* no flow control */
 #define COS_SEGMENT 04                 /* segment flow control */
 #define COS_MESSAGE 010                        /* message flow control */
-#define COS_CRYPTSER 020               /* cryptographic services requested */
 #define COS_DEFAULT 1                  /* default value for field */
 
 #define COI_MASK 3                     /* mask for version field */
 #define COS_DEFAULT 1                  /* default value for field */
 
 #define COI_MASK 3                     /* mask for version field */
@@ -482,380 +455,356 @@ struct dcmsg                            /* disconnect confirm message */
   };
 
 /* Forwards */
   };
 
 /* Forwards */
-static int print_decnet_ctlmsg(const union routehdr *, u_int, u_int);
-static void print_t_info(int);
-static int print_l1_routes(const char *, u_int);
-static int print_l2_routes(const char *, u_int);
-static void print_i_info(int);
-static int print_elist(const char *, u_int);
-static int print_nsp(const u_char *, u_int);
-static void print_reason(int);
-#ifdef PRINT_NSPDATA
-static void pdata(u_char *, int);
-#endif
-
-#ifndef HAVE_NETDNET_DNETDB_H_DNET_HTOA
-extern char *dnet_htoa(struct dn_naddr *);
-#endif
+static int print_decnet_ctlmsg(netdissect_options *, const union routehdr *, u_int, u_int);
+static void print_t_info(netdissect_options *, u_int);
+static void print_l1_routes(netdissect_options *, const u_char *, u_int);
+static void print_l2_routes(netdissect_options *, const u_char *, u_int);
+static void print_i_info(netdissect_options *, u_int);
+static void print_elist(const u_char *, u_int);
+static int print_nsp(netdissect_options *, const u_char *, u_int);
+static void print_reason(netdissect_options *, u_int);
 
 void
 
 void
-decnet_print(register const u_char *ap, register u_int length,
-            register u_int caplen)
+decnet_print(netdissect_options *ndo,
+             const u_char *ap, u_int length,
+             u_int caplen)
 {
 {
-       register const union routehdr *rhp;
-       register int mflags;
-       int dst, src, hops;
+       const union routehdr *rhp;
+       u_int mflags;
+       uint16_t dst, src;
+       u_int hops;
        u_int nsplen, pktlen;
        const u_char *nspp;
 
        u_int nsplen, pktlen;
        const u_char *nspp;
 
+       ndo->ndo_protocol = "decnet";
        if (length < sizeof(struct shorthdr)) {
        if (length < sizeof(struct shorthdr)) {
-               (void)printf("%s", tstr);
-               return;
+               ND_PRINT(" (length %u < %zu)", length, sizeof(struct shorthdr));
+               goto invalid;
        }
 
        }
 
-       TCHECK2(*ap, sizeof(short));
-       pktlen = EXTRACT_LE_16BITS(ap);
+       pktlen = GET_LE_U_2(ap);
        if (pktlen < sizeof(struct shorthdr)) {
        if (pktlen < sizeof(struct shorthdr)) {
-               (void)printf("%s", tstr);
-               return;
+               ND_PRINT(" (pktlen %u < %zu)", pktlen, sizeof(struct shorthdr));
+               goto invalid;
        }
        if (pktlen > length) {
        }
        if (pktlen > length) {
-               (void)printf("%s", tstr);
-               return;
+               ND_PRINT(" (pktlen %u > %u)", pktlen, length);
+               goto invalid;
        }
        length = pktlen;
 
        }
        length = pktlen;
 
-       rhp = (const union routehdr *)&(ap[sizeof(short)]);
-       TCHECK(rhp->rh_short.sh_flags);
-       mflags = EXTRACT_LE_8BITS(rhp->rh_short.sh_flags);
+       rhp = (const union routehdr *)(ap + sizeof(short));
+       mflags = GET_U_1(rhp->rh_short.sh_flags);
 
        if (mflags & RMF_PAD) {
            /* pad bytes of some sort in front of message */
            u_int padlen = mflags & RMF_PADMASK;
 
        if (mflags & RMF_PAD) {
            /* pad bytes of some sort in front of message */
            u_int padlen = mflags & RMF_PADMASK;
-           if (vflag)
-               (void) printf("[pad:%d] ", padlen);
+           if (ndo->ndo_vflag)
+               ND_PRINT("[pad:%u] ", padlen);
            if (length < padlen + 2) {
            if (length < padlen + 2) {
-               (void)printf("%s", tstr);
-               return;
+               ND_PRINT(" (length %u < %u)", length, padlen + 2);
+               goto invalid;
            }
            }
-           TCHECK2(ap[sizeof(short)], padlen);
+           ND_TCHECK_LEN(ap + sizeof(short), padlen);
            ap += padlen;
            length -= padlen;
            caplen -= padlen;
            ap += padlen;
            length -= padlen;
            caplen -= padlen;
-           rhp = (const union routehdr *)&(ap[sizeof(short)]);
-           mflags = EXTRACT_LE_8BITS(rhp->rh_short.sh_flags);
+           rhp = (const union routehdr *)(ap + sizeof(short));
+           mflags = GET_U_1(rhp->rh_short.sh_flags);
        }
 
        if (mflags & RMF_FVER) {
        }
 
        if (mflags & RMF_FVER) {
-               (void) printf("future-version-decnet");
-               default_print(ap, min(length, caplen));
+               ND_PRINT("future-version-decnet");
+               ND_DEFAULTPRINT(ap, ND_MIN(length, caplen));
                return;
        }
 
        /* is it a control message? */
        if (mflags & RMF_CTLMSG) {
                return;
        }
 
        /* is it a control message? */
        if (mflags & RMF_CTLMSG) {
-               if (!print_decnet_ctlmsg(rhp, length, caplen))
-                       goto trunc;
+               if (!print_decnet_ctlmsg(ndo, rhp, length, caplen))
+                       goto invalid;
                return;
        }
 
        switch (mflags & RMF_MASK) {
        case RMF_LONG:
            if (length < sizeof(struct longhdr)) {
                return;
        }
 
        switch (mflags & RMF_MASK) {
        case RMF_LONG:
            if (length < sizeof(struct longhdr)) {
-               (void)printf("%s", tstr);
-               return;
+               ND_PRINT(" (length %u < %zu)", length, sizeof(struct longhdr));
+               goto invalid;
            }
            }
-           TCHECK(rhp->rh_long);
+           ND_TCHECK_SIZE(&rhp->rh_long);
            dst =
            dst =
-               EXTRACT_LE_16BITS(rhp->rh_long.lg_dst.dne_remote.dne_nodeaddr);
+               GET_LE_U_2(rhp->rh_long.lg_dst.dne_remote.dne_nodeaddr);
            src =
            src =
-               EXTRACT_LE_16BITS(rhp->rh_long.lg_src.dne_remote.dne_nodeaddr);
-           hops = EXTRACT_LE_8BITS(rhp->rh_long.lg_visits);
-           nspp = &(ap[sizeof(short) + sizeof(struct longhdr)]);
+               GET_LE_U_2(rhp->rh_long.lg_src.dne_remote.dne_nodeaddr);
+           hops = GET_U_1(rhp->rh_long.lg_visits);
+           nspp = ap + sizeof(short) + sizeof(struct longhdr);
            nsplen = length - sizeof(struct longhdr);
            break;
        case RMF_SHORT:
            nsplen = length - sizeof(struct longhdr);
            break;
        case RMF_SHORT:
-           TCHECK(rhp->rh_short);
-           dst = EXTRACT_LE_16BITS(rhp->rh_short.sh_dst);
-           src = EXTRACT_LE_16BITS(rhp->rh_short.sh_src);
-           hops = (EXTRACT_LE_8BITS(rhp->rh_short.sh_visits) & VIS_MASK)+1;
-           nspp = &(ap[sizeof(short) + sizeof(struct shorthdr)]);
+           dst = GET_LE_U_2(rhp->rh_short.sh_dst);
+           src = GET_LE_U_2(rhp->rh_short.sh_src);
+           hops = (GET_U_1(rhp->rh_short.sh_visits) & VIS_MASK)+1;
+           nspp = ap + sizeof(short) + sizeof(struct shorthdr);
            nsplen = length - sizeof(struct shorthdr);
            break;
        default:
            nsplen = length - sizeof(struct shorthdr);
            break;
        default:
-           (void) printf("unknown message flags under mask");
-           default_print((u_char *)ap, min(length, caplen));
+           ND_PRINT("unknown message flags under mask");
+           ND_DEFAULTPRINT((const u_char *)ap, ND_MIN(length, caplen));
            return;
        }
 
            return;
        }
 
-       (void)printf("%s > %s %d ",
-                       dnaddr_string(src), dnaddr_string(dst), pktlen);
-       if (vflag) {
+       ND_PRINT("%s > %s %u ",
+                       dnaddr_string(ndo, src), dnaddr_string(ndo, dst), pktlen);
+       if (ndo->ndo_vflag) {
            if (mflags & RMF_RQR)
            if (mflags & RMF_RQR)
-               (void)printf("RQR ");
+               ND_PRINT("RQR ");
            if (mflags & RMF_RTS)
            if (mflags & RMF_RTS)
-               (void)printf("RTS ");
+               ND_PRINT("RTS ");
            if (mflags & RMF_IE)
            if (mflags & RMF_IE)
-               (void)printf("IE ");
-           (void)printf("%d hops ", hops);
+               ND_PRINT("IE ");
+           ND_PRINT("%u hops ", hops);
        }
 
        }
 
-       if (!print_nsp(nspp, nsplen))
-               goto trunc;
+       if (!print_nsp(ndo, nspp, nsplen))
+               goto invalid;
        return;
 
        return;
 
-trunc:
-       (void)printf("%s", tstr);
-       return;
+invalid:
+       nd_print_invalid(ndo);
 }
 
 static int
 }
 
 static int
-print_decnet_ctlmsg(register const union routehdr *rhp, u_int length,
-    u_int caplen)
+print_decnet_ctlmsg(netdissect_options *ndo,
+                    const union routehdr *rhp, u_int length,
+                    u_int caplen)
 {
 {
-       int mflags = EXTRACT_LE_8BITS(rhp->rh_short.sh_flags);
-       register union controlmsg *cmp = (union controlmsg *)rhp;
-       int src, dst, info, blksize, eco, ueco, hello, other, vers;
-       etheraddr srcea, rtea;
-       int priority;
-       char *rhpx = (char *)rhp;
-       int ret;
+       /* Our caller has already checked for mflags */
+       u_int mflags = GET_U_1(rhp->rh_short.sh_flags);
+       const union controlmsg *cmp = (const union controlmsg *)rhp;
+       uint16_t src, dst;
+       u_int info, blksize, eco, ueco, hello, other, vers;
+       u_int priority;
+       const u_char *rhpx = (const u_char *)rhp;
 
        switch (mflags & RMF_CTLMASK) {
        case RMF_INIT:
 
        switch (mflags & RMF_CTLMASK) {
        case RMF_INIT:
-           (void)printf("init ");
+           ND_PRINT("init ");
            if (length < sizeof(struct initmsg))
            if (length < sizeof(struct initmsg))
-               goto trunc;
-           TCHECK(cmp->cm_init);
-           src = EXTRACT_LE_16BITS(cmp->cm_init.in_src);
-           info = EXTRACT_LE_8BITS(cmp->cm_init.in_info);
-           blksize = EXTRACT_LE_16BITS(cmp->cm_init.in_blksize);
-           vers = EXTRACT_LE_8BITS(cmp->cm_init.in_vers);
-           eco = EXTRACT_LE_8BITS(cmp->cm_init.in_eco);
-           ueco = EXTRACT_LE_8BITS(cmp->cm_init.in_ueco);
-           hello = EXTRACT_LE_16BITS(cmp->cm_init.in_hello);
-           print_t_info(info);
-           (void)printf(
-               "src %sblksize %d vers %d eco %d ueco %d hello %d",
-                       dnaddr_string(src), blksize, vers, eco, ueco,
+               goto invalid;
+           ND_TCHECK_SIZE(&cmp->cm_init);
+           src = GET_LE_U_2(cmp->cm_init.in_src);
+           info = GET_U_1(cmp->cm_init.in_info);
+           blksize = GET_LE_U_2(cmp->cm_init.in_blksize);
+           vers = GET_U_1(cmp->cm_init.in_vers);
+           eco = GET_U_1(cmp->cm_init.in_eco);
+           ueco = GET_U_1(cmp->cm_init.in_ueco);
+           hello = GET_LE_U_2(cmp->cm_init.in_hello);
+           print_t_info(ndo, info);
+           ND_PRINT("src %sblksize %u vers %u eco %u ueco %u hello %u",
+                       dnaddr_string(ndo, src), blksize, vers, eco, ueco,
                        hello);
                        hello);
-           ret = 1;
            break;
        case RMF_VER:
            break;
        case RMF_VER:
-           (void)printf("verification ");
+           ND_PRINT("verification ");
            if (length < sizeof(struct verifmsg))
            if (length < sizeof(struct verifmsg))
-               goto trunc;
-           TCHECK(cmp->cm_ver);
-           src = EXTRACT_LE_16BITS(cmp->cm_ver.ve_src);
-           other = EXTRACT_LE_8BITS(cmp->cm_ver.ve_fcnval);
-           (void)printf("src %s fcnval %o", dnaddr_string(src), other);
-           ret = 1;
+               goto invalid;
+           src = GET_LE_U_2(cmp->cm_ver.ve_src);
+           other = GET_U_1(cmp->cm_ver.ve_fcnval);
+           ND_PRINT("src %s fcnval %o", dnaddr_string(ndo, src), other);
            break;
        case RMF_TEST:
            break;
        case RMF_TEST:
-           (void)printf("test ");
+           ND_PRINT("test ");
            if (length < sizeof(struct testmsg))
            if (length < sizeof(struct testmsg))
-               goto trunc;
-           TCHECK(cmp->cm_test);
-           src = EXTRACT_LE_16BITS(cmp->cm_test.te_src);
-           other = EXTRACT_LE_8BITS(cmp->cm_test.te_data);
-           (void)printf("src %s data %o", dnaddr_string(src), other);
-           ret = 1;
+               goto invalid;
+           src = GET_LE_U_2(cmp->cm_test.te_src);
+           other = GET_U_1(cmp->cm_test.te_data);
+           ND_PRINT("src %s data %o", dnaddr_string(ndo, src), other);
            break;
        case RMF_L1ROUT:
            break;
        case RMF_L1ROUT:
-           (void)printf("lev-1-routing ");
+           ND_PRINT("lev-1-routing ");
            if (length < sizeof(struct l1rout))
            if (length < sizeof(struct l1rout))
-               goto trunc;
-           TCHECK(cmp->cm_l1rou);
-           src = EXTRACT_LE_16BITS(cmp->cm_l1rou.r1_src);
-           (void)printf("src %s ", dnaddr_string(src));
-           ret = print_l1_routes(&(rhpx[sizeof(struct l1rout)]),
+               goto invalid;
+           ND_TCHECK_SIZE(&cmp->cm_l1rou);
+           src = GET_LE_U_2(cmp->cm_l1rou.r1_src);
+           ND_PRINT("src %s ", dnaddr_string(ndo, src));
+           print_l1_routes(ndo, &(rhpx[sizeof(struct l1rout)]),
                                length - sizeof(struct l1rout));
            break;
        case RMF_L2ROUT:
                                length - sizeof(struct l1rout));
            break;
        case RMF_L2ROUT:
-           (void)printf("lev-2-routing ");
+           ND_PRINT("lev-2-routing ");
            if (length < sizeof(struct l2rout))
            if (length < sizeof(struct l2rout))
-               goto trunc;
-           TCHECK(cmp->cm_l2rout);
-           src = EXTRACT_LE_16BITS(cmp->cm_l2rout.r2_src);
-           (void)printf("src %s ", dnaddr_string(src));
-           ret = print_l2_routes(&(rhpx[sizeof(struct l2rout)]),
+               goto invalid;
+           ND_TCHECK_SIZE(&cmp->cm_l2rout);
+           src = GET_LE_U_2(cmp->cm_l2rout.r2_src);
+           ND_PRINT("src %s ", dnaddr_string(ndo, src));
+           print_l2_routes(ndo, &(rhpx[sizeof(struct l2rout)]),
                                length - sizeof(struct l2rout));
            break;
        case RMF_RHELLO:
                                length - sizeof(struct l2rout));
            break;
        case RMF_RHELLO:
-           (void)printf("router-hello ");
+           ND_PRINT("router-hello ");
            if (length < sizeof(struct rhellomsg))
            if (length < sizeof(struct rhellomsg))
-               goto trunc;
-           TCHECK(cmp->cm_rhello);
-           vers = EXTRACT_LE_8BITS(cmp->cm_rhello.rh_vers);
-           eco = EXTRACT_LE_8BITS(cmp->cm_rhello.rh_eco);
-           ueco = EXTRACT_LE_8BITS(cmp->cm_rhello.rh_ueco);
-           memcpy((char *)&srcea, (char *)&(cmp->cm_rhello.rh_src),
-               sizeof(srcea));
-           src = EXTRACT_LE_16BITS(srcea.dne_remote.dne_nodeaddr);
-           info = EXTRACT_LE_8BITS(cmp->cm_rhello.rh_info);
-           blksize = EXTRACT_LE_16BITS(cmp->cm_rhello.rh_blksize);
-           priority = EXTRACT_LE_8BITS(cmp->cm_rhello.rh_priority);
-           hello = EXTRACT_LE_16BITS(cmp->cm_rhello.rh_hello);
-           print_i_info(info);
-           (void)printf(
-           "vers %d eco %d ueco %d src %s blksize %d pri %d hello %d",
-                       vers, eco, ueco, dnaddr_string(src),
+               goto invalid;
+           ND_TCHECK_SIZE(&cmp->cm_rhello);
+           vers = GET_U_1(cmp->cm_rhello.rh_vers);
+           eco = GET_U_1(cmp->cm_rhello.rh_eco);
+           ueco = GET_U_1(cmp->cm_rhello.rh_ueco);
+           src =
+               GET_LE_U_2(cmp->cm_rhello.rh_src.dne_remote.dne_nodeaddr);
+           info = GET_U_1(cmp->cm_rhello.rh_info);
+           blksize = GET_LE_U_2(cmp->cm_rhello.rh_blksize);
+           priority = GET_U_1(cmp->cm_rhello.rh_priority);
+           hello = GET_LE_U_2(cmp->cm_rhello.rh_hello);
+           print_i_info(ndo, info);
+           ND_PRINT("vers %u eco %u ueco %u src %s blksize %u pri %u hello %u",
+                       vers, eco, ueco, dnaddr_string(ndo, src),
                        blksize, priority, hello);
                        blksize, priority, hello);
-           ret = print_elist(&(rhpx[sizeof(struct rhellomsg)]),
+           print_elist(&(rhpx[sizeof(struct rhellomsg)]),
                                length - sizeof(struct rhellomsg));
            break;
        case RMF_EHELLO:
                                length - sizeof(struct rhellomsg));
            break;
        case RMF_EHELLO:
-           (void)printf("endnode-hello ");
+           ND_PRINT("endnode-hello ");
            if (length < sizeof(struct ehellomsg))
            if (length < sizeof(struct ehellomsg))
-               goto trunc;
-           TCHECK(cmp->cm_ehello);
-           vers = EXTRACT_LE_8BITS(cmp->cm_ehello.eh_vers);
-           eco = EXTRACT_LE_8BITS(cmp->cm_ehello.eh_eco);
-           ueco = EXTRACT_LE_8BITS(cmp->cm_ehello.eh_ueco);
-           memcpy((char *)&srcea, (char *)&(cmp->cm_ehello.eh_src),
-               sizeof(srcea));
-           src = EXTRACT_LE_16BITS(srcea.dne_remote.dne_nodeaddr);
-           info = EXTRACT_LE_8BITS(cmp->cm_ehello.eh_info);
-           blksize = EXTRACT_LE_16BITS(cmp->cm_ehello.eh_blksize);
+               goto invalid;
+           vers = GET_U_1(cmp->cm_ehello.eh_vers);
+           eco = GET_U_1(cmp->cm_ehello.eh_eco);
+           ueco = GET_U_1(cmp->cm_ehello.eh_ueco);
+           src =
+               GET_LE_U_2(cmp->cm_ehello.eh_src.dne_remote.dne_nodeaddr);
+           info = GET_U_1(cmp->cm_ehello.eh_info);
+           blksize = GET_LE_U_2(cmp->cm_ehello.eh_blksize);
            /*seed*/
            /*seed*/
-           memcpy((char *)&rtea, (char *)&(cmp->cm_ehello.eh_router),
-               sizeof(rtea));
-           dst = EXTRACT_LE_16BITS(rtea.dne_remote.dne_nodeaddr);
-           hello = EXTRACT_LE_16BITS(cmp->cm_ehello.eh_hello);
-           other = EXTRACT_LE_8BITS(cmp->cm_ehello.eh_data);
-           print_i_info(info);
-           (void)printf(
-       "vers %d eco %d ueco %d src %s blksize %d rtr %s hello %d data %o",
-                       vers, eco, ueco, dnaddr_string(src),
-                       blksize, dnaddr_string(dst), hello, other);
-           ret = 1;
+           dst =
+               GET_LE_U_2(cmp->cm_ehello.eh_router.dne_remote.dne_nodeaddr);
+           hello = GET_LE_U_2(cmp->cm_ehello.eh_hello);
+           other = GET_U_1(cmp->cm_ehello.eh_data);
+           print_i_info(ndo, info);
+           ND_PRINT("vers %u eco %u ueco %u src %s blksize %u rtr %s hello %u data %o",
+                       vers, eco, ueco, dnaddr_string(ndo, src),
+                       blksize, dnaddr_string(ndo, dst), hello, other);
            break;
 
        default:
            break;
 
        default:
-           (void)printf("unknown control message");
-           default_print((u_char *)rhp, min(length, caplen));
-           ret = 1;
+           ND_PRINT("unknown control message");
+           ND_DEFAULTPRINT((const u_char *)rhp, ND_MIN(length, caplen));
            break;
        }
            break;
        }
-       return (ret);
+       return (1);
 
 
-trunc:
+invalid:
        return (0);
 }
 
 static void
        return (0);
 }
 
 static void
-print_t_info(int info)
+print_t_info(netdissect_options *ndo,
+             u_int info)
 {
 {
-       int ntype = info & 3;
+       u_int ntype = info & 3;
        switch (ntype) {
        switch (ntype) {
-       case 0: (void)printf("reserved-ntype? "); break;
-       case TI_L2ROUT: (void)printf("l2rout "); break;
-       case TI_L1ROUT: (void)printf("l1rout "); break;
-       case TI_ENDNODE: (void)printf("endnode "); break;
+       case 0: ND_PRINT("reserved-ntype? "); break;
+       case TI_L2ROUT: ND_PRINT("l2rout "); break;
+       case TI_L1ROUT: ND_PRINT("l1rout "); break;
+       case TI_ENDNODE: ND_PRINT("endnode "); break;
        }
        if (info & TI_VERIF)
        }
        if (info & TI_VERIF)
-           (void)printf("verif ");
+           ND_PRINT("verif ");
        if (info & TI_BLOCK)
        if (info & TI_BLOCK)
-           (void)printf("blo ");
+           ND_PRINT("blo ");
 }
 
 }
 
-static int
-print_l1_routes(const char *rp, u_int len)
+static void
+print_l1_routes(netdissect_options *ndo,
+                const u_char *rp, u_int len)
 {
 {
-       int count;
-       int id;
-       int info;
+       u_int count;
+       u_int id;
+       u_int info;
 
        /* The last short is a checksum */
        while (len > (3 * sizeof(short))) {
 
        /* The last short is a checksum */
        while (len > (3 * sizeof(short))) {
-           TCHECK2(*rp, 3 * sizeof(short));
-           count = EXTRACT_LE_16BITS(rp);
+           ND_TCHECK_LEN(rp, 3 * sizeof(short));
+           count = GET_LE_U_2(rp);
            if (count > 1024)
            if (count > 1024)
-               return (1);     /* seems to be bogus from here on */
+               return; /* seems to be bogus from here on */
            rp += sizeof(short);
            len -= sizeof(short);
            rp += sizeof(short);
            len -= sizeof(short);
-           id = EXTRACT_LE_16BITS(rp);
+           id = GET_LE_U_2(rp);
            rp += sizeof(short);
            len -= sizeof(short);
            rp += sizeof(short);
            len -= sizeof(short);
-           info = EXTRACT_LE_16BITS(rp);
+           info = GET_LE_U_2(rp);
            rp += sizeof(short);
            len -= sizeof(short);
            rp += sizeof(short);
            len -= sizeof(short);
-           (void)printf("{ids %d-%d cost %d hops %d} ", id, id + count,
+           ND_PRINT("{ids %u-%u cost %u hops %u} ", id, id + count,
                            RI_COST(info), RI_HOPS(info));
        }
                            RI_COST(info), RI_HOPS(info));
        }
-       return (1);
-
-trunc:
-       return (0);
 }
 
 }
 
-static int
-print_l2_routes(const char *rp, u_int len)
+static void
+print_l2_routes(netdissect_options *ndo,
+                const u_char *rp, u_int len)
 {
 {
-       int count;
-       int area;
-       int info;
+       u_int count;
+       u_int area;
+       u_int info;
 
        /* The last short is a checksum */
        while (len > (3 * sizeof(short))) {
 
        /* The last short is a checksum */
        while (len > (3 * sizeof(short))) {
-           TCHECK2(*rp, 3 * sizeof(short));
-           count = EXTRACT_LE_16BITS(rp);
+           ND_TCHECK_LEN(rp, 3 * sizeof(short));
+           count = GET_LE_U_2(rp);
            if (count > 1024)
            if (count > 1024)
-               return (1);     /* seems to be bogus from here on */
+               return; /* seems to be bogus from here on */
            rp += sizeof(short);
            len -= sizeof(short);
            rp += sizeof(short);
            len -= sizeof(short);
-           area = EXTRACT_LE_16BITS(rp);
+           area = GET_LE_U_2(rp);
            rp += sizeof(short);
            len -= sizeof(short);
            rp += sizeof(short);
            len -= sizeof(short);
-           info = EXTRACT_LE_16BITS(rp);
+           info = GET_LE_U_2(rp);
            rp += sizeof(short);
            len -= sizeof(short);
            rp += sizeof(short);
            len -= sizeof(short);
-           (void)printf("{areas %d-%d cost %d hops %d} ", area, area + count,
+           ND_PRINT("{areas %u-%u cost %u hops %u} ", area, area + count,
                            RI_COST(info), RI_HOPS(info));
        }
                            RI_COST(info), RI_HOPS(info));
        }
-       return (1);
-
-trunc:
-       return (0);
 }
 
 static void
 }
 
 static void
-print_i_info(int info)
+print_i_info(netdissect_options *ndo,
+             u_int info)
 {
 {
-       int ntype = info & II_TYPEMASK;
+       u_int ntype = info & II_TYPEMASK;
        switch (ntype) {
        switch (ntype) {
-       case 0: (void)printf("reserved-ntype? "); break;
-       case II_L2ROUT: (void)printf("l2rout "); break;
-       case II_L1ROUT: (void)printf("l1rout "); break;
-       case II_ENDNODE: (void)printf("endnode "); break;
+       case 0: ND_PRINT("reserved-ntype? "); break;
+       case II_L2ROUT: ND_PRINT("l2rout "); break;
+       case II_L1ROUT: ND_PRINT("l1rout "); break;
+       case II_ENDNODE: ND_PRINT("endnode "); break;
        }
        if (info & II_VERIF)
        }
        if (info & II_VERIF)
-           (void)printf("verif ");
+           ND_PRINT("verif ");
        if (info & II_NOMCAST)
        if (info & II_NOMCAST)
-           (void)printf("nomcast ");
+           ND_PRINT("nomcast ");
        if (info & II_BLOCK)
        if (info & II_BLOCK)
-           (void)printf("blo ");
+           ND_PRINT("blo ");
 }
 
 }
 
-static int
-print_elist(const char *elp _U_, u_int len _U_)
+static void
+print_elist(const u_char *elp _U_, u_int len _U_)
 {
        /* Not enough examples available for me to debug this */
 {
        /* Not enough examples available for me to debug this */
-       return (1);
 }
 
 static int
 }
 
 static int
-print_nsp(const u_char *nspp, u_int nsplen)
+print_nsp(netdissect_options *ndo,
+          const u_char *nspp, u_int nsplen)
 {
 {
-       const struct nsphdr *nsphp = (struct nsphdr *)nspp;
-       int dst, src, flags;
+       const struct nsphdr *nsphp = (const struct nsphdr *)nspp;
+       u_int dst, src, flags;
 
 
-       if (nsplen < sizeof(struct nsphdr))
-               goto trunc;
-       TCHECK(*nsphp);
-       flags = EXTRACT_LE_8BITS(nsphp->nh_flags);
-       dst = EXTRACT_LE_16BITS(nsphp->nh_dst);
-       src = EXTRACT_LE_16BITS(nsphp->nh_src);
+       if (nsplen < sizeof(struct nsphdr)) {
+               ND_PRINT(" (nsplen %u < %zu)", nsplen, sizeof(struct nsphdr));
+               goto invalid;
+       }
+       flags = GET_U_1(nsphp->nh_flags);
+       dst = GET_LE_U_2(nsphp->nh_dst);
+       src = GET_LE_U_2(nsphp->nh_src);
 
        switch (flags & NSP_TYPEMASK) {
        case MFT_DATA:
 
        switch (flags & NSP_TYPEMASK) {
        case MFT_DATA:
@@ -864,219 +813,188 @@ print_nsp(const u_char *nspp, u_int nsplen)
            case MFS_MOM:
            case MFS_EOM:
            case MFS_BOM+MFS_EOM:
            case MFS_MOM:
            case MFS_EOM:
            case MFS_BOM+MFS_EOM:
-               printf("data %d>%d ", src, dst);
+               ND_PRINT("data %u>%u ", src, dst);
                {
                {
-                   struct seghdr *shp = (struct seghdr *)nspp;
-                   int ack;
-#ifdef PRINT_NSPDATA
-                   u_char *dp;
-#endif
+                   const struct seghdr *shp = (const struct seghdr *)nspp;
+                   u_int ack;
                    u_int data_off = sizeof(struct minseghdr);
 
                    if (nsplen < data_off)
                    u_int data_off = sizeof(struct minseghdr);
 
                    if (nsplen < data_off)
-                       goto trunc;
-                   TCHECK(shp->sh_seq[0]);
-                   ack = EXTRACT_LE_16BITS(shp->sh_seq[0]);
+                       goto invalid;
+                   ack = GET_LE_U_2(shp->sh_seq[0]);
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
-                           (void)printf("nak %d ", ack & SGQ_MASK);
+                           ND_PRINT("nak %u ", ack & SGQ_MASK);
                        else
                        else
-                           (void)printf("ack %d ", ack & SGQ_MASK);
+                           ND_PRINT("ack %u ", ack & SGQ_MASK);
                        data_off += sizeof(short);
                        if (nsplen < data_off)
                        data_off += sizeof(short);
                        if (nsplen < data_off)
-                           goto trunc;
-                       TCHECK(shp->sh_seq[1]);
-                       ack = EXTRACT_LE_16BITS(shp->sh_seq[1]);
+                           goto invalid;
+                       ack = GET_LE_U_2(shp->sh_seq[1]);
                        if (ack & SGQ_OACK) {   /* ackoth field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
                        if (ack & SGQ_OACK) {   /* ackoth field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
-                               (void)printf("onak %d ", ack & SGQ_MASK);
+                               ND_PRINT("onak %u ", ack & SGQ_MASK);
                            else
                            else
-                               (void)printf("oack %d ", ack & SGQ_MASK);
+                               ND_PRINT("oack %u ", ack & SGQ_MASK);
                            data_off += sizeof(short);
                            if (nsplen < data_off)
                            data_off += sizeof(short);
                            if (nsplen < data_off)
-                               goto trunc;
-                           TCHECK(shp->sh_seq[2]);
-                           ack = EXTRACT_LE_16BITS(shp->sh_seq[2]);
+                               goto invalid;
+                           ack = GET_LE_U_2(shp->sh_seq[2]);
                        }
                    }
                        }
                    }
-                   (void)printf("seg %d ", ack & SGQ_MASK);
-#ifdef PRINT_NSPDATA
-                   if (nsplen > data_off) {
-                       dp = &(nspp[data_off]);
-                       TCHECK2(*dp, nsplen - data_off);
-                       pdata(dp, nsplen - data_off);
-                   }
-#endif
+                   ND_PRINT("seg %u ", ack & SGQ_MASK);
                }
                break;
            case MFS_ILS+MFS_INT:
                }
                break;
            case MFS_ILS+MFS_INT:
-               printf("intr ");
+               ND_PRINT("intr ");
                {
                {
-                   struct seghdr *shp = (struct seghdr *)nspp;
-                   int ack;
-#ifdef PRINT_NSPDATA
-                   u_char *dp;
-#endif
+                   const struct seghdr *shp = (const struct seghdr *)nspp;
+                   u_int ack;
                    u_int data_off = sizeof(struct minseghdr);
 
                    if (nsplen < data_off)
                    u_int data_off = sizeof(struct minseghdr);
 
                    if (nsplen < data_off)
-                       goto trunc;
-                   TCHECK(shp->sh_seq[0]);
-                   ack = EXTRACT_LE_16BITS(shp->sh_seq[0]);
+                       goto invalid;
+                   ack = GET_LE_U_2(shp->sh_seq[0]);
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
-                           (void)printf("nak %d ", ack & SGQ_MASK);
+                           ND_PRINT("nak %u ", ack & SGQ_MASK);
                        else
                        else
-                           (void)printf("ack %d ", ack & SGQ_MASK);
+                           ND_PRINT("ack %u ", ack & SGQ_MASK);
                        data_off += sizeof(short);
                        if (nsplen < data_off)
                        data_off += sizeof(short);
                        if (nsplen < data_off)
-                           goto trunc;
-                       TCHECK(shp->sh_seq[1]);
-                       ack = EXTRACT_LE_16BITS(shp->sh_seq[1]);
+                           goto invalid;
+                       ack = GET_LE_U_2(shp->sh_seq[1]);
                        if (ack & SGQ_OACK) {   /* ackdat field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
                        if (ack & SGQ_OACK) {   /* ackdat field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
-                               (void)printf("nakdat %d ", ack & SGQ_MASK);
+                               ND_PRINT("nakdat %u ", ack & SGQ_MASK);
                            else
                            else
-                               (void)printf("ackdat %d ", ack & SGQ_MASK);
+                               ND_PRINT("ackdat %u ", ack & SGQ_MASK);
                            data_off += sizeof(short);
                            if (nsplen < data_off)
                            data_off += sizeof(short);
                            if (nsplen < data_off)
-                               goto trunc;
-                           TCHECK(shp->sh_seq[2]);
-                           ack = EXTRACT_LE_16BITS(shp->sh_seq[2]);
+                               goto invalid;
+                           ack = GET_LE_U_2(shp->sh_seq[2]);
                        }
                    }
                        }
                    }
-                   (void)printf("seg %d ", ack & SGQ_MASK);
-#ifdef PRINT_NSPDATA
-                   if (nsplen > data_off) {
-                       dp = &(nspp[data_off]);
-                       TCHECK2(*dp, nsplen - data_off);
-                       pdata(dp, nsplen - data_off);
-                   }
-#endif
+                   ND_PRINT("seg %u ", ack & SGQ_MASK);
                }
                break;
            case MFS_ILS:
                }
                break;
            case MFS_ILS:
-               (void)printf("link-service %d>%d ", src, dst);
+               ND_PRINT("link-service %u>%u ", src, dst);
                {
                {
-                   struct seghdr *shp = (struct seghdr *)nspp;
-                   struct lsmsg *lsmp =
-                       (struct lsmsg *)&(nspp[sizeof(struct seghdr)]);
-                   int ack;
-                   int lsflags, fcval;
+                   const struct seghdr *shp = (const struct seghdr *)nspp;
+                   const struct lsmsg *lsmp =
+                       (const struct lsmsg *)(nspp + sizeof(struct seghdr));
+                   u_int ack;
+                   u_int lsflags, fcval;
 
                    if (nsplen < sizeof(struct seghdr) + sizeof(struct lsmsg))
 
                    if (nsplen < sizeof(struct seghdr) + sizeof(struct lsmsg))
-                       goto trunc;
-                   TCHECK(shp->sh_seq[0]);
-                   ack = EXTRACT_LE_16BITS(shp->sh_seq[0]);
+                       goto invalid;
+                   ack = GET_LE_U_2(shp->sh_seq[0]);
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
-                           (void)printf("nak %d ", ack & SGQ_MASK);
+                           ND_PRINT("nak %u ", ack & SGQ_MASK);
                        else
                        else
-                           (void)printf("ack %d ", ack & SGQ_MASK);
-                       TCHECK(shp->sh_seq[1]);
-                       ack = EXTRACT_LE_16BITS(shp->sh_seq[1]);
+                           ND_PRINT("ack %u ", ack & SGQ_MASK);
+                       ack = GET_LE_U_2(shp->sh_seq[1]);
                        if (ack & SGQ_OACK) {   /* ackdat field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
                        if (ack & SGQ_OACK) {   /* ackdat field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
-                               (void)printf("nakdat %d ", ack & SGQ_MASK);
+                               ND_PRINT("nakdat %u ", ack & SGQ_MASK);
                            else
                            else
-                               (void)printf("ackdat %d ", ack & SGQ_MASK);
-                           TCHECK(shp->sh_seq[2]);
-                           ack = EXTRACT_LE_16BITS(shp->sh_seq[2]);
+                               ND_PRINT("ackdat %u ", ack & SGQ_MASK);
+                           ack = GET_LE_U_2(shp->sh_seq[2]);
                        }
                    }
                        }
                    }
-                   (void)printf("seg %d ", ack & SGQ_MASK);
-                   TCHECK(*lsmp);
-                   lsflags = EXTRACT_LE_8BITS(lsmp->ls_lsflags);
-                   fcval = EXTRACT_LE_8BITS(lsmp->ls_fcval);
+                   ND_PRINT("seg %u ", ack & SGQ_MASK);
+                   lsflags = GET_U_1(lsmp->ls_lsflags);
+                   fcval = GET_U_1(lsmp->ls_fcval);
                    switch (lsflags & LSI_MASK) {
                    case LSI_DATA:
                    switch (lsflags & LSI_MASK) {
                    case LSI_DATA:
-                       (void)printf("dat seg count %d ", fcval);
+                       ND_PRINT("dat seg count %u ", fcval);
                        switch (lsflags & LSM_MASK) {
                        case LSM_NOCHANGE:
                            break;
                        case LSM_DONOTSEND:
                        switch (lsflags & LSM_MASK) {
                        case LSM_NOCHANGE:
                            break;
                        case LSM_DONOTSEND:
-                           (void)printf("donotsend-data ");
+                           ND_PRINT("donotsend-data ");
                            break;
                        case LSM_SEND:
                            break;
                        case LSM_SEND:
-                           (void)printf("send-data ");
+                           ND_PRINT("send-data ");
                            break;
                        default:
                            break;
                        default:
-                           (void)printf("reserved-fcmod? %x", lsflags);
+                           ND_PRINT("reserved-fcmod? %x", lsflags);
                            break;
                        }
                        break;
                    case LSI_INTR:
                            break;
                        }
                        break;
                    case LSI_INTR:
-                       (void)printf("intr req count %d ", fcval);
+                       ND_PRINT("intr req count %u ", fcval);
                        break;
                    default:
                        break;
                    default:
-                       (void)printf("reserved-fcval-int? %x", lsflags);
+                       ND_PRINT("reserved-fcval-int? %x", lsflags);
                        break;
                    }
                }
                break;
            default:
                        break;
                    }
                }
                break;
            default:
-               (void)printf("reserved-subtype? %x %d > %d", flags, src, dst);
+               ND_PRINT("reserved-subtype? %x %u > %u", flags, src, dst);
                break;
            }
            break;
        case MFT_ACK:
            switch (flags & NSP_SUBMASK) {
            case MFS_DACK:
                break;
            }
            break;
        case MFT_ACK:
            switch (flags & NSP_SUBMASK) {
            case MFS_DACK:
-               (void)printf("data-ack %d>%d ", src, dst);
+               ND_PRINT("data-ack %u>%u ", src, dst);
                {
                {
-                   struct ackmsg *amp = (struct ackmsg *)nspp;
-                   int ack;
+                   const struct ackmsg *amp = (const struct ackmsg *)nspp;
+                   u_int ack;
 
                    if (nsplen < sizeof(struct ackmsg))
 
                    if (nsplen < sizeof(struct ackmsg))
-                       goto trunc;
-                   TCHECK(*amp);
-                   ack = EXTRACT_LE_16BITS(amp->ak_acknum[0]);
+                       goto invalid;
+                   ND_TCHECK_SIZE(amp);
+                   ack = GET_LE_U_2(amp->ak_acknum[0]);
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
-                           (void)printf("nak %d ", ack & SGQ_MASK);
+                           ND_PRINT("nak %u ", ack & SGQ_MASK);
                        else
                        else
-                           (void)printf("ack %d ", ack & SGQ_MASK);
-                       ack = EXTRACT_LE_16BITS(amp->ak_acknum[1]);
+                           ND_PRINT("ack %u ", ack & SGQ_MASK);
+                       ack = GET_LE_U_2(amp->ak_acknum[1]);
                        if (ack & SGQ_OACK) {   /* ackoth field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
                        if (ack & SGQ_OACK) {   /* ackoth field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
-                               (void)printf("onak %d ", ack & SGQ_MASK);
+                               ND_PRINT("onak %u ", ack & SGQ_MASK);
                            else
                            else
-                               (void)printf("oack %d ", ack & SGQ_MASK);
+                               ND_PRINT("oack %u ", ack & SGQ_MASK);
                        }
                    }
                }
                break;
            case MFS_IACK:
                        }
                    }
                }
                break;
            case MFS_IACK:
-               (void)printf("ils-ack %d>%d ", src, dst);
+               ND_PRINT("ils-ack %u>%u ", src, dst);
                {
                {
-                   struct ackmsg *amp = (struct ackmsg *)nspp;
-                   int ack;
+                   const struct ackmsg *amp = (const struct ackmsg *)nspp;
+                   u_int ack;
 
                    if (nsplen < sizeof(struct ackmsg))
 
                    if (nsplen < sizeof(struct ackmsg))
-                       goto trunc;
-                   TCHECK(*amp);
-                   ack = EXTRACT_LE_16BITS(amp->ak_acknum[0]);
+                       goto invalid;
+                   ND_TCHECK_SIZE(amp);
+                   ack = GET_LE_U_2(amp->ak_acknum[0]);
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
                    if (ack & SGQ_ACK) {        /* acknum field */
                        if ((ack & SGQ_NAK) == SGQ_NAK)
-                           (void)printf("nak %d ", ack & SGQ_MASK);
+                           ND_PRINT("nak %u ", ack & SGQ_MASK);
                        else
                        else
-                           (void)printf("ack %d ", ack & SGQ_MASK);
-                       TCHECK(amp->ak_acknum[1]);
-                       ack = EXTRACT_LE_16BITS(amp->ak_acknum[1]);
+                           ND_PRINT("ack %u ", ack & SGQ_MASK);
+                       ack = GET_LE_U_2(amp->ak_acknum[1]);
                        if (ack & SGQ_OACK) {   /* ackdat field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
                        if (ack & SGQ_OACK) {   /* ackdat field */
                            if ((ack & SGQ_ONAK) == SGQ_ONAK)
-                               (void)printf("nakdat %d ", ack & SGQ_MASK);
+                               ND_PRINT("nakdat %u ", ack & SGQ_MASK);
                            else
                            else
-                               (void)printf("ackdat %d ", ack & SGQ_MASK);
+                               ND_PRINT("ackdat %u ", ack & SGQ_MASK);
                        }
                    }
                }
                break;
            case MFS_CACK:
                        }
                    }
                }
                break;
            case MFS_CACK:
-               (void)printf("conn-ack %d", dst);
+               ND_PRINT("conn-ack %u", dst);
                break;
            default:
                break;
            default:
-               (void)printf("reserved-acktype? %x %d > %d", flags, src, dst);
+               ND_PRINT("reserved-acktype? %x %u > %u", flags, src, dst);
                break;
            }
            break;
                break;
            }
            break;
@@ -1085,172 +1003,132 @@ print_nsp(const u_char *nspp, u_int nsplen)
            case MFS_CI:
            case MFS_RCI:
                if ((flags & NSP_SUBMASK) == MFS_CI)
            case MFS_CI:
            case MFS_RCI:
                if ((flags & NSP_SUBMASK) == MFS_CI)
-                   (void)printf("conn-initiate ");
+                   ND_PRINT("conn-initiate ");
                else
                else
-                   (void)printf("retrans-conn-initiate ");
-               (void)printf("%d>%d ", src, dst);
+                   ND_PRINT("retrans-conn-initiate ");
+               ND_PRINT("%u>%u ", src, dst);
                {
                {
-                   struct cimsg *cimp = (struct cimsg *)nspp;
-                   int services, info, segsize;
-#ifdef PRINT_NSPDATA
-                   u_char *dp;
-#endif
+                   const struct cimsg *cimp = (const struct cimsg *)nspp;
+                   u_int services, info, segsize;
 
                    if (nsplen < sizeof(struct cimsg))
 
                    if (nsplen < sizeof(struct cimsg))
-                       goto trunc;
-                   TCHECK(*cimp);
-                   services = EXTRACT_LE_8BITS(cimp->ci_services);
-                   info = EXTRACT_LE_8BITS(cimp->ci_info);
-                   segsize = EXTRACT_LE_16BITS(cimp->ci_segsize);
+                       goto invalid;
+                   services = GET_U_1(cimp->ci_services);
+                   info = GET_U_1(cimp->ci_info);
+                   segsize = GET_LE_U_2(cimp->ci_segsize);
 
                    switch (services & COS_MASK) {
                    case COS_NONE:
                        break;
                    case COS_SEGMENT:
 
                    switch (services & COS_MASK) {
                    case COS_NONE:
                        break;
                    case COS_SEGMENT:
-                       (void)printf("seg ");
+                       ND_PRINT("seg ");
                        break;
                    case COS_MESSAGE:
                        break;
                    case COS_MESSAGE:
-                       (void)printf("msg ");
-                       break;
-                   case COS_CRYPTSER:
-                       (void)printf("crypt ");
+                       ND_PRINT("msg ");
                        break;
                    }
                    switch (info & COI_MASK) {
                    case COI_32:
                        break;
                    }
                    switch (info & COI_MASK) {
                    case COI_32:
-                       (void)printf("ver 3.2 ");
+                       ND_PRINT("ver 3.2 ");
                        break;
                    case COI_31:
                        break;
                    case COI_31:
-                       (void)printf("ver 3.1 ");
+                       ND_PRINT("ver 3.1 ");
                        break;
                    case COI_40:
                        break;
                    case COI_40:
-                       (void)printf("ver 4.0 ");
+                       ND_PRINT("ver 4.0 ");
                        break;
                    case COI_41:
                        break;
                    case COI_41:
-                       (void)printf("ver 4.1 ");
+                       ND_PRINT("ver 4.1 ");
                        break;
                    }
                        break;
                    }
-                   (void)printf("segsize %d ", segsize);
-#ifdef PRINT_NSPDATA
-                   if (nsplen > sizeof(struct cimsg)) {
-                       dp = &(nspp[sizeof(struct cimsg)]);
-                       TCHECK2(*dp, nsplen - sizeof(struct cimsg));
-                       pdata(dp, nsplen - sizeof(struct cimsg));
-                   }
-#endif
+                   ND_PRINT("segsize %u ", segsize);
                }
                break;
            case MFS_CC:
                }
                break;
            case MFS_CC:
-               (void)printf("conn-confirm %d>%d ", src, dst);
+               ND_PRINT("conn-confirm %u>%u ", src, dst);
                {
                {
-                   struct ccmsg *ccmp = (struct ccmsg *)nspp;
-                   int services, info;
+                   const struct ccmsg *ccmp = (const struct ccmsg *)nspp;
+                   u_int services, info;
                    u_int segsize, optlen;
                    u_int segsize, optlen;
-#ifdef PRINT_NSPDATA
-                   u_char *dp;
-#endif
 
                    if (nsplen < sizeof(struct ccmsg))
 
                    if (nsplen < sizeof(struct ccmsg))
-                       goto trunc;
-                   TCHECK(*ccmp);
-                   services = EXTRACT_LE_8BITS(ccmp->cc_services);
-                   info = EXTRACT_LE_8BITS(ccmp->cc_info);
-                   segsize = EXTRACT_LE_16BITS(ccmp->cc_segsize);
-                   optlen = EXTRACT_LE_8BITS(ccmp->cc_optlen);
+                       goto invalid;
+                   services = GET_U_1(ccmp->cc_services);
+                   info = GET_U_1(ccmp->cc_info);
+                   segsize = GET_LE_U_2(ccmp->cc_segsize);
+                   optlen = GET_U_1(ccmp->cc_optlen);
 
                    switch (services & COS_MASK) {
                    case COS_NONE:
                        break;
                    case COS_SEGMENT:
 
                    switch (services & COS_MASK) {
                    case COS_NONE:
                        break;
                    case COS_SEGMENT:
-                       (void)printf("seg ");
+                       ND_PRINT("seg ");
                        break;
                    case COS_MESSAGE:
                        break;
                    case COS_MESSAGE:
-                       (void)printf("msg ");
-                       break;
-                   case COS_CRYPTSER:
-                       (void)printf("crypt ");
+                       ND_PRINT("msg ");
                        break;
                    }
                    switch (info & COI_MASK) {
                    case COI_32:
                        break;
                    }
                    switch (info & COI_MASK) {
                    case COI_32:
-                       (void)printf("ver 3.2 ");
+                       ND_PRINT("ver 3.2 ");
                        break;
                    case COI_31:
                        break;
                    case COI_31:
-                       (void)printf("ver 3.1 ");
+                       ND_PRINT("ver 3.1 ");
                        break;
                    case COI_40:
                        break;
                    case COI_40:
-                       (void)printf("ver 4.0 ");
+                       ND_PRINT("ver 4.0 ");
                        break;
                    case COI_41:
                        break;
                    case COI_41:
-                       (void)printf("ver 4.1 ");
+                       ND_PRINT("ver 4.1 ");
                        break;
                    }
                        break;
                    }
-                   (void)printf("segsize %d ", segsize);
+                   ND_PRINT("segsize %u ", segsize);
                    if (optlen) {
                    if (optlen) {
-                       (void)printf("optlen %d ", optlen);
-#ifdef PRINT_NSPDATA
-                       if (optlen > nsplen - sizeof(struct ccmsg))
-                           goto trunc;
-                       dp = &(nspp[sizeof(struct ccmsg)]);
-                       TCHECK2(*dp, optlen);
-                       pdata(dp, optlen);
-#endif
+                       ND_PRINT("optlen %u ", optlen);
                    }
                }
                break;
            case MFS_DI:
                    }
                }
                break;
            case MFS_DI:
-               (void)printf("disconn-initiate %d>%d ", src, dst);
+               ND_PRINT("disconn-initiate %u>%u ", src, dst);
                {
                {
-                   struct dimsg *dimp = (struct dimsg *)nspp;
-                   int reason;
+                   const struct dimsg *dimp = (const struct dimsg *)nspp;
+                   u_int reason;
                    u_int optlen;
                    u_int optlen;
-#ifdef PRINT_NSPDATA
-                   u_char *dp;
-#endif
 
                    if (nsplen < sizeof(struct dimsg))
 
                    if (nsplen < sizeof(struct dimsg))
-                       goto trunc;
-                   TCHECK(*dimp);
-                   reason = EXTRACT_LE_16BITS(dimp->di_reason);
-                   optlen = EXTRACT_LE_8BITS(dimp->di_optlen);
+                       goto invalid;
+                   reason = GET_LE_U_2(dimp->di_reason);
+                   optlen = GET_U_1(dimp->di_optlen);
 
 
-                   print_reason(reason);
+                   print_reason(ndo, reason);
                    if (optlen) {
                    if (optlen) {
-                       (void)printf("optlen %d ", optlen);
-#ifdef PRINT_NSPDATA
-                       if (optlen > nsplen - sizeof(struct dimsg))
-                           goto trunc;
-                       dp = &(nspp[sizeof(struct dimsg)]);
-                       TCHECK2(*dp, optlen);
-                       pdata(dp, optlen);
-#endif
+                       ND_PRINT("optlen %u ", optlen);
                    }
                }
                break;
            case MFS_DC:
                    }
                }
                break;
            case MFS_DC:
-               (void)printf("disconn-confirm %d>%d ", src, dst);
+               ND_PRINT("disconn-confirm %u>%u ", src, dst);
                {
                {
-                   struct dcmsg *dcmp = (struct dcmsg *)nspp;
-                   int reason;
+                   const struct dcmsg *dcmp = (const struct dcmsg *)nspp;
+                   u_int reason;
 
 
-                   TCHECK(*dcmp);
-                   reason = EXTRACT_LE_16BITS(dcmp->dc_reason);
+                   reason = GET_LE_U_2(dcmp->dc_reason);
 
 
-                   print_reason(reason);
+                   print_reason(ndo, reason);
                }
                break;
            default:
                }
                break;
            default:
-               (void)printf("reserved-ctltype? %x %d > %d", flags, src, dst);
+               ND_PRINT("reserved-ctltype? %x %u > %u", flags, src, dst);
                break;
            }
            break;
        default:
                break;
            }
            break;
        default:
-           (void)printf("reserved-type? %x %d > %d", flags, src, dst);
+           ND_PRINT("reserved-type? %x %u > %u", flags, src, dst);
            break;
        }
        return (1);
 
            break;
        }
        return (1);
 
-trunc:
+invalid:
        return (0);
 }
 
        return (0);
 }
 
@@ -1281,50 +1159,24 @@ static const struct tok reason2str[] = {
 };
 
 static void
 };
 
 static void
-print_reason(register int reason)
+print_reason(netdissect_options *ndo,
+             u_int reason)
 {
 {
-       printf("%s ", tok2str(reason2str, "reason-%d", reason));
+       ND_PRINT("%s ", tok2str(reason2str, "reason-%u", reason));
 }
 
 const char *
 }
 
 const char *
-dnnum_string(u_short dnaddr)
+dnnum_string(netdissect_options *ndo, u_short dnaddr)
 {
        char *str;
        size_t siz;
 {
        char *str;
        size_t siz;
-       int area = (u_short)(dnaddr & AREAMASK) >> AREASHIFT;
-       int node = dnaddr & NODEMASK;
+       u_int area = (u_short)(dnaddr & AREAMASK) >> AREASHIFT;
+       u_int node = dnaddr & NODEMASK;
 
 
+       /* malloc() return used by the 'dnaddrtable' hash table: do not free() */
        str = (char *)malloc(siz = sizeof("00.0000"));
        if (str == NULL)
        str = (char *)malloc(siz = sizeof("00.0000"));
        if (str == NULL)
-               error("dnnum_string: malloc");
-       snprintf(str, siz, "%d.%d", area, node);
+               (*ndo->ndo_error)(ndo, S_ERR_ND_MEM_ALLOC, "%s: malloc", __func__);
+       snprintf(str, siz, "%u.%u", area, node);
        return(str);
 }
        return(str);
 }
-
-const char *
-dnname_string(u_short dnaddr)
-{
-#ifdef HAVE_DNET_HTOA
-       struct dn_naddr dna;
-
-       dna.a_len = sizeof(short);
-       memcpy((char *)dna.a_addr, (char *)&dnaddr, sizeof(short));
-       return (strdup(dnet_htoa(&dna)));
-#else
-       return(dnnum_string(dnaddr));   /* punt */
-#endif
-}
-
-#ifdef PRINT_NSPDATA
-static void
-pdata(u_char *dp, u_int maxlen)
-{
-       char c;
-       u_int x = maxlen;
-
-       while (x-- > 0) {
-           c = *dp++;
-           safeputchar(gndo, c);
-       }
-}
-#endif
[mirror] the TCPdump network dissector