]> The Tcpdump Group git mirrors - tcpdump/blobdiff - print-nfs.c
More nd_ification of structures.
[tcpdump] / print-nfs.c
index a2d3aafe75f614968b98bbdd7ea8c9fc99d3b6b8..a4823842a7be7e365684eba865bfbf68b61ceda0 100644 (file)
  * MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE.
  */
 
-#define NETDISSECT_REWORKED
+/* \summary: Network File System (NFS) printer */
+
 #ifdef HAVE_CONFIG_H
 #include "config.h"
 #endif
 
-#include <tcpdump-stdinc.h>
+#include <netdissect-stdinc.h>
 
 #include <stdio.h>
 #include <string.h>
 
-#include "interface.h"
+#include "netdissect.h"
 #include "addrtoname.h"
 #include "extract.h"
 
 #include "nfsfh.h"
 
 #include "ip.h"
-#ifdef INET6
 #include "ip6.h"
-#endif
 #include "rpc_auth.h"
 #include "rpc_msg.h"
 
 static const char tstr[] = " [|nfs]";
 
-static void nfs_printfh(netdissect_options *, const u_int32_t *, const u_int);
+static void nfs_printfh(netdissect_options *, const uint32_t *, const u_int);
 static int xid_map_enter(netdissect_options *, const struct sunrpc_msg *, const u_char *);
 static int xid_map_find(const struct sunrpc_msg *, const u_char *,
-                           u_int32_t *, u_int32_t *);
-static void interp_reply(netdissect_options *, const struct sunrpc_msg *, u_int32_t, u_int32_t, int);
-static const u_int32_t *parse_post_op_attr(netdissect_options *, const u_int32_t *, int);
+                           uint32_t *, uint32_t *);
+static void interp_reply(netdissect_options *, const struct sunrpc_msg *, uint32_t, uint32_t, int);
+static const uint32_t *parse_post_op_attr(netdissect_options *, const uint32_t *, int);
 
 /*
  * Mapping of old NFS Version 2 RPC numbers to generic numbers.
  */
-u_int32_t nfsv3_procid[NFS_NPROCS] = {
+static uint32_t nfsv3_procid[NFS_NPROCS] = {
        NFSPROC_NULL,
        NFSPROC_GETATTR,
        NFSPROC_SETATTR,
@@ -84,6 +83,33 @@ u_int32_t nfsv3_procid[NFS_NPROCS] = {
        NFSPROC_NOOP
 };
 
+static const struct tok nfsproc_str[] = {
+       { NFSPROC_NOOP,        "nop"         },
+       { NFSPROC_NULL,        "null"        },
+       { NFSPROC_GETATTR,     "getattr"     },
+       { NFSPROC_SETATTR,     "setattr"     },
+       { NFSPROC_LOOKUP,      "lookup"      },
+       { NFSPROC_ACCESS,      "access"      },
+       { NFSPROC_READLINK,    "readlink"    },
+       { NFSPROC_READ,        "read"        },
+       { NFSPROC_WRITE,       "write"       },
+       { NFSPROC_CREATE,      "create"      },
+       { NFSPROC_MKDIR,       "mkdir"       },
+       { NFSPROC_SYMLINK,     "symlink"     },
+       { NFSPROC_MKNOD,       "mknod"       },
+       { NFSPROC_REMOVE,      "remove"      },
+       { NFSPROC_RMDIR,       "rmdir"       },
+       { NFSPROC_RENAME,      "rename"      },
+       { NFSPROC_LINK,        "link"        },
+       { NFSPROC_READDIR,     "readdir"     },
+       { NFSPROC_READDIRPLUS, "readdirplus" },
+       { NFSPROC_FSSTAT,      "fsstat"      },
+       { NFSPROC_FSINFO,      "fsinfo"      },
+       { NFSPROC_PATHCONF,    "pathconf"    },
+       { NFSPROC_COMMIT,      "commit"      },
+       { 0, NULL }
+};
+
 /*
  * NFS V2 and V3 status values.
  *
@@ -151,37 +177,49 @@ static const struct tok type2str[] = {
        { 0,            NULL }
 };
 
+static const struct tok sunrpc_auth_str[] = {
+       { SUNRPC_AUTH_OK,           "OK"                                                     },
+       { SUNRPC_AUTH_BADCRED,      "Bogus Credentials (seal broken)"                        },
+       { SUNRPC_AUTH_REJECTEDCRED, "Rejected Credentials (client should begin new session)" },
+       { SUNRPC_AUTH_BADVERF,      "Bogus Verifier (seal broken)"                           },
+       { SUNRPC_AUTH_REJECTEDVERF, "Verifier expired or was replayed"                       },
+       { SUNRPC_AUTH_TOOWEAK,      "Credentials are too weak"                               },
+       { SUNRPC_AUTH_INVALIDRESP,  "Bogus response verifier"                                },
+       { SUNRPC_AUTH_FAILED,       "Unknown failure"                                        },
+       { 0, NULL }
+};
+
+static const struct tok sunrpc_str[] = {
+       { SUNRPC_PROG_UNAVAIL,  "PROG_UNAVAIL"  },
+       { SUNRPC_PROG_MISMATCH, "PROG_MISMATCH" },
+       { SUNRPC_PROC_UNAVAIL,  "PROC_UNAVAIL"  },
+       { SUNRPC_GARBAGE_ARGS,  "GARBAGE_ARGS"  },
+       { SUNRPC_SYSTEM_ERR,    "SYSTEM_ERR"    },
+       { 0, NULL }
+};
+
 static void
 print_nfsaddr(netdissect_options *ndo,
               const u_char *bp, const char *s, const char *d)
 {
-       struct ip *ip;
-#ifdef INET6
-       struct ip6_hdr *ip6;
+       const struct ip *ip;
+       const struct ip6_hdr *ip6;
        char srcaddr[INET6_ADDRSTRLEN], dstaddr[INET6_ADDRSTRLEN];
-#else
-#ifndef INET_ADDRSTRLEN
-#define INET_ADDRSTRLEN        16
-#endif
-       char srcaddr[INET_ADDRSTRLEN], dstaddr[INET_ADDRSTRLEN];
-#endif
 
        srcaddr[0] = dstaddr[0] = '\0';
-       switch (IP_V((struct ip *)bp)) {
+       switch (IP_V((const struct ip *)bp)) {
        case 4:
-               ip = (struct ip *)bp;
-               strlcpy(srcaddr, ipaddr_string(&ip->ip_src), sizeof(srcaddr));
-               strlcpy(dstaddr, ipaddr_string(&ip->ip_dst), sizeof(dstaddr));
+               ip = (const struct ip *)bp;
+               strlcpy(srcaddr, ipaddr_string(ndo, &ip->ip_src), sizeof(srcaddr));
+               strlcpy(dstaddr, ipaddr_string(ndo, &ip->ip_dst), sizeof(dstaddr));
                break;
-#ifdef INET6
        case 6:
-               ip6 = (struct ip6_hdr *)bp;
-               strlcpy(srcaddr, ip6addr_string(&ip6->ip6_src),
+               ip6 = (const struct ip6_hdr *)bp;
+               strlcpy(srcaddr, ip6addr_string(ndo, &ip6->ip6_src),
                    sizeof(srcaddr));
-               strlcpy(dstaddr, ip6addr_string(&ip6->ip6_dst),
+               strlcpy(dstaddr, ip6addr_string(ndo, &ip6->ip6_dst),
                    sizeof(dstaddr));
                break;
-#endif
        default:
                strlcpy(srcaddr, "?", sizeof(srcaddr));
                strlcpy(dstaddr, "?", sizeof(dstaddr));
@@ -191,65 +229,65 @@ print_nfsaddr(netdissect_options *ndo,
        ND_PRINT((ndo, "%s.%s > %s.%s: ", srcaddr, s, dstaddr, d));
 }
 
-static const u_int32_t *
+static const uint32_t *
 parse_sattr3(netdissect_options *ndo,
-             const u_int32_t *dp, struct nfsv3_sattr *sa3)
+             const uint32_t *dp, struct nfsv3_sattr *sa3)
 {
        ND_TCHECK(dp[0]);
-       sa3->sa_modeset = EXTRACT_32BITS(dp);
+       sa3->sa_modeset = EXTRACT_BE_U_4(dp);
        dp++;
        if (sa3->sa_modeset) {
                ND_TCHECK(dp[0]);
-               sa3->sa_mode = EXTRACT_32BITS(dp);
+               sa3->sa_mode = EXTRACT_BE_U_4(dp);
                dp++;
        }
 
        ND_TCHECK(dp[0]);
-       sa3->sa_uidset = EXTRACT_32BITS(dp);
+       sa3->sa_uidset = EXTRACT_BE_U_4(dp);
        dp++;
        if (sa3->sa_uidset) {
                ND_TCHECK(dp[0]);
-               sa3->sa_uid = EXTRACT_32BITS(dp);
+               sa3->sa_uid = EXTRACT_BE_U_4(dp);
                dp++;
        }
 
        ND_TCHECK(dp[0]);
-       sa3->sa_gidset = EXTRACT_32BITS(dp);
+       sa3->sa_gidset = EXTRACT_BE_U_4(dp);
        dp++;
        if (sa3->sa_gidset) {
                ND_TCHECK(dp[0]);
-               sa3->sa_gid = EXTRACT_32BITS(dp);
+               sa3->sa_gid = EXTRACT_BE_U_4(dp);
                dp++;
        }
 
        ND_TCHECK(dp[0]);
-       sa3->sa_sizeset = EXTRACT_32BITS(dp);
+       sa3->sa_sizeset = EXTRACT_BE_U_4(dp);
        dp++;
        if (sa3->sa_sizeset) {
                ND_TCHECK(dp[0]);
-               sa3->sa_size = EXTRACT_32BITS(dp);
+               sa3->sa_size = EXTRACT_BE_U_4(dp);
                dp++;
        }
 
        ND_TCHECK(dp[0]);
-       sa3->sa_atimetype = EXTRACT_32BITS(dp);
+       sa3->sa_atimetype = EXTRACT_BE_U_4(dp);
        dp++;
        if (sa3->sa_atimetype == NFSV3SATTRTIME_TOCLIENT) {
                ND_TCHECK(dp[1]);
-               sa3->sa_atime.nfsv3_sec = EXTRACT_32BITS(dp);
+               sa3->sa_atime.nfsv3_sec = EXTRACT_BE_U_4(dp);
                dp++;
-               sa3->sa_atime.nfsv3_nsec = EXTRACT_32BITS(dp);
+               sa3->sa_atime.nfsv3_nsec = EXTRACT_BE_U_4(dp);
                dp++;
        }
 
        ND_TCHECK(dp[0]);
-       sa3->sa_mtimetype = EXTRACT_32BITS(dp);
+       sa3->sa_mtimetype = EXTRACT_BE_U_4(dp);
        dp++;
        if (sa3->sa_mtimetype == NFSV3SATTRTIME_TOCLIENT) {
                ND_TCHECK(dp[1]);
-               sa3->sa_mtime.nfsv3_sec = EXTRACT_32BITS(dp);
+               sa3->sa_mtime.nfsv3_sec = EXTRACT_BE_U_4(dp);
                dp++;
-               sa3->sa_mtime.nfsv3_nsec = EXTRACT_32BITS(dp);
+               sa3->sa_mtime.nfsv3_nsec = EXTRACT_BE_U_4(dp);
                dp++;
        }
 
@@ -295,15 +333,15 @@ nfsreply_print(netdissect_options *ndo,
        if (!ndo->ndo_nflag) {
                strlcpy(srcid, "nfs", sizeof(srcid));
                snprintf(dstid, sizeof(dstid), "%u",
-                   EXTRACT_32BITS(&rp->rm_xid));
+                   EXTRACT_BE_U_4(&rp->rm_xid));
        } else {
                snprintf(srcid, sizeof(srcid), "%u", NFS_PORT);
                snprintf(dstid, sizeof(dstid), "%u",
-                   EXTRACT_32BITS(&rp->rm_xid));
+                   EXTRACT_BE_U_4(&rp->rm_xid));
        }
        print_nfsaddr(ndo, bp2, srcid, dstid);
 
-       nfsreply_print_noaddr(ndo, bp, length, bp2);
+       nfsreply_noaddr_print(ndo, bp, length, bp2);
        return;
 
 trunc:
@@ -312,22 +350,22 @@ trunc:
 }
 
 void
-nfsreply_print_noaddr(netdissect_options *ndo,
+nfsreply_noaddr_print(netdissect_options *ndo,
                       register const u_char *bp, u_int length,
                       register const u_char *bp2)
 {
        register const struct sunrpc_msg *rp;
-       u_int32_t proc, vers, reply_stat;
+       uint32_t proc, vers, reply_stat;
        enum sunrpc_reject_stat rstat;
-       u_int32_t rlow;
-       u_int32_t rhigh;
+       uint32_t rlow;
+       uint32_t rhigh;
        enum sunrpc_auth_stat rwhy;
 
        nfserr = 0;             /* assume no error */
        rp = (const struct sunrpc_msg *)bp;
 
        ND_TCHECK(rp->rm_reply.rp_stat);
-       reply_stat = EXTRACT_32BITS(&rp->rm_reply.rp_stat);
+       reply_stat = EXTRACT_BE_U_4(&rp->rm_reply.rp_stat);
        switch (reply_stat) {
 
        case SUNRPC_MSG_ACCEPTED:
@@ -339,58 +377,20 @@ nfsreply_print_noaddr(netdissect_options *ndo,
        case SUNRPC_MSG_DENIED:
                ND_PRINT((ndo, "reply ERR %u: ", length));
                ND_TCHECK(rp->rm_reply.rp_reject.rj_stat);
-               rstat = EXTRACT_32BITS(&rp->rm_reply.rp_reject.rj_stat);
+               rstat = EXTRACT_BE_U_4(&rp->rm_reply.rp_reject.rj_stat);
                switch (rstat) {
 
                case SUNRPC_RPC_MISMATCH:
                        ND_TCHECK(rp->rm_reply.rp_reject.rj_vers.high);
-                       rlow = EXTRACT_32BITS(&rp->rm_reply.rp_reject.rj_vers.low);
-                       rhigh = EXTRACT_32BITS(&rp->rm_reply.rp_reject.rj_vers.high);
+                       rlow = EXTRACT_BE_U_4(&rp->rm_reply.rp_reject.rj_vers.low);
+                       rhigh = EXTRACT_BE_U_4(&rp->rm_reply.rp_reject.rj_vers.high);
                        ND_PRINT((ndo, "RPC Version mismatch (%u-%u)", rlow, rhigh));
                        break;
 
                case SUNRPC_AUTH_ERROR:
                        ND_TCHECK(rp->rm_reply.rp_reject.rj_why);
-                       rwhy = EXTRACT_32BITS(&rp->rm_reply.rp_reject.rj_why);
-                       ND_PRINT((ndo, "Auth "));
-                       switch (rwhy) {
-
-                       case SUNRPC_AUTH_OK:
-                               ND_PRINT((ndo, "OK"));
-                               break;
-
-                       case SUNRPC_AUTH_BADCRED:
-                               ND_PRINT((ndo, "Bogus Credentials (seal broken)"));
-                               break;
-
-                       case SUNRPC_AUTH_REJECTEDCRED:
-                               ND_PRINT((ndo, "Rejected Credentials (client should begin new session)"));
-                               break;
-
-                       case SUNRPC_AUTH_BADVERF:
-                               ND_PRINT((ndo, "Bogus Verifier (seal broken)"));
-                               break;
-
-                       case SUNRPC_AUTH_REJECTEDVERF:
-                               ND_PRINT((ndo, "Verifier expired or was replayed"));
-                               break;
-
-                       case SUNRPC_AUTH_TOOWEAK:
-                               ND_PRINT((ndo, "Credentials are too weak"));
-                               break;
-
-                       case SUNRPC_AUTH_INVALIDRESP:
-                               ND_PRINT((ndo, "Bogus response verifier"));
-                               break;
-
-                       case SUNRPC_AUTH_FAILED:
-                               ND_PRINT((ndo, "Unknown failure"));
-                               break;
-
-                       default:
-                               ND_PRINT((ndo, "Invalid failure code %u", (unsigned int)rwhy));
-                               break;
-                       }
+                       rwhy = EXTRACT_BE_U_4(&rp->rm_reply.rp_reject.rj_why);
+                       ND_PRINT((ndo, "Auth %s", tok2str(sunrpc_auth_str, "Invalid failure code %u", rwhy)));
                        break;
 
                default:
@@ -414,23 +414,23 @@ trunc:
  * Return a pointer to the first file handle in the packet.
  * If the packet was truncated, return 0.
  */
-static const u_int32_t *
+static const uint32_t *
 parsereq(netdissect_options *ndo,
          register const struct sunrpc_msg *rp, register u_int length)
 {
-       register const u_int32_t *dp;
+       register const uint32_t *dp;
        register u_int len;
 
        /*
         * find the start of the req data (if we captured it)
         */
-       dp = (u_int32_t *)&rp->rm_call.cb_cred;
+       dp = (const uint32_t *)&rp->rm_call.cb_cred;
        ND_TCHECK(dp[1]);
-       len = EXTRACT_32BITS(&dp[1]);
+       len = EXTRACT_BE_U_4(dp + 1);
        if (len < length) {
                dp += (len + (2 * sizeof(*dp) + 3)) / sizeof(*dp);
                ND_TCHECK(dp[1]);
-               len = EXTRACT_32BITS(&dp[1]);
+               len = EXTRACT_BE_U_4(dp + 1);
                if (len < length) {
                        dp += (len + (2 * sizeof(*dp) + 3)) / sizeof(*dp);
                        ND_TCHECK2(dp[0], 0);
@@ -445,15 +445,15 @@ trunc:
  * Print out an NFS file handle and return a pointer to following word.
  * If packet was truncated, return 0.
  */
-static const u_int32_t *
+static const uint32_t *
 parsefh(netdissect_options *ndo,
-        register const u_int32_t *dp, int v3)
+        register const uint32_t *dp, int v3)
 {
        u_int len;
 
        if (v3) {
                ND_TCHECK(dp[0]);
-               len = EXTRACT_32BITS(dp) / 4;
+               len = EXTRACT_BE_U_4(dp) / 4;
                dp++;
        } else
                len = NFSX_V2FH / 4;
@@ -470,11 +470,11 @@ trunc:
  * Print out a file name and return pointer to 32-bit word past it.
  * If packet was truncated, return 0.
  */
-static const u_int32_t *
+static const uint32_t *
 parsefn(netdissect_options *ndo,
-        register const u_int32_t *dp)
+        register const uint32_t *dp)
 {
-       register u_int32_t len;
+       register uint32_t len;
        register const u_char *cp;
 
        /* Bail if we don't have the string length */
@@ -486,11 +486,11 @@ parsefn(netdissect_options *ndo,
 
        ND_TCHECK2(*dp, ((len + 3) & ~3));
 
-       cp = (u_char *)dp;
+       cp = (const u_char *)dp;
        /* Update 32-bit pointer (NFS filenames padded to 32-bit boundaries) */
        dp += ((len + 3) & ~3) / sizeof(*dp);
        ND_PRINT((ndo, "\""));
-       if (fn_printn(cp, len, ndo->ndo_snapend)) {
+       if (fn_printn(ndo, cp, len, ndo->ndo_snapend)) {
                ND_PRINT((ndo, "\""));
                goto trunc;
        }
@@ -506,9 +506,9 @@ trunc:
  * Return pointer to 32-bit word past file name.
  * If packet was truncated (or there was some other error), return 0.
  */
-static const u_int32_t *
+static const uint32_t *
 parsefhn(netdissect_options *ndo,
-         register const u_int32_t *dp, int v3)
+         register const uint32_t *dp, int v3)
 {
        dp = parsefh(ndo, dp, v3);
        if (dp == NULL)
@@ -518,47 +518,16 @@ parsefhn(netdissect_options *ndo,
 }
 
 void
-nfsreq_print(netdissect_options *ndo,
-             register const u_char *bp, u_int length,
-             register const u_char *bp2)
-{
-       register const struct sunrpc_msg *rp;
-       char srcid[20], dstid[20];      /*fits 32bit*/
-
-       nfserr = 0;             /* assume no error */
-       rp = (const struct sunrpc_msg *)bp;
-
-       ND_TCHECK(rp->rm_xid);
-       if (!ndo->ndo_nflag) {
-               snprintf(srcid, sizeof(srcid), "%u",
-                   EXTRACT_32BITS(&rp->rm_xid));
-               strlcpy(dstid, "nfs", sizeof(dstid));
-       } else {
-               snprintf(srcid, sizeof(srcid), "%u",
-                   EXTRACT_32BITS(&rp->rm_xid));
-               snprintf(dstid, sizeof(dstid), "%u", NFS_PORT);
-       }
-       print_nfsaddr(ndo, bp2, srcid, dstid);
-
-       nfsreq_print_noaddr(ndo, bp, length, bp2);
-       return;
-
-trunc:
-       if (!nfserr)
-               ND_PRINT((ndo, "%s", tstr));
-}
-
-void
-nfsreq_print_noaddr(netdissect_options *ndo,
+nfsreq_noaddr_print(netdissect_options *ndo,
                     register const u_char *bp, u_int length,
                     register const u_char *bp2)
 {
        register const struct sunrpc_msg *rp;
-       register const u_int32_t *dp;
+       register const uint32_t *dp;
        nfs_type type;
        int v3;
-       u_int32_t proc;
-       u_int32_t access_flags;
+       uint32_t proc;
+       uint32_t access_flags;
        struct nfsv3_sattr sa3;
 
        ND_PRINT((ndo, "%d", length));
@@ -568,47 +537,41 @@ nfsreq_print_noaddr(netdissect_options *ndo,
        if (!xid_map_enter(ndo, rp, bp2))       /* record proc number for later on */
                goto trunc;
 
-       v3 = (EXTRACT_32BITS(&rp->rm_call.cb_vers) == NFS_VER3);
-       proc = EXTRACT_32BITS(&rp->rm_call.cb_proc);
+       v3 = (EXTRACT_BE_U_4(&rp->rm_call.cb_vers) == NFS_VER3);
+       proc = EXTRACT_BE_U_4(&rp->rm_call.cb_proc);
 
        if (!v3 && proc < NFS_NPROCS)
                proc =  nfsv3_procid[proc];
 
+       ND_PRINT((ndo, " %s", tok2str(nfsproc_str, "proc-%u", proc)));
        switch (proc) {
-       case NFSPROC_NOOP:
-               ND_PRINT((ndo, " nop"));
-               return;
-       case NFSPROC_NULL:
-               ND_PRINT((ndo, " null"));
-               return;
 
        case NFSPROC_GETATTR:
-               ND_PRINT((ndo, " getattr"));
-               if ((dp = parsereq(ndo, rp, length)) != NULL &&
-                   parsefh(ndo, dp, v3) != NULL)
-                       return;
-               break;
-
        case NFSPROC_SETATTR:
-               ND_PRINT((ndo, " setattr"));
+       case NFSPROC_READLINK:
+       case NFSPROC_FSSTAT:
+       case NFSPROC_FSINFO:
+       case NFSPROC_PATHCONF:
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    parsefh(ndo, dp, v3) != NULL)
                        return;
                break;
 
        case NFSPROC_LOOKUP:
-               ND_PRINT((ndo, " lookup"));
+       case NFSPROC_CREATE:
+       case NFSPROC_MKDIR:
+       case NFSPROC_REMOVE:
+       case NFSPROC_RMDIR:
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    parsefhn(ndo, dp, v3) != NULL)
                        return;
                break;
 
        case NFSPROC_ACCESS:
-               ND_PRINT((ndo, " access"));
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    (dp = parsefh(ndo, dp, v3)) != NULL) {
                        ND_TCHECK(dp[0]);
-                       access_flags = EXTRACT_32BITS(&dp[0]);
+                       access_flags = EXTRACT_BE_U_4(dp);
                        if (access_flags & ~NFSV3ACCESS_FULL) {
                                /* NFSV3ACCESS definitions aren't up to date */
                                ND_PRINT((ndo, " %04x", access_flags));
@@ -643,82 +606,57 @@ nfsreq_print_noaddr(netdissect_options *ndo,
                }
                break;
 
-       case NFSPROC_READLINK:
-               ND_PRINT((ndo, " readlink"));
-               if ((dp = parsereq(ndo, rp, length)) != NULL &&
-                   parsefh(ndo, dp, v3) != NULL)
-                       return;
-               break;
-
        case NFSPROC_READ:
-               ND_PRINT((ndo, " read"));
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    (dp = parsefh(ndo, dp, v3)) != NULL) {
                        if (v3) {
                                ND_TCHECK(dp[2]);
                                ND_PRINT((ndo, " %u bytes @ %" PRIu64,
-                                      EXTRACT_32BITS(&dp[2]),
-                                      EXTRACT_64BITS(&dp[0])));
+                                      EXTRACT_BE_U_4(dp + 2),
+                                      EXTRACT_BE_U_8(dp)));
                        } else {
                                ND_TCHECK(dp[1]);
                                ND_PRINT((ndo, " %u bytes @ %u",
-                                   EXTRACT_32BITS(&dp[1]),
-                                   EXTRACT_32BITS(&dp[0])));
+                                   EXTRACT_BE_U_4(dp + 1),
+                                   EXTRACT_BE_U_4(dp)));
                        }
                        return;
                }
                break;
 
        case NFSPROC_WRITE:
-               ND_PRINT((ndo, " write"));
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    (dp = parsefh(ndo, dp, v3)) != NULL) {
                        if (v3) {
-                               ND_TCHECK(dp[2]);
+                               ND_TCHECK(dp[4]);
                                ND_PRINT((ndo, " %u (%u) bytes @ %" PRIu64,
-                                               EXTRACT_32BITS(&dp[4]),
-                                               EXTRACT_32BITS(&dp[2]),
-                                               EXTRACT_64BITS(&dp[0])));
+                                               EXTRACT_BE_U_4(dp + 4),
+                                               EXTRACT_BE_U_4(dp + 2),
+                                               EXTRACT_BE_U_8(dp)));
                                if (ndo->ndo_vflag) {
-                                       dp += 3;
-                                       ND_TCHECK(dp[0]);
                                        ND_PRINT((ndo, " <%s>",
                                                tok2str(nfsv3_writemodes,
-                                                       NULL, EXTRACT_32BITS(dp))));
+                                                       NULL, EXTRACT_BE_U_4(dp + 3))));
                                }
                        } else {
                                ND_TCHECK(dp[3]);
                                ND_PRINT((ndo, " %u (%u) bytes @ %u (%u)",
-                                               EXTRACT_32BITS(&dp[3]),
-                                               EXTRACT_32BITS(&dp[2]),
-                                               EXTRACT_32BITS(&dp[1]),
-                                               EXTRACT_32BITS(&dp[0])));
+                                               EXTRACT_BE_U_4(dp + 3),
+                                               EXTRACT_BE_U_4(dp + 2),
+                                               EXTRACT_BE_U_4(dp + 1),
+                                               EXTRACT_BE_U_4(dp)));
                        }
                        return;
                }
                break;
 
-       case NFSPROC_CREATE:
-               ND_PRINT((ndo, " create"));
-               if ((dp = parsereq(ndo, rp, length)) != NULL &&
-                   parsefhn(ndo, dp, v3) != NULL)
-                       return;
-               break;
-
-       case NFSPROC_MKDIR:
-               ND_PRINT((ndo, " mkdir"));
-               if ((dp = parsereq(ndo, rp, length)) != 0 && parsefhn(ndo, dp, v3) != 0)
-                       return;
-               break;
-
        case NFSPROC_SYMLINK:
-               ND_PRINT((ndo, " symlink"));
-               if ((dp = parsereq(ndo, rp, length)) != 0 &&
-                   (dp = parsefhn(ndo, dp, v3)) != 0) {
+               if ((dp = parsereq(ndo, rp, length)) != NULL &&
+                   (dp = parsefhn(ndo, dp, v3)) != NULL) {
                        ND_PRINT((ndo, " ->"));
-                       if (v3 && (dp = parse_sattr3(ndo, dp, &sa3)) == 0)
+                       if (v3 && (dp = parse_sattr3(ndo, dp, &sa3)) == NULL)
                                break;
-                       if (parsefn(ndo, dp) == 0)
+                       if (parsefn(ndo, dp) == NULL)
                                break;
                        if (v3 && ndo->ndo_vflag)
                                print_sattr3(ndo, &sa3, ndo->ndo_vflag);
@@ -727,20 +665,19 @@ nfsreq_print_noaddr(netdissect_options *ndo,
                break;
 
        case NFSPROC_MKNOD:
-               ND_PRINT((ndo, " mknod"));
-               if ((dp = parsereq(ndo, rp, length)) != 0 &&
-                   (dp = parsefhn(ndo, dp, v3)) != 0) {
+               if ((dp = parsereq(ndo, rp, length)) != NULL &&
+                   (dp = parsefhn(ndo, dp, v3)) != NULL) {
                        ND_TCHECK(*dp);
-                       type = (nfs_type)EXTRACT_32BITS(dp);
+                       type = (nfs_type) EXTRACT_BE_U_4(dp);
                        dp++;
-                       if ((dp = parse_sattr3(ndo, dp, &sa3)) == 0)
+                       if ((dp = parse_sattr3(ndo, dp, &sa3)) == NULL)
                                break;
                        ND_PRINT((ndo, " %s", tok2str(type2str, "unk-ft %d", type)));
                        if (ndo->ndo_vflag && (type == NFCHR || type == NFBLK)) {
                                ND_TCHECK(dp[1]);
                                ND_PRINT((ndo, " %u/%u",
-                                      EXTRACT_32BITS(&dp[0]),
-                                      EXTRACT_32BITS(&dp[1])));
+                                      EXTRACT_BE_U_4(dp),
+                                      EXTRACT_BE_U_4(dp + 1)));
                                dp += 2;
                        }
                        if (ndo->ndo_vflag)
@@ -749,22 +686,7 @@ nfsreq_print_noaddr(netdissect_options *ndo,
                }
                break;
 
-       case NFSPROC_REMOVE:
-               ND_PRINT((ndo, " remove"));
-               if ((dp = parsereq(ndo, rp, length)) != NULL &&
-                   parsefhn(ndo, dp, v3) != NULL)
-                       return;
-               break;
-
-       case NFSPROC_RMDIR:
-               ND_PRINT((ndo, " rmdir"));
-               if ((dp = parsereq(ndo, rp, length)) != NULL &&
-                   parsefhn(ndo, dp, v3) != NULL)
-                       return;
-               break;
-
        case NFSPROC_RENAME:
-               ND_PRINT((ndo, " rename"));
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    (dp = parsefhn(ndo, dp, v3)) != NULL) {
                        ND_PRINT((ndo, " ->"));
@@ -774,7 +696,6 @@ nfsreq_print_noaddr(netdissect_options *ndo,
                break;
 
        case NFSPROC_LINK:
-               ND_PRINT((ndo, " link"));
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    (dp = parsefh(ndo, dp, v3)) != NULL) {
                        ND_PRINT((ndo, " ->"));
@@ -784,7 +705,6 @@ nfsreq_print_noaddr(netdissect_options *ndo,
                break;
 
        case NFSPROC_READDIR:
-               ND_PRINT((ndo, " readdir"));
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    (dp = parsefh(ndo, dp, v3)) != NULL) {
                        if (v3) {
@@ -794,8 +714,8 @@ nfsreq_print_noaddr(netdissect_options *ndo,
                                 * offset cookie here.
                                 */
                                ND_PRINT((ndo, " %u bytes @ %" PRId64,
-                                   EXTRACT_32BITS(&dp[4]),
-                                   EXTRACT_64BITS(&dp[0])));
+                                   EXTRACT_BE_U_4(dp + 4),
+                                   EXTRACT_BE_U_8(dp)));
                                if (ndo->ndo_vflag)
                                        ND_PRINT((ndo, " verf %08x%08x", dp[2], dp[3]));
                        } else {
@@ -805,15 +725,14 @@ nfsreq_print_noaddr(netdissect_options *ndo,
                                 * common, but offsets > 2^31 aren't.
                                 */
                                ND_PRINT((ndo, " %u bytes @ %d",
-                                   EXTRACT_32BITS(&dp[1]),
-                                   EXTRACT_32BITS(&dp[0])));
+                                   EXTRACT_BE_U_4(dp + 1),
+                                   EXTRACT_BE_U_4(dp)));
                        }
                        return;
                }
                break;
 
        case NFSPROC_READDIRPLUS:
-               ND_PRINT((ndo, " readdirplus"));
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    (dp = parsefh(ndo, dp, v3)) != NULL) {
                        ND_TCHECK(dp[4]);
@@ -822,52 +741,29 @@ nfsreq_print_noaddr(netdissect_options *ndo,
                         * cookie here.
                         */
                        ND_PRINT((ndo, " %u bytes @ %" PRId64,
-                               EXTRACT_32BITS(&dp[4]),
-                               EXTRACT_64BITS(&dp[0])));
+                               EXTRACT_BE_U_4(dp + 4),
+                               EXTRACT_BE_U_8(dp)));
                        if (ndo->ndo_vflag) {
                                ND_TCHECK(dp[5]);
                                ND_PRINT((ndo, " max %u verf %08x%08x",
-                                      EXTRACT_32BITS(&dp[5]), dp[2], dp[3]));
+                                      EXTRACT_BE_U_4(dp + 5), dp[2], dp[3]));
                        }
                        return;
                }
                break;
 
-       case NFSPROC_FSSTAT:
-               ND_PRINT((ndo, " fsstat"));
-               if ((dp = parsereq(ndo, rp, length)) != NULL &&
-                   parsefh(ndo, dp, v3) != NULL)
-                       return;
-               break;
-
-       case NFSPROC_FSINFO:
-               ND_PRINT((ndo, " fsinfo"));
-               if ((dp = parsereq(ndo, rp, length)) != NULL &&
-                   parsefh(ndo, dp, v3) != NULL)
-                       return;
-               break;
-
-       case NFSPROC_PATHCONF:
-               ND_PRINT((ndo, " pathconf"));
-               if ((dp = parsereq(ndo, rp, length)) != NULL &&
-                   parsefh(ndo, dp, v3) != NULL)
-                       return;
-               break;
-
        case NFSPROC_COMMIT:
-               ND_PRINT((ndo, " commit"));
                if ((dp = parsereq(ndo, rp, length)) != NULL &&
                    (dp = parsefh(ndo, dp, v3)) != NULL) {
                        ND_TCHECK(dp[2]);
                        ND_PRINT((ndo, " %u bytes @ %" PRIu64,
-                               EXTRACT_32BITS(&dp[2]),
-                               EXTRACT_64BITS(&dp[0])));
+                               EXTRACT_BE_U_4(dp + 2),
+                               EXTRACT_BE_U_8(dp)));
                        return;
                }
                break;
 
        default:
-               ND_PRINT((ndo, " proc-%u", EXTRACT_32BITS(&rp->rm_call.cb_proc)));
                return;
        }
 
@@ -887,10 +783,10 @@ trunc:
  */
 static void
 nfs_printfh(netdissect_options *ndo,
-            register const u_int32_t *dp, const u_int len)
+            register const uint32_t *dp, const u_int len)
 {
        my_fsid fsid;
-       u_int32_t ino;
+       uint32_t ino;
        const char *sfsname = NULL;
        char *spacep;
 
@@ -911,11 +807,15 @@ nfs_printfh(netdissect_options *ndo,
 
        if (sfsname) {
                /* file system ID is ASCII, not numeric, for this server OS */
-               static char temp[NFSX_V3FHMAX+1];
+               char temp[NFSX_V3FHMAX+1];
+               u_int stringlen;
 
                /* Make sure string is null-terminated */
-               strncpy(temp, sfsname, NFSX_V3FHMAX);
-               temp[sizeof(temp) - 1] = '\0';
+               stringlen = len;
+               if (stringlen > NFSX_V3FHMAX)
+                       stringlen = NFSX_V3FHMAX;
+               strncpy(temp, sfsname, stringlen);
+               temp[stringlen] = '\0';
                /* Remove trailing spaces */
                spacep = strchr(temp, ' ');
                if (spacep)
@@ -941,17 +841,12 @@ nfs_printfh(netdissect_options *ndo,
  */
 
 struct xid_map_entry {
-       u_int32_t       xid;            /* transaction ID (net order) */
+       uint32_t        xid;            /* transaction ID (net order) */
        int ipver;                      /* IP version (4 or 6) */
-#ifdef INET6
        struct in6_addr client;         /* client IP address (net order) */
        struct in6_addr server;         /* server IP address (net order) */
-#else
-       struct in_addr  client;         /* client IP address (net order) */
-       struct in_addr  server;         /* server IP address (net order) */
-#endif
-       u_int32_t       proc;           /* call proc number (host order) */
-       u_int32_t       vers;           /* program version (host order) */
+       uint32_t        proc;           /* call proc number (host order) */
+       uint32_t        vers;           /* program version (host order) */
 };
 
 /*
@@ -962,32 +857,28 @@ struct xid_map_entry {
 
 #define        XIDMAPSIZE      64
 
-struct xid_map_entry xid_map[XIDMAPSIZE];
+static struct xid_map_entry xid_map[XIDMAPSIZE];
 
-int    xid_map_next = 0;
-int    xid_map_hint = 0;
+static int xid_map_next = 0;
+static int xid_map_hint = 0;
 
 static int
 xid_map_enter(netdissect_options *ndo,
               const struct sunrpc_msg *rp, const u_char *bp)
 {
-       struct ip *ip = NULL;
-#ifdef INET6
-       struct ip6_hdr *ip6 = NULL;
-#endif
+       const struct ip *ip = NULL;
+       const struct ip6_hdr *ip6 = NULL;
        struct xid_map_entry *xmep;
 
-       if (!ND_TTEST(rp->rm_call.cb_vers))
+       if (!ND_TTEST(rp->rm_call.cb_proc))
                return (0);
-       switch (IP_V((struct ip *)bp)) {
+       switch (IP_V((const struct ip *)bp)) {
        case 4:
-               ip = (struct ip *)bp;
+               ip = (const struct ip *)bp;
                break;
-#ifdef INET6
        case 6:
-               ip6 = (struct ip6_hdr *)bp;
+               ip6 = (const struct ip6_hdr *)bp;
                break;
-#endif
        default:
                return (1);
        }
@@ -997,21 +888,19 @@ xid_map_enter(netdissect_options *ndo,
        if (++xid_map_next >= XIDMAPSIZE)
                xid_map_next = 0;
 
-       xmep->xid = rp->rm_xid;
+       UNALIGNED_MEMCPY(&xmep->xid, &rp->rm_xid, sizeof(xmep->xid));
        if (ip) {
                xmep->ipver = 4;
                UNALIGNED_MEMCPY(&xmep->client, &ip->ip_src, sizeof(ip->ip_src));
                UNALIGNED_MEMCPY(&xmep->server, &ip->ip_dst, sizeof(ip->ip_dst));
        }
-#ifdef INET6
        else if (ip6) {
                xmep->ipver = 6;
                UNALIGNED_MEMCPY(&xmep->client, &ip6->ip6_src, sizeof(ip6->ip6_src));
                UNALIGNED_MEMCPY(&xmep->server, &ip6->ip6_dst, sizeof(ip6->ip6_dst));
        }
-#endif
-       xmep->proc = EXTRACT_32BITS(&rp->rm_call.cb_proc);
-       xmep->vers = EXTRACT_32BITS(&rp->rm_call.cb_vers);
+       xmep->proc = EXTRACT_BE_U_4(&rp->rm_call.cb_proc);
+       xmep->vers = EXTRACT_BE_U_4(&rp->rm_call.cb_vers);
        return (1);
 }
 
@@ -1020,18 +909,17 @@ xid_map_enter(netdissect_options *ndo,
  * version in vers return, or returns -1 on failure
  */
 static int
-xid_map_find(const struct sunrpc_msg *rp, const u_char *bp, u_int32_t *proc,
-            u_int32_t *vers)
+xid_map_find(const struct sunrpc_msg *rp, const u_char *bp, uint32_t *proc,
+            uint32_t *vers)
 {
        int i;
        struct xid_map_entry *xmep;
-       u_int32_t xid = rp->rm_xid;
-       struct ip *ip = (struct ip *)bp;
-#ifdef INET6
-       struct ip6_hdr *ip6 = (struct ip6_hdr *)bp;
-#endif
+       uint32_t xid;
+       const struct ip *ip = (const struct ip *)bp;
+       const struct ip6_hdr *ip6 = (const struct ip6_hdr *)bp;
        int cmp;
 
+       UNALIGNED_MEMCPY(&xid, &rp->rm_xid, sizeof(xmep->xid));
        /* Start searching from where we last left off */
        i = xid_map_hint;
        do {
@@ -1048,7 +936,6 @@ xid_map_find(const struct sunrpc_msg *rp, const u_char *bp, u_int32_t *proc,
                                cmp = 0;
                        }
                        break;
-#ifdef INET6
                case 6:
                        if (UNALIGNED_MEMCMP(&ip6->ip6_src, &xmep->server,
                                   sizeof(ip6->ip6_src)) != 0 ||
@@ -1057,7 +944,6 @@ xid_map_find(const struct sunrpc_msg *rp, const u_char *bp, u_int32_t *proc,
                                cmp = 0;
                        }
                        break;
-#endif
                default:
                        cmp = 0;
                        break;
@@ -1086,11 +972,11 @@ xid_map_find(const struct sunrpc_msg *rp, const u_char *bp, u_int32_t *proc,
  * Return a pointer to the beginning of the actual results.
  * If the packet was truncated, return 0.
  */
-static const u_int32_t *
+static const uint32_t *
 parserep(netdissect_options *ndo,
          register const struct sunrpc_msg *rp, register u_int length)
 {
-       register const u_int32_t *dp;
+       register const uint32_t *dp;
        u_int len;
        enum sunrpc_accept_stat astat;
 
@@ -1098,7 +984,7 @@ parserep(netdissect_options *ndo,
         * Portability note:
         * Here we find the address of the ar_verf credentials.
         * Originally, this calculation was
-        *      dp = (u_int32_t *)&rp->rm_reply.rp_acpt.ar_verf
+        *      dp = (uint32_t *)&rp->rm_reply.rp_acpt.ar_verf
         * On the wire, the rp_acpt field starts immediately after
         * the (32 bit) rp_stat field.  However, rp_acpt (which is a
         * "struct accepted_reply") contains a "struct opaque_auth",
@@ -1109,72 +995,42 @@ parserep(netdissect_options *ndo,
         * representation.  Instead, we skip past the rp_stat field,
         * which is an "enum" and so occupies one 32-bit word.
         */
-       dp = ((const u_int32_t *)&rp->rm_reply) + 1;
+       dp = ((const uint32_t *)&rp->rm_reply) + 1;
        ND_TCHECK(dp[1]);
-       len = EXTRACT_32BITS(&dp[1]);
+       len = EXTRACT_BE_U_4(dp + 1);
        if (len >= length)
                return (NULL);
        /*
         * skip past the ar_verf credentials.
         */
-       dp += (len + (2*sizeof(u_int32_t) + 3)) / sizeof(u_int32_t);
-       ND_TCHECK2(dp[0], 0);
+       dp += (len + (2*sizeof(uint32_t) + 3)) / sizeof(uint32_t);
 
        /*
         * now we can check the ar_stat field
         */
-       astat = (enum sunrpc_accept_stat) EXTRACT_32BITS(dp);
-       switch (astat) {
-
-       case SUNRPC_SUCCESS:
-               break;
-
-       case SUNRPC_PROG_UNAVAIL:
-               ND_PRINT((ndo, " PROG_UNAVAIL"));
-               nfserr = 1;             /* suppress trunc string */
-               return (NULL);
-
-       case SUNRPC_PROG_MISMATCH:
-               ND_PRINT((ndo, " PROG_MISMATCH"));
-               nfserr = 1;             /* suppress trunc string */
-               return (NULL);
-
-       case SUNRPC_PROC_UNAVAIL:
-               ND_PRINT((ndo, " PROC_UNAVAIL"));
-               nfserr = 1;             /* suppress trunc string */
-               return (NULL);
-
-       case SUNRPC_GARBAGE_ARGS:
-               ND_PRINT((ndo, " GARBAGE_ARGS"));
-               nfserr = 1;             /* suppress trunc string */
-               return (NULL);
-
-       case SUNRPC_SYSTEM_ERR:
-               ND_PRINT((ndo, " SYSTEM_ERR"));
-               nfserr = 1;             /* suppress trunc string */
-               return (NULL);
-
-       default:
-               ND_PRINT((ndo, " ar_stat %d", astat));
+       ND_TCHECK(dp[0]);
+       astat = (enum sunrpc_accept_stat) EXTRACT_BE_U_4(dp);
+       if (astat != SUNRPC_SUCCESS) {
+               ND_PRINT((ndo, " %s", tok2str(sunrpc_str, "ar_stat %d", astat)));
                nfserr = 1;             /* suppress trunc string */
                return (NULL);
        }
        /* successful return */
        ND_TCHECK2(*dp, sizeof(astat));
-       return ((u_int32_t *) (sizeof(astat) + ((char *)dp)));
+       return ((const uint32_t *) (sizeof(astat) + ((const char *)dp)));
 trunc:
        return (0);
 }
 
-static const u_int32_t *
+static const uint32_t *
 parsestatus(netdissect_options *ndo,
-            const u_int32_t *dp, int *er)
+            const uint32_t *dp, int *er)
 {
        int errnum;
 
        ND_TCHECK(dp[0]);
 
-       errnum = EXTRACT_32BITS(&dp[0]);
+       errnum = EXTRACT_BE_U_4(dp);
        if (er)
                *er = errnum;
        if (errnum != 0) {
@@ -1188,9 +1044,9 @@ trunc:
        return NULL;
 }
 
-static const u_int32_t *
+static const uint32_t *
 parsefattr(netdissect_options *ndo,
-           const u_int32_t *dp, int verbose, int v3)
+           const uint32_t *dp, int verbose, int v3)
 {
        const struct nfs_fattr *fap;
 
@@ -1199,17 +1055,17 @@ parsefattr(netdissect_options *ndo,
        if (verbose) {
                ND_PRINT((ndo, " %s %o ids %d/%d",
                    tok2str(type2str, "unk-ft %d ",
-                   EXTRACT_32BITS(&fap->fa_type)),
-                   EXTRACT_32BITS(&fap->fa_mode),
-                   EXTRACT_32BITS(&fap->fa_uid),
-                   EXTRACT_32BITS(&fap->fa_gid)));
+                   EXTRACT_BE_U_4(&fap->fa_type)),
+                   EXTRACT_BE_U_4(&fap->fa_mode),
+                   EXTRACT_BE_U_4(&fap->fa_uid),
+                   EXTRACT_BE_U_4(&fap->fa_gid)));
                if (v3) {
                        ND_TCHECK(fap->fa3_size);
                        ND_PRINT((ndo, " sz %" PRIu64,
-                               EXTRACT_64BITS((u_int32_t *)&fap->fa3_size)));
+                               EXTRACT_BE_U_8((const uint32_t *)&fap->fa3_size)));
                } else {
                        ND_TCHECK(fap->fa2_size);
-                       ND_PRINT((ndo, " sz %d", EXTRACT_32BITS(&fap->fa2_size)));
+                       ND_PRINT((ndo, " sz %d", EXTRACT_BE_U_4(&fap->fa2_size)));
                }
        }
        /* print lots more stuff */
@@ -1217,41 +1073,41 @@ parsefattr(netdissect_options *ndo,
                if (v3) {
                        ND_TCHECK(fap->fa3_ctime);
                        ND_PRINT((ndo, " nlink %d rdev %d/%d",
-                              EXTRACT_32BITS(&fap->fa_nlink),
-                              EXTRACT_32BITS(&fap->fa3_rdev.specdata1),
-                              EXTRACT_32BITS(&fap->fa3_rdev.specdata2)));
+                              EXTRACT_BE_U_4(&fap->fa_nlink),
+                              EXTRACT_BE_U_4(&fap->fa3_rdev.specdata1),
+                              EXTRACT_BE_U_4(&fap->fa3_rdev.specdata2)));
                        ND_PRINT((ndo, " fsid %" PRIx64,
-                               EXTRACT_64BITS((u_int32_t *)&fap->fa3_fsid)));
+                               EXTRACT_BE_U_8((const uint32_t *)&fap->fa3_fsid)));
                        ND_PRINT((ndo, " fileid %" PRIx64,
-                               EXTRACT_64BITS((u_int32_t *)&fap->fa3_fileid)));
+                               EXTRACT_BE_U_8((const uint32_t *)&fap->fa3_fileid)));
                        ND_PRINT((ndo, " a/m/ctime %u.%06u",
-                              EXTRACT_32BITS(&fap->fa3_atime.nfsv3_sec),
-                              EXTRACT_32BITS(&fap->fa3_atime.nfsv3_nsec)));
+                              EXTRACT_BE_U_4(&fap->fa3_atime.nfsv3_sec),
+                              EXTRACT_BE_U_4(&fap->fa3_atime.nfsv3_nsec)));
                        ND_PRINT((ndo, " %u.%06u",
-                              EXTRACT_32BITS(&fap->fa3_mtime.nfsv3_sec),
-                              EXTRACT_32BITS(&fap->fa3_mtime.nfsv3_nsec)));
+                              EXTRACT_BE_U_4(&fap->fa3_mtime.nfsv3_sec),
+                              EXTRACT_BE_U_4(&fap->fa3_mtime.nfsv3_nsec)));
                        ND_PRINT((ndo, " %u.%06u",
-                              EXTRACT_32BITS(&fap->fa3_ctime.nfsv3_sec),
-                              EXTRACT_32BITS(&fap->fa3_ctime.nfsv3_nsec)));
+                              EXTRACT_BE_U_4(&fap->fa3_ctime.nfsv3_sec),
+                              EXTRACT_BE_U_4(&fap->fa3_ctime.nfsv3_nsec)));
                } else {
                        ND_TCHECK(fap->fa2_ctime);
-                       ND_PRINT((ndo, " nlink %d rdev %x fsid %x nodeid %x a/m/ctime",
-                              EXTRACT_32BITS(&fap->fa_nlink),
-                              EXTRACT_32BITS(&fap->fa2_rdev),
-                              EXTRACT_32BITS(&fap->fa2_fsid),
-                              EXTRACT_32BITS(&fap->fa2_fileid)));
+                       ND_PRINT((ndo, " nlink %d rdev 0x%x fsid 0x%x nodeid 0x%x a/m/ctime",
+                              EXTRACT_BE_U_4(&fap->fa_nlink),
+                              EXTRACT_BE_U_4(&fap->fa2_rdev),
+                              EXTRACT_BE_U_4(&fap->fa2_fsid),
+                              EXTRACT_BE_U_4(&fap->fa2_fileid)));
                        ND_PRINT((ndo, " %u.%06u",
-                              EXTRACT_32BITS(&fap->fa2_atime.nfsv2_sec),
-                              EXTRACT_32BITS(&fap->fa2_atime.nfsv2_usec)));
+                              EXTRACT_BE_U_4(&fap->fa2_atime.nfsv2_sec),
+                              EXTRACT_BE_U_4(&fap->fa2_atime.nfsv2_usec)));
                        ND_PRINT((ndo, " %u.%06u",
-                              EXTRACT_32BITS(&fap->fa2_mtime.nfsv2_sec),
-                              EXTRACT_32BITS(&fap->fa2_mtime.nfsv2_usec)));
+                              EXTRACT_BE_U_4(&fap->fa2_mtime.nfsv2_sec),
+                              EXTRACT_BE_U_4(&fap->fa2_mtime.nfsv2_usec)));
                        ND_PRINT((ndo, " %u.%06u",
-                              EXTRACT_32BITS(&fap->fa2_ctime.nfsv2_sec),
-                              EXTRACT_32BITS(&fap->fa2_ctime.nfsv2_usec)));
+                              EXTRACT_BE_U_4(&fap->fa2_ctime.nfsv2_sec),
+                              EXTRACT_BE_U_4(&fap->fa2_ctime.nfsv2_usec)));
                }
        }
-       return ((const u_int32_t *)((unsigned char *)dp +
+       return ((const uint32_t *)((const unsigned char *)dp +
                (v3 ? NFSX_V3FATTR : NFSX_V2FATTR)));
 trunc:
        return (NULL);
@@ -1259,7 +1115,7 @@ trunc:
 
 static int
 parseattrstat(netdissect_options *ndo,
-              const u_int32_t *dp, int verbose, int v3)
+              const uint32_t *dp, int verbose, int v3)
 {
        int er;
 
@@ -1274,7 +1130,7 @@ parseattrstat(netdissect_options *ndo,
 
 static int
 parsediropres(netdissect_options *ndo,
-              const u_int32_t *dp)
+              const uint32_t *dp)
 {
        int er;
 
@@ -1292,7 +1148,7 @@ parsediropres(netdissect_options *ndo,
 
 static int
 parselinkres(netdissect_options *ndo,
-             const u_int32_t *dp, int v3)
+             const uint32_t *dp, int v3)
 {
        int er;
 
@@ -1309,7 +1165,7 @@ parselinkres(netdissect_options *ndo,
 
 static int
 parsestatfs(netdissect_options *ndo,
-            const u_int32_t *dp, int v3)
+            const uint32_t *dp, int v3)
 {
        const struct nfs_statfs *sfsp;
        int er;
@@ -1336,23 +1192,23 @@ parsestatfs(netdissect_options *ndo,
 
        if (v3) {
                ND_PRINT((ndo, " tbytes %" PRIu64 " fbytes %" PRIu64 " abytes %" PRIu64,
-                       EXTRACT_64BITS((u_int32_t *)&sfsp->sf_tbytes),
-                       EXTRACT_64BITS((u_int32_t *)&sfsp->sf_fbytes),
-                       EXTRACT_64BITS((u_int32_t *)&sfsp->sf_abytes)));
+                       EXTRACT_BE_U_8((const uint32_t *)&sfsp->sf_tbytes),
+                       EXTRACT_BE_U_8((const uint32_t *)&sfsp->sf_fbytes),
+                       EXTRACT_BE_U_8((const uint32_t *)&sfsp->sf_abytes)));
                if (ndo->ndo_vflag) {
                        ND_PRINT((ndo, " tfiles %" PRIu64 " ffiles %" PRIu64 " afiles %" PRIu64 " invar %u",
-                              EXTRACT_64BITS((u_int32_t *)&sfsp->sf_tfiles),
-                              EXTRACT_64BITS((u_int32_t *)&sfsp->sf_ffiles),
-                              EXTRACT_64BITS((u_int32_t *)&sfsp->sf_afiles),
-                              EXTRACT_32BITS(&sfsp->sf_invarsec)));
+                              EXTRACT_BE_U_8((const uint32_t *)&sfsp->sf_tfiles),
+                              EXTRACT_BE_U_8((const uint32_t *)&sfsp->sf_ffiles),
+                              EXTRACT_BE_U_8((const uint32_t *)&sfsp->sf_afiles),
+                              EXTRACT_BE_U_4(&sfsp->sf_invarsec)));
                }
        } else {
                ND_PRINT((ndo, " tsize %d bsize %d blocks %d bfree %d bavail %d",
-                       EXTRACT_32BITS(&sfsp->sf_tsize),
-                       EXTRACT_32BITS(&sfsp->sf_bsize),
-                       EXTRACT_32BITS(&sfsp->sf_blocks),
-                       EXTRACT_32BITS(&sfsp->sf_bfree),
-                       EXTRACT_32BITS(&sfsp->sf_bavail)));
+                       EXTRACT_BE_U_4(&sfsp->sf_tsize),
+                       EXTRACT_BE_U_4(&sfsp->sf_bsize),
+                       EXTRACT_BE_U_4(&sfsp->sf_blocks),
+                       EXTRACT_BE_U_4(&sfsp->sf_bfree),
+                       EXTRACT_BE_U_4(&sfsp->sf_bavail)));
        }
 
        return (1);
@@ -1362,7 +1218,7 @@ trunc:
 
 static int
 parserddires(netdissect_options *ndo,
-             const u_int32_t *dp)
+             const uint32_t *dp)
 {
        int er;
 
@@ -1375,8 +1231,8 @@ parserddires(netdissect_options *ndo,
                return (1);
 
        ND_TCHECK(dp[2]);
-       ND_PRINT((ndo, " offset %x size %d ",
-              EXTRACT_32BITS(&dp[0]), EXTRACT_32BITS(&dp[1])));
+       ND_PRINT((ndo, " offset 0x%x size %d ",
+              EXTRACT_BE_U_4(dp), EXTRACT_BE_U_4(dp + 1)));
        if (dp[2] != 0)
                ND_PRINT((ndo, " eof"));
 
@@ -1385,26 +1241,27 @@ trunc:
        return (0);
 }
 
-static const u_int32_t *
+static const uint32_t *
 parse_wcc_attr(netdissect_options *ndo,
-               const u_int32_t *dp)
+               const uint32_t *dp)
 {
-       ND_PRINT((ndo, " sz %" PRIu64, EXTRACT_64BITS(&dp[0])));
+       /* Our caller has already checked this */
+       ND_PRINT((ndo, " sz %" PRIu64, EXTRACT_BE_U_8(dp)));
        ND_PRINT((ndo, " mtime %u.%06u ctime %u.%06u",
-              EXTRACT_32BITS(&dp[2]), EXTRACT_32BITS(&dp[3]),
-              EXTRACT_32BITS(&dp[4]), EXTRACT_32BITS(&dp[5])));
+              EXTRACT_BE_U_4(dp + 2), EXTRACT_BE_U_4(dp + 3),
+              EXTRACT_BE_U_4(dp + 4), EXTRACT_BE_U_4(dp + 5)));
        return (dp + 6);
 }
 
 /*
  * Pre operation attributes. Print only if vflag > 1.
  */
-static const u_int32_t *
+static const uint32_t *
 parse_pre_op_attr(netdissect_options *ndo,
-                  const u_int32_t *dp, int verbose)
+                  const uint32_t *dp, int verbose)
 {
        ND_TCHECK(dp[0]);
-       if (!EXTRACT_32BITS(&dp[0]))
+       if (!EXTRACT_BE_U_4(dp))
                return (dp + 1);
        dp++;
        ND_TCHECK2(*dp, 24);
@@ -1421,25 +1278,25 @@ trunc:
 /*
  * Post operation attributes are printed if vflag >= 1
  */
-static const u_int32_t *
+static const uint32_t *
 parse_post_op_attr(netdissect_options *ndo,
-                   const u_int32_t *dp, int verbose)
+                   const uint32_t *dp, int verbose)
 {
        ND_TCHECK(dp[0]);
-       if (!EXTRACT_32BITS(&dp[0]))
+       if (!EXTRACT_BE_U_4(dp))
                return (dp + 1);
        dp++;
        if (verbose) {
                return parsefattr(ndo, dp, verbose, 1);
        } else
-               return (dp + (NFSX_V3FATTR / sizeof (u_int32_t)));
+               return (dp + (NFSX_V3FATTR / sizeof (uint32_t)));
 trunc:
        return (NULL);
 }
 
-static const u_int32_t *
+static const uint32_t *
 parse_wcc_data(netdissect_options *ndo,
-               const u_int32_t *dp, int verbose)
+               const uint32_t *dp, int verbose)
 {
        if (verbose > 1)
                ND_PRINT((ndo, " PRE:"));
@@ -1451,9 +1308,9 @@ parse_wcc_data(netdissect_options *ndo,
        return parse_post_op_attr(ndo, dp, verbose);
 }
 
-static const u_int32_t *
+static const uint32_t *
 parsecreateopres(netdissect_options *ndo,
-                 const u_int32_t *dp, int verbose)
+                 const uint32_t *dp, int verbose)
 {
        int er;
 
@@ -1463,7 +1320,7 @@ parsecreateopres(netdissect_options *ndo,
                dp = parse_wcc_data(ndo, dp, verbose);
        else {
                ND_TCHECK(dp[0]);
-               if (!EXTRACT_32BITS(&dp[0]))
+               if (!EXTRACT_BE_U_4(dp))
                        return (dp + 1);
                dp++;
                if (!(dp = parsefh(ndo, dp, 1)))
@@ -1484,18 +1341,18 @@ trunc:
 
 static int
 parsewccres(netdissect_options *ndo,
-            const u_int32_t *dp, int verbose)
+            const uint32_t *dp, int verbose)
 {
        int er;
 
        if (!(dp = parsestatus(ndo, dp, &er)))
                return (0);
-       return parse_wcc_data(ndo, dp, verbose) != 0;
+       return parse_wcc_data(ndo, dp, verbose) != NULL;
 }
 
-static const u_int32_t *
+static const uint32_t *
 parsev3rddirres(netdissect_options *ndo,
-                const u_int32_t *dp, int verbose)
+                const uint32_t *dp, int verbose)
 {
        int er;
 
@@ -1519,9 +1376,9 @@ trunc:
 
 static int
 parsefsinfo(netdissect_options *ndo,
-            const u_int32_t *dp)
+            const uint32_t *dp)
 {
-       struct nfsv3_fsinfo *sfp;
+       const struct nfsv3_fsinfo *sfp;
        int er;
 
        if (!(dp = parsestatus(ndo, dp, &er)))
@@ -1533,22 +1390,22 @@ parsefsinfo(netdissect_options *ndo,
        if (er)
                return (1);
 
-       sfp = (struct nfsv3_fsinfo *)dp;
+       sfp = (const struct nfsv3_fsinfo *)dp;
        ND_TCHECK(*sfp);
        ND_PRINT((ndo, " rtmax %u rtpref %u wtmax %u wtpref %u dtpref %u",
-              EXTRACT_32BITS(&sfp->fs_rtmax),
-              EXTRACT_32BITS(&sfp->fs_rtpref),
-              EXTRACT_32BITS(&sfp->fs_wtmax),
-              EXTRACT_32BITS(&sfp->fs_wtpref),
-              EXTRACT_32BITS(&sfp->fs_dtpref)));
+              EXTRACT_BE_U_4(&sfp->fs_rtmax),
+              EXTRACT_BE_U_4(&sfp->fs_rtpref),
+              EXTRACT_BE_U_4(&sfp->fs_wtmax),
+              EXTRACT_BE_U_4(&sfp->fs_wtpref),
+              EXTRACT_BE_U_4(&sfp->fs_dtpref)));
        if (ndo->ndo_vflag) {
                ND_PRINT((ndo, " rtmult %u wtmult %u maxfsz %" PRIu64,
-                      EXTRACT_32BITS(&sfp->fs_rtmult),
-                      EXTRACT_32BITS(&sfp->fs_wtmult),
-                      EXTRACT_64BITS((u_int32_t *)&sfp->fs_maxfilesize)));
+                      EXTRACT_BE_U_4(&sfp->fs_rtmult),
+                      EXTRACT_BE_U_4(&sfp->fs_wtmult),
+                      EXTRACT_BE_U_8((const uint32_t *)&sfp->fs_maxfilesize)));
                ND_PRINT((ndo, " delta %u.%06u ",
-                      EXTRACT_32BITS(&sfp->fs_timedelta.nfsv3_sec),
-                      EXTRACT_32BITS(&sfp->fs_timedelta.nfsv3_nsec)));
+                      EXTRACT_BE_U_4(&sfp->fs_timedelta.nfsv3_sec),
+                      EXTRACT_BE_U_4(&sfp->fs_timedelta.nfsv3_nsec)));
        }
        return (1);
 trunc:
@@ -1557,10 +1414,10 @@ trunc:
 
 static int
 parsepathconf(netdissect_options *ndo,
-              const u_int32_t *dp)
+              const uint32_t *dp)
 {
        int er;
-       struct nfsv3_pathconf *spp;
+       const struct nfsv3_pathconf *spp;
 
        if (!(dp = parsestatus(ndo, dp, &er)))
                return (0);
@@ -1571,16 +1428,16 @@ parsepathconf(netdissect_options *ndo,
        if (er)
                return (1);
 
-       spp = (struct nfsv3_pathconf *)dp;
+       spp = (const struct nfsv3_pathconf *)dp;
        ND_TCHECK(*spp);
 
        ND_PRINT((ndo, " linkmax %u namemax %u %s %s %s %s",
-              EXTRACT_32BITS(&spp->pc_linkmax),
-              EXTRACT_32BITS(&spp->pc_namemax),
-              EXTRACT_32BITS(&spp->pc_notrunc) ? "notrunc" : "",
-              EXTRACT_32BITS(&spp->pc_chownrestricted) ? "chownres" : "",
-              EXTRACT_32BITS(&spp->pc_caseinsensitive) ? "igncase" : "",
-              EXTRACT_32BITS(&spp->pc_casepreserving) ? "keepcase" : ""));
+              EXTRACT_BE_U_4(&spp->pc_linkmax),
+              EXTRACT_BE_U_4(&spp->pc_namemax),
+              EXTRACT_BE_U_4(&spp->pc_notrunc) ? "notrunc" : "",
+              EXTRACT_BE_U_4(&spp->pc_chownrestricted) ? "chownres" : "",
+              EXTRACT_BE_U_4(&spp->pc_caseinsensitive) ? "igncase" : "",
+              EXTRACT_BE_U_4(&spp->pc_casepreserving) ? "keepcase" : ""));
        return (1);
 trunc:
        return (0);
@@ -1588,9 +1445,9 @@ trunc:
 
 static void
 interp_reply(netdissect_options *ndo,
-             const struct sunrpc_msg *rp, u_int32_t proc, u_int32_t vers, int length)
+             const struct sunrpc_msg *rp, uint32_t proc, uint32_t vers, int length)
 {
-       register const u_int32_t *dp;
+       register const uint32_t *dp;
        register int v3;
        int er;
 
@@ -1599,25 +1456,16 @@ interp_reply(netdissect_options *ndo,
        if (!v3 && proc < NFS_NPROCS)
                proc = nfsv3_procid[proc];
 
+       ND_PRINT((ndo, " %s", tok2str(nfsproc_str, "proc-%u", proc)));
        switch (proc) {
 
-       case NFSPROC_NOOP:
-               ND_PRINT((ndo, " nop"));
-               return;
-
-       case NFSPROC_NULL:
-               ND_PRINT((ndo, " null"));
-               return;
-
        case NFSPROC_GETATTR:
-               ND_PRINT((ndo, " getattr"));
                dp = parserep(ndo, rp, length);
                if (dp != NULL && parseattrstat(ndo, dp, !ndo->ndo_qflag, v3) != 0)
                        return;
                break;
 
        case NFSPROC_SETATTR:
-               ND_PRINT((ndo, " setattr"));
                if (!(dp = parserep(ndo, rp, length)))
                        return;
                if (v3) {
@@ -1630,7 +1478,6 @@ interp_reply(netdissect_options *ndo,
                break;
 
        case NFSPROC_LOOKUP:
-               ND_PRINT((ndo, " lookup"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (v3) {
@@ -1659,7 +1506,6 @@ interp_reply(netdissect_options *ndo,
                break;
 
        case NFSPROC_ACCESS:
-               ND_PRINT((ndo, " access"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (!(dp = parsestatus(ndo, dp, &er)))
@@ -1668,19 +1514,19 @@ interp_reply(netdissect_options *ndo,
                        ND_PRINT((ndo, " attr:"));
                if (!(dp = parse_post_op_attr(ndo, dp, ndo->ndo_vflag)))
                        break;
-               if (!er)
-                       ND_PRINT((ndo, " c %04x", EXTRACT_32BITS(&dp[0])));
+               if (!er) {
+                       ND_TCHECK(dp[0]);
+                       ND_PRINT((ndo, " c %04x", EXTRACT_BE_U_4(dp)));
+               }
                return;
 
        case NFSPROC_READLINK:
-               ND_PRINT((ndo, " readlink"));
                dp = parserep(ndo, rp, length);
                if (dp != NULL && parselinkres(ndo, dp, v3) != 0)
                        return;
                break;
 
        case NFSPROC_READ:
-               ND_PRINT((ndo, " read"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (v3) {
@@ -1692,8 +1538,8 @@ interp_reply(netdissect_options *ndo,
                                return;
                        if (ndo->ndo_vflag) {
                                ND_TCHECK(dp[1]);
-                               ND_PRINT((ndo, " %u bytes", EXTRACT_32BITS(&dp[0])));
-                               if (EXTRACT_32BITS(&dp[1]))
+                               ND_PRINT((ndo, " %u bytes", EXTRACT_BE_U_4(dp)));
+                               if (EXTRACT_BE_U_4(dp + 1))
                                        ND_PRINT((ndo, " EOF"));
                        }
                        return;
@@ -1704,7 +1550,6 @@ interp_reply(netdissect_options *ndo,
                break;
 
        case NFSPROC_WRITE:
-               ND_PRINT((ndo, " write"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (v3) {
@@ -1716,12 +1561,12 @@ interp_reply(netdissect_options *ndo,
                                return;
                        if (ndo->ndo_vflag) {
                                ND_TCHECK(dp[0]);
-                               ND_PRINT((ndo, " %u bytes", EXTRACT_32BITS(&dp[0])));
+                               ND_PRINT((ndo, " %u bytes", EXTRACT_BE_U_4(dp)));
                                if (ndo->ndo_vflag > 1) {
                                        ND_TCHECK(dp[1]);
                                        ND_PRINT((ndo, " <%s>",
                                                tok2str(nfsv3_writemodes,
-                                                       NULL, EXTRACT_32BITS(&dp[1]))));
+                                                       NULL, EXTRACT_BE_U_4(dp + 1))));
                                }
                                return;
                        }
@@ -1732,24 +1577,11 @@ interp_reply(netdissect_options *ndo,
                break;
 
        case NFSPROC_CREATE:
-               ND_PRINT((ndo, " create"));
-               if (!(dp = parserep(ndo, rp, length)))
-                       break;
-               if (v3) {
-                       if (parsecreateopres(ndo, dp, ndo->ndo_vflag) != 0)
-                               return;
-               } else {
-                       if (parsediropres(ndo, dp) != 0)
-                               return;
-               }
-               break;
-
        case NFSPROC_MKDIR:
-               ND_PRINT((ndo, " mkdir"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (v3) {
-                       if (parsecreateopres(ndo, dp, ndo->ndo_vflag) != 0)
+                       if (parsecreateopres(ndo, dp, ndo->ndo_vflag) != NULL)
                                return;
                } else {
                        if (parsediropres(ndo, dp) != 0)
@@ -1758,54 +1590,38 @@ interp_reply(netdissect_options *ndo,
                break;
 
        case NFSPROC_SYMLINK:
-               ND_PRINT((ndo, " symlink"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (v3) {
-                       if (parsecreateopres(ndo, dp, ndo->ndo_vflag) != 0)
+                       if (parsecreateopres(ndo, dp, ndo->ndo_vflag) != NULL)
                                return;
                } else {
-                       if (parsestatus(ndo, dp, &er) != 0)
+                       if (parsestatus(ndo, dp, &er) != NULL)
                                return;
                }
                break;
 
        case NFSPROC_MKNOD:
-               ND_PRINT((ndo, " mknod"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
-               if (parsecreateopres(ndo, dp, ndo->ndo_vflag) != 0)
+               if (parsecreateopres(ndo, dp, ndo->ndo_vflag) != NULL)
                        return;
                break;
 
        case NFSPROC_REMOVE:
-               ND_PRINT((ndo, " remove"));
-               if (!(dp = parserep(ndo, rp, length)))
-                       break;
-               if (v3) {
-                       if (parsewccres(ndo, dp, ndo->ndo_vflag))
-                               return;
-               } else {
-                       if (parsestatus(ndo, dp, &er) != 0)
-                               return;
-               }
-               break;
-
        case NFSPROC_RMDIR:
-               ND_PRINT((ndo, " rmdir"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (v3) {
                        if (parsewccres(ndo, dp, ndo->ndo_vflag))
                                return;
                } else {
-                       if (parsestatus(ndo, dp, &er) != 0)
+                       if (parsestatus(ndo, dp, &er) != NULL)
                                return;
                }
                break;
 
        case NFSPROC_RENAME:
-               ND_PRINT((ndo, " rename"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (v3) {
@@ -1821,13 +1637,12 @@ interp_reply(netdissect_options *ndo,
                        }
                        return;
                } else {
-                       if (parsestatus(ndo, dp, &er) != 0)
+                       if (parsestatus(ndo, dp, &er) != NULL)
                                return;
                }
                break;
 
        case NFSPROC_LINK:
-               ND_PRINT((ndo, " link"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (v3) {
@@ -1843,13 +1658,12 @@ interp_reply(netdissect_options *ndo,
                                return;
                        }
                } else {
-                       if (parsestatus(ndo, dp, &er) != 0)
+                       if (parsestatus(ndo, dp, &er) != NULL)
                                return;
                }
                break;
 
        case NFSPROC_READDIR:
-               ND_PRINT((ndo, " readdir"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (v3) {
@@ -1862,7 +1676,6 @@ interp_reply(netdissect_options *ndo,
                break;
 
        case NFSPROC_READDIRPLUS:
-               ND_PRINT((ndo, " readdirplus"));
                if (!(dp = parserep(ndo, rp, length)))
                        break;
                if (parsev3rddirres(ndo, dp, ndo->ndo_vflag))
@@ -1870,35 +1683,30 @@ interp_reply(netdissect_options *ndo,
                break;
 
        case NFSPROC_FSSTAT:
-               ND_PRINT((ndo, " fsstat"));
                dp = parserep(ndo, rp, length);
                if (dp != NULL && parsestatfs(ndo, dp, v3) != 0)
                        return;
                break;
 
        case NFSPROC_FSINFO:
-               ND_PRINT((ndo, " fsinfo"));
                dp = parserep(ndo, rp, length);
                if (dp != NULL && parsefsinfo(ndo, dp) != 0)
                        return;
                break;
 
        case NFSPROC_PATHCONF:
-               ND_PRINT((ndo, " pathconf"));
                dp = parserep(ndo, rp, length);
                if (dp != NULL && parsepathconf(ndo, dp) != 0)
                        return;
                break;
 
        case NFSPROC_COMMIT:
-               ND_PRINT((ndo, " commit"));
                dp = parserep(ndo, rp, length);
                if (dp != NULL && parsewccres(ndo, dp, ndo->ndo_vflag) != 0)
                        return;
                break;
 
        default:
-               ND_PRINT((ndo, " proc-%u", proc));
                return;
        }
 trunc: