Use more the EXTRACT_8BITS() macro to fetch a one-byte value (17/n)

[tcpdump] / smbutil.c

diff --git a/smbutil.c b/smbutil.c
index b38d73afbce32a7cf0086a5c3928d02168af9146..847979e40e98ac41d9af6abc80c1cd2e745f8423 100644 (file)
--- a/smbutil.c
+++ b/smbutil.c
@@ -133,7 +133,8 @@ name_interpret(netdissect_options *ndo,
     if (in >= maxbuf)
        return(-1);     /* name goes past the end of the buffer */
     ND_TCHECK2(*in, 1);
-    len = (*in++) / 2;
+    len = EXTRACT_8BITS(in) / 2;
+    in++;
 
     *out=0;
 
@@ -185,7 +186,7 @@ name_ptr(netdissect_options *ndo,
        ND_TCHECK2(*p, 2);
        if ((p + 1) >= maxbuf)
            return(NULL);       /* name goes past the end of the buffer */
-       l = EXTRACT_16BITS(p) & 0x3FFF;
+       l = EXTRACT_BE_16BITS(p) & 0x3FFF;
        if (l == 0) {
            /* We have a pointer that points to itself. */
            return(NULL);
@@ -237,6 +238,7 @@ name_len(netdissect_options *ndo,
            return(-1); /* name goes past the end of the buffer */
        ND_TCHECK2(*s, 1);
        s += (*s) + 1;
+       ND_TCHECK2(*s, 1);
     }
     return(PTR_DIFF(s, s0) + 1);
 
@@ -503,7 +505,7 @@ smb_fdata1(netdissect_options *ndo,
          {
            unsigned int x;
            ND_TCHECK2(buf[0], 2);
-           x = reverse ? EXTRACT_16BITS(buf) :
+           x = reverse ? EXTRACT_BE_16BITS(buf) :
                          EXTRACT_LE_16BITS(buf);
            ND_PRINT((ndo, "%d (0x%x)", x, x));
            buf += 2;
@@ -514,7 +516,7 @@ smb_fdata1(netdissect_options *ndo,
          {
            unsigned int x;
            ND_TCHECK2(buf[0], 4);
-           x = reverse ? EXTRACT_32BITS(buf) :
+           x = reverse ? EXTRACT_BE_32BITS(buf) :
                          EXTRACT_LE_32BITS(buf);
            ND_PRINT((ndo, "%d (0x%x)", x, x));
            buf += 4;
@@ -525,7 +527,7 @@ smb_fdata1(netdissect_options *ndo,
          {
            uint64_t x;
            ND_TCHECK2(buf[0], 8);
-           x = reverse ? EXTRACT_64BITS(buf) :
+           x = reverse ? EXTRACT_BE_64BITS(buf) :
                          EXTRACT_LE_64BITS(buf);
            ND_PRINT((ndo, "%" PRIu64 " (0x%" PRIx64 ")", x, x));
            buf += 8;
@@ -538,9 +540,9 @@ smb_fdata1(netdissect_options *ndo,
            uint32_t x1, x2;
            uint64_t x;
            ND_TCHECK2(buf[0], 8);
-           x1 = reverse ? EXTRACT_32BITS(buf) :
+           x1 = reverse ? EXTRACT_BE_32BITS(buf) :
                           EXTRACT_LE_32BITS(buf);
-           x2 = reverse ? EXTRACT_32BITS(buf + 4) :
+           x2 = reverse ? EXTRACT_BE_32BITS(buf + 4) :
                           EXTRACT_LE_32BITS(buf + 4);
            x = (((uint64_t)x1) << 32) | x2;
            ND_PRINT((ndo, "%" PRIu64 " (0x%" PRIx64 ")", x, x));
@@ -562,7 +564,7 @@ smb_fdata1(netdissect_options *ndo,
          {
            unsigned int x;
            ND_TCHECK2(buf[0], 2);
-           x = reverse ? EXTRACT_16BITS(buf) :
+           x = reverse ? EXTRACT_BE_16BITS(buf) :
                          EXTRACT_LE_16BITS(buf);
            ND_PRINT((ndo, "0x%X", x));
            buf += 2;
@@ -573,7 +575,7 @@ smb_fdata1(netdissect_options *ndo,
          {
            unsigned int x;
            ND_TCHECK2(buf[0], 4);
-           x = reverse ? EXTRACT_32BITS(buf) :
+           x = reverse ? EXTRACT_BE_32BITS(buf) :
                          EXTRACT_LE_32BITS(buf);
            ND_PRINT((ndo, "0x%X", x));
            buf += 4;
@@ -594,7 +596,7 @@ smb_fdata1(netdissect_options *ndo,
 
            case 'd':
                ND_TCHECK2(buf[0], 2);
-               stringlen = reverse ? EXTRACT_16BITS(buf) :
+               stringlen = reverse ? EXTRACT_BE_16BITS(buf) :
                                      EXTRACT_LE_16BITS(buf);
                ND_PRINT((ndo, "%u", stringlen));
                buf += 2;
@@ -602,7 +604,7 @@ smb_fdata1(netdissect_options *ndo,
 
            case 'D':
                ND_TCHECK2(buf[0], 4);
-               stringlen = reverse ? EXTRACT_32BITS(buf) :
+               stringlen = reverse ? EXTRACT_BE_32BITS(buf) :
                                      EXTRACT_LE_32BITS(buf);
                ND_PRINT((ndo, "%u", stringlen));
                buf += 4;