]> The Tcpdump Group git mirrors - tcpdump/blobdiff - print-smb.c
More UNALIGNED_MEM{CPY,CMP} on IP addresses.
[tcpdump] / print-smb.c
index 6687c826afb846a1176c53416922eca6a6ef041c..32edd196d261152dabff6f9340110d0e7cedcd7c 100644 (file)
 #include "config.h"
 #endif
 
-#ifndef lint
-static const char rcsid[] =
-     "@(#) $Header: /tcpdump/master/tcpdump/print-smb.c,v 1.17 2001-07-28 22:59:30 guy Exp $";
-#endif
+#include <tcpdump-stdinc.h>
 
 #include <stdio.h>
 #include <string.h>
-#include <sys/types.h>
 
 #include "interface.h"
+#include "extract.h"
 #include "smb.h"
 
+static const char tstr[] = "[|SMB]";
+
 static int request = 0;
+static int unicodestr = 0;
 
-const uchar *startbuf = NULL;
+const u_char *startbuf = NULL;
 
 struct smbdescript {
-    char *req_f1;
-    char *req_f2;
-    char *rep_f1;
-    char *rep_f2;
-    /*
-     * sometimes (u_char *, u_char *, u_char *, u_char *) and
-     * sometimes (u_char *, u_char *, int, int)
-     */
-    void (*fn)(const u_char *, const u_char *, ...);
+    const char *req_f1;
+    const char *req_f2;
+    const char *rep_f1;
+    const char *rep_f2;
+    void (*fn)(const u_char *, const u_char *, const u_char *, const u_char *);
+};
+
+struct smbdescriptint {
+    const char *req_f1;
+    const char *req_f2;
+    const char *rep_f1;
+    const char *rep_f2;
+    void (*fn)(const u_char *, const u_char *, int, int);
 };
 
 struct smbfns
 {
     int id;
-    char *name;
+    const char *name;
     int flags;
     struct smbdescript descript;
 };
 
+struct smbfnsint
+{
+    int id;
+    const char *name;
+    int flags;
+    struct smbdescriptint descript;
+};
+
 #define DEFDESCRIPT    { NULL, NULL, NULL, NULL, NULL }
 
 #define FLG_CHAIN      (1 << 0)
 
-static struct smbfns *
-smbfind(int id, struct smbfns *list)
+static const struct smbfns *
+smbfind(int id, const struct smbfns *list)
 {
     int sindex;
 
@@ -62,24 +74,29 @@ smbfind(int id, struct smbfns *list)
     return(&list[0]);
 }
 
-static void
-trans2_findfirst(const uchar *param, const uchar *data, ...)
+static const struct smbfnsint *
+smbfindint(int id, const struct smbfnsint *list)
 {
-    char *fmt;
-    va_list ap;
-    int pcnt, dcnt;
+    int sindex;
+
+    for (sindex = 0; list[sindex].name; sindex++)
+       if (list[sindex].id == id)
+           return(&list[sindex]);
 
-    va_start(ap, data);
-    pcnt = va_arg(ap, int);
-    dcnt = va_arg(ap, int);
-    va_end(ap);
+    return(&list[0]);
+}
+
+static void
+trans2_findfirst(const u_char *param, const u_char *data, int pcnt, int dcnt)
+{
+    const char *fmt;
 
     if (request)
-       fmt = "Attribute=[A]\nSearchCount=[d]\nFlags=[w]\nLevel=[dP5]\nFile=[S]\n";
+       fmt = "Attribute=[A]\nSearchCount=[d]\nFlags=[w]\nLevel=[dP4]\nFile=[S]\n";
     else
        fmt = "Handle=[w]\nCount=[d]\nEOS=[w]\nEoffset=[d]\nLastNameOfs=[w]\n";
 
-    fdata(param, fmt, param + pcnt);
+    smb_fdata(param, fmt, param + pcnt, unicodestr);
     if (dcnt) {
        printf("data:\n");
        print_data(data, dcnt);
@@ -87,46 +104,44 @@ trans2_findfirst(const uchar *param, const uchar *data, ...)
 }
 
 static void
-trans2_qfsinfo(const uchar *param, const uchar *data, ...)
+trans2_qfsinfo(const u_char *param, const u_char *data, int pcnt, int dcnt)
 {
     static int level = 0;
-    char *fmt="";
-    va_list ap;
-    int pcnt, dcnt;
-
-    va_start(ap, data);
-    pcnt = va_arg(ap, int);
-    dcnt = va_arg(ap, int);
-    va_end(ap);
+    const char *fmt="";
 
     if (request) {
-       level = SVAL(param, 0);
+       TCHECK2(*param, 2);
+       level = EXTRACT_LE_16BITS(param);
        fmt = "InfoLevel=[d]\n";
-       fdata(param, fmt, param + pcnt);
+       smb_fdata(param, fmt, param + pcnt, unicodestr);
     } else {
        switch (level) {
        case 1:
            fmt = "idFileSystem=[W]\nSectorUnit=[D]\nUnit=[D]\nAvail=[D]\nSectorSize=[d]\n";
            break;
        case 2:
-           fmt = "CreationTime=[T2]VolNameLength=[B]\nVolumeLabel=[s12]\n";
+           fmt = "CreationTime=[T2]VolNameLength=[lb]\nVolumeLabel=[c]\n";
            break;
        case 0x105:
-           fmt = "Capabilities=[W]\nMaxFileLen=[D]\nVolNameLen=[D]\nVolume=[S]\n";
+           fmt = "Capabilities=[W]\nMaxFileLen=[D]\nVolNameLen=[lD]\nVolume=[C]\n";
            break;
        default:
            fmt = "UnknownLevel\n";
            break;
        }
-       fdata(data, fmt, data + dcnt);
+       smb_fdata(data, fmt, data + dcnt, unicodestr);
     }
     if (dcnt) {
        printf("data:\n");
        print_data(data, dcnt);
     }
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
-struct smbfns trans2_fns[] = {
+static const struct smbfnsint trans2_fns[] = {
     { 0, "TRANSACT2_OPEN", 0,
        { "Flags2=[w]\nMode=[w]\nSearchAttrib=[A]\nAttrib=[A]\nTime=[T2]\nOFun=[w]\nSize=[D]\nRes=([w, w, w, w, w])\nPath=[S]",
          NULL,
@@ -152,286 +167,357 @@ struct smbfns trans2_fns[] = {
 
 
 static void
-print_trans2(const uchar *words, const uchar *dat, ...)
+print_trans2(const u_char *words, const u_char *dat, const u_char *buf, const u_char *maxbuf)
 {
-    static struct smbfns *fn = &trans2_fns[0];
-    uchar *data, *param;
-    uchar *f1 = NULL, *f2 = NULL;
+    u_int bcc;
+    static const struct smbfnsint *fn = &trans2_fns[0];
+    const u_char *data, *param;
+    const u_char *w = words + 1;
+    const char *f1 = NULL, *f2 = NULL;
     int pcnt, dcnt;
-    va_list ap;
-    uchar *buf, *maxbuf;
-
-    va_start(ap, dat);
-    buf = va_arg(ap, uchar *);
-    maxbuf = va_arg(ap, uchar *);
-    va_end(ap);
 
+    TCHECK(words[0]);
     if (request) {
-       fn = smbfind(SVAL(words + 1, 14 * 2), trans2_fns);
-       data = buf+SVAL(words + 1, 12 * 2);
-       param = buf+SVAL(words + 1, 10 * 2);
-       pcnt = SVAL(words + 1, 9 * 2);
-       dcnt = SVAL(words + 1, 11 * 2);
+       TCHECK2(w[14 * 2], 2);
+       pcnt = EXTRACT_LE_16BITS(w + 9 * 2);
+       param = buf + EXTRACT_LE_16BITS(w + 10 * 2);
+       dcnt = EXTRACT_LE_16BITS(w + 11 * 2);
+       data = buf + EXTRACT_LE_16BITS(w + 12 * 2);
+       fn = smbfindint(EXTRACT_LE_16BITS(w + 14 * 2), trans2_fns);
     } else {
-       data = buf+SVAL(words + 1, 7 * 2);
-       param = buf+SVAL(words + 1, 4 * 2);
-       pcnt = SVAL(words + 1, 3 * 2);
-       dcnt = SVAL(words + 1, 6 * 2);
+       if (words[0] == 0) {
+           printf("%s\n", fn->name);
+           printf("Trans2Interim\n");
+           return;
+       }
+       TCHECK2(w[7 * 2], 2);
+       pcnt = EXTRACT_LE_16BITS(w + 3 * 2);
+       param = buf + EXTRACT_LE_16BITS(w + 4 * 2);
+       dcnt = EXTRACT_LE_16BITS(w + 6 * 2);
+       data = buf + EXTRACT_LE_16BITS(w + 7 * 2);
     }
 
     printf("%s param_length=%d data_length=%d\n", fn->name, pcnt, dcnt);
 
     if (request) {
-       if (CVAL(words, 0) == 8) {
-           fdata(words + 1,
+       if (words[0] == 8) {
+           smb_fdata(words + 1,
                "Trans2Secondary\nTotParam=[d]\nTotData=[d]\nParamCnt=[d]\nParamOff=[d]\nParamDisp=[d]\nDataCnt=[d]\nDataOff=[d]\nDataDisp=[d]\nHandle=[d]\n",
-               maxbuf);
+               maxbuf, unicodestr);
            return;
        } else {
-           fdata(words + 1,
-               "TotParam=[d]\nTotData=[d]\nMaxParam=[d]\nMaxData=[d]\nMaxSetup=[d]\nFlags=[w]\nTimeOut=[D]\nRes1=[w]\nParamCnt=[d]\nParamOff=[d]\nDataCnt=[d]\nDataOff=[d]\nSetupCnt=[d]\n",
-               words + 1 + 14 * 2);
-           fdata(data + 1, "TransactionName=[S]\n%", maxbuf);
+           smb_fdata(words + 1,
+               "TotParam=[d]\nTotData=[d]\nMaxParam=[d]\nMaxData=[d]\nMaxSetup=[b][P1]\nFlags=[w]\nTimeOut=[D]\nRes1=[w]\nParamCnt=[d]\nParamOff=[d]\nDataCnt=[d]\nDataOff=[d]\nSetupCnt=[b][P1]\n",
+               words + 1 + 14 * 2, unicodestr);
        }
        f1 = fn->descript.req_f1;
        f2 = fn->descript.req_f2;
     } else {
-       if (CVAL(words, 0) == 0) {
-           printf("Trans2Interim\n");
-           return;
-       } else {
-           fdata(words + 1,
-               "TotParam=[d]\nTotData=[d]\nRes1=[w]\nParamCnt=[d]\nParamOff=[d]\nParamDisp[d]\nDataCnt=[d]\nDataOff=[d]\nDataDisp=[d]\nSetupCnt=[d]\n",
-               words + 1 + 10 * 2);
-       }
+       smb_fdata(words + 1,
+           "TotParam=[d]\nTotData=[d]\nRes1=[w]\nParamCnt=[d]\nParamOff=[d]\nParamDisp[d]\nDataCnt=[d]\nDataOff=[d]\nDataDisp=[d]\nSetupCnt=[b][P1]\n",
+           words + 1 + 10 * 2, unicodestr);
        f1 = fn->descript.rep_f1;
        f2 = fn->descript.rep_f2;
     }
 
+    TCHECK2(*dat, 2);
+    bcc = EXTRACT_LE_16BITS(dat);
+    printf("smb_bcc=%u\n", bcc);
     if (fn->descript.fn)
-       fn->descript.fn(param, data, pcnt, dcnt);
+       (*fn->descript.fn)(param, data, pcnt, dcnt);
     else {
-       fdata(param, f1 ? f1 : (uchar *)"Paramaters=\n", param + pcnt);
-       fdata(data, f2 ? f2 : (uchar *)"Data=\n", data + dcnt);
+       smb_fdata(param, f1 ? f1 : "Parameters=\n", param + pcnt, unicodestr);
+       smb_fdata(data, f2 ? f2 : "Data=\n", data + dcnt, unicodestr);
     }
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
 
 static void
-print_browse(uchar *param, int paramlen, const uchar *data, int datalen)
+print_browse(const u_char *param, int paramlen, const u_char *data, int datalen)
 {
-    const uchar *maxbuf = data + datalen;
-    int command = CVAL(data, 0);
+    const u_char *maxbuf = data + datalen;
+    int command;
 
-    fdata(param, "BROWSE PACKET\n|Param ", param+paramlen);
+    TCHECK(data[0]);
+    command = data[0];
+
+    smb_fdata(param, "BROWSE PACKET\n|Param ", param+paramlen, unicodestr);
 
     switch (command) {
     case 0xF:
-       data = fdata(data,
+       data = smb_fdata(data,
            "BROWSE PACKET:\nType=[B] (LocalMasterAnnouncement)\nUpdateCount=[w]\nRes1=[B]\nAnnounceInterval=[d]\nName=[n2]\nMajorVersion=[B]\nMinorVersion=[B]\nServerType=[W]\nElectionVersion=[w]\nBrowserConstant=[w]\n",
-           maxbuf);
+           maxbuf, unicodestr);
        break;
 
     case 0x1:
-       data = fdata(data,
+       data = smb_fdata(data,
            "BROWSE PACKET:\nType=[B] (HostAnnouncement)\nUpdateCount=[w]\nRes1=[B]\nAnnounceInterval=[d]\nName=[n2]\nMajorVersion=[B]\nMinorVersion=[B]\nServerType=[W]\nElectionVersion=[w]\nBrowserConstant=[w]\n",
-           maxbuf);
+           maxbuf, unicodestr);
        break;
 
     case 0x2:
-       data = fdata(data,
+       data = smb_fdata(data,
            "BROWSE PACKET:\nType=[B] (AnnouncementRequest)\nFlags=[B]\nReplySystemName=[S]\n",
-           maxbuf);
+           maxbuf, unicodestr);
        break;
 
     case 0xc:
-       data = fdata(data,
+       data = smb_fdata(data,
            "BROWSE PACKET:\nType=[B] (WorkgroupAnnouncement)\nUpdateCount=[w]\nRes1=[B]\nAnnounceInterval=[d]\nName=[n2]\nMajorVersion=[B]\nMinorVersion=[B]\nServerType=[W]\nCommentPointer=[W]\nServerName=[S]\n",
-           maxbuf);
+           maxbuf, unicodestr);
        break;
 
     case 0x8:
-       data = fdata(data,
+       data = smb_fdata(data,
            "BROWSE PACKET:\nType=[B] (ElectionFrame)\nElectionVersion=[B]\nOSSummary=[W]\nUptime=[(W, W)]\nServerName=[S]\n",
-           maxbuf);
+           maxbuf, unicodestr);
        break;
 
     case 0xb:
-       data = fdata(data,
+       data = smb_fdata(data,
            "BROWSE PACKET:\nType=[B] (BecomeBackupBrowser)\nName=[S]\n",
-           maxbuf);
+           maxbuf, unicodestr);
        break;
 
     case 0x9:
-       data = fdata(data,
-           "BROWSE PACKET:\nType=[B] (GetBackupList)\nListCount?=[B]\nToken?=[B]\n",
-           maxbuf);
+       data = smb_fdata(data,
+           "BROWSE PACKET:\nType=[B] (GetBackupList)\nListCount?=[B]\nToken=[W]\n",
+           maxbuf, unicodestr);
        break;
 
     case 0xa:
-       data = fdata(data,
-           "BROWSE PACKET:\nType=[B] (BackupListResponse)\nServerCount?=[B]\nToken?=[B]*Name=[S]\n",
-           maxbuf);
+       data = smb_fdata(data,
+           "BROWSE PACKET:\nType=[B] (BackupListResponse)\nServerCount?=[B]\nToken=[W]\n*Name=[S]\n",
+           maxbuf, unicodestr);
        break;
 
     case 0xd:
-       data = fdata(data,
+       data = smb_fdata(data,
            "BROWSE PACKET:\nType=[B] (MasterAnnouncement)\nMasterName=[S]\n",
-           maxbuf);
+           maxbuf, unicodestr);
        break;
 
     case 0xe:
-       data = fdata(data,
-           "BROWSE PACKET:\nType=[B] (ResetBrowser)\nOptions=[B]\n", maxbuf);
+       data = smb_fdata(data,
+           "BROWSE PACKET:\nType=[B] (ResetBrowser)\nOptions=[B]\n", maxbuf, unicodestr);
        break;
 
     default:
-       data = fdata(data, "Unknown Browser Frame ", maxbuf);
+       data = smb_fdata(data, "Unknown Browser Frame ", maxbuf, unicodestr);
        break;
     }
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
 
 static void
-print_ipc(uchar *param, int paramlen, uchar *data, int datalen)
+print_ipc(const u_char *param, int paramlen, const u_char *data, int datalen)
 {
     if (paramlen)
-       fdata(param, "Command=[w]\nStr1=[S]\nStr2=[S]\n", param + paramlen);
+       smb_fdata(param, "Command=[w]\nStr1=[S]\nStr2=[S]\n", param + paramlen,
+           unicodestr);
     if (datalen)
-       fdata(data, "IPC ", data + datalen);
+       smb_fdata(data, "IPC ", data + datalen, unicodestr);
 }
 
 
 static void
-print_trans(const uchar *words, const uchar *data1, ...)
+print_trans(const u_char *words, const u_char *data1, const u_char *buf, const u_char *maxbuf)
 {
-    uchar *f1, *f2, *f3, *f4;
-    uchar *data, *param;
+    u_int bcc;
+    const char *f1, *f2, *f3, *f4;
+    const u_char *data, *param;
+    const u_char *w = words + 1;
     int datalen, paramlen;
-    va_list ap;
-    uchar *buf, *maxbuf;
-
-    va_start(ap, data1);
-    buf = va_arg(ap, uchar *);
-    maxbuf = va_arg(ap, uchar *);
-    va_end(ap);
 
     if (request) {
-       paramlen = SVAL(words + 1, 9 * 2);
-       param = buf + SVAL(words + 1, 10 * 2);
-       datalen = SVAL(words + 1, 11 * 2);
-       data = buf + SVAL(words + 1, 12 * 2);
+       TCHECK2(w[12 * 2], 2);
+       paramlen = EXTRACT_LE_16BITS(w + 9 * 2);
+       param = buf + EXTRACT_LE_16BITS(w + 10 * 2);
+       datalen = EXTRACT_LE_16BITS(w + 11 * 2);
+       data = buf + EXTRACT_LE_16BITS(w + 12 * 2);
        f1 = "TotParamCnt=[d] \nTotDataCnt=[d] \nMaxParmCnt=[d] \nMaxDataCnt=[d]\nMaxSCnt=[d] \nTransFlags=[w] \nRes1=[w] \nRes2=[w] \nRes3=[w]\nParamCnt=[d] \nParamOff=[d] \nDataCnt=[d] \nDataOff=[d] \nSUCnt=[d]\n";
        f2 = "|Name=[S]\n";
        f3 = "|Param ";
        f4 = "|Data ";
     } else {
-       paramlen = SVAL(words + 1, 3 * 2);
-       param = buf + SVAL(words + 1, 4 * 2);
-       datalen = SVAL(words + 1, 6 * 2);
-       data = buf + SVAL(words + 1, 7 * 2);
+       TCHECK2(w[7 * 2], 2);
+       paramlen = EXTRACT_LE_16BITS(w + 3 * 2);
+       param = buf + EXTRACT_LE_16BITS(w + 4 * 2);
+       datalen = EXTRACT_LE_16BITS(w + 6 * 2);
+       data = buf + EXTRACT_LE_16BITS(w + 7 * 2);
        f1 = "TotParamCnt=[d] \nTotDataCnt=[d] \nRes1=[d]\nParamCnt=[d] \nParamOff=[d] \nRes2=[d] \nDataCnt=[d] \nDataOff=[d] \nRes3=[d]\nLsetup=[d]\n";
        f2 = "|Unknown ";
        f3 = "|Param ";
        f4 = "|Data ";
     }
 
-    fdata(words + 1, f1, MIN(words + 1 + 2 * CVAL(words, 0), maxbuf));
-    fdata(data1 + 2, f2, maxbuf - (paramlen + datalen));
+    smb_fdata(words + 1, f1, SMBMIN(words + 1 + 2 * words[0], maxbuf),
+        unicodestr);
 
-    if (!strcmp(data1 + 2, "\\MAILSLOT\\BROWSE")) {
-       print_browse(param, paramlen, data, datalen);
-       return;
-    }
+    TCHECK2(*data1, 2);
+    bcc = EXTRACT_LE_16BITS(data1);
+    printf("smb_bcc=%u\n", bcc);
+    if (bcc > 0) {
+       smb_fdata(data1 + 2, f2, maxbuf - (paramlen + datalen), unicodestr);
 
-    if (!strcmp(data1 + 2, "\\PIPE\\LANMAN")) {
-       print_ipc(param, paramlen, data, datalen);
-       return;
-    }
+       if (strcmp((const char *)(data1 + 2), "\\MAILSLOT\\BROWSE") == 0) {
+           print_browse(param, paramlen, data, datalen);
+           return;
+       }
 
-    if (paramlen)
-       fdata(param, f3, MIN(param + paramlen, maxbuf));
-    if (datalen)
-       fdata(data, f4, MIN(data + datalen, maxbuf));
+       if (strcmp((const char *)(data1 + 2), "\\PIPE\\LANMAN") == 0) {
+           print_ipc(param, paramlen, data, datalen);
+           return;
+       }
+
+       if (paramlen)
+           smb_fdata(param, f3, SMBMIN(param + paramlen, maxbuf), unicodestr);
+       if (datalen)
+           smb_fdata(data, f4, SMBMIN(data + datalen, maxbuf), unicodestr);
+    }
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
 
 static void
-print_negprot(const uchar *words, const uchar *data, ...)
+print_negprot(const u_char *words, const u_char *data, const u_char *buf _U_, const u_char *maxbuf)
 {
-    uchar *f1 = NULL, *f2 = NULL;
-    va_list ap;
-    uchar *buf, *maxbuf;
-
-    va_start(ap, data);
-    buf = va_arg(ap, uchar *);
-    maxbuf = va_arg(ap, uchar *);
-    va_end(ap);
+    u_int wct, bcc;
+    const char *f1 = NULL, *f2 = NULL;
 
+    TCHECK(words[0]);
+    wct = words[0];
     if (request)
-       f2 = "*|Dialect=[Z]\n";
+       f2 = "*|Dialect=[Y]\n";
     else {
-       if (CVAL(words, 0) == 1)
+       if (wct == 1)
            f1 = "Core Protocol\nDialectIndex=[d]";
-       else if (CVAL(words, 0) == 17)
+       else if (wct == 17)
            f1 = "NT1 Protocol\nDialectIndex=[d]\nSecMode=[B]\nMaxMux=[d]\nNumVcs=[d]\nMaxBuffer=[D]\nRawSize=[D]\nSessionKey=[W]\nCapabilities=[W]\nServerTime=[T3]TimeZone=[d]\nCryptKey=";
-       else if (CVAL(words, 0) == 13)
+       else if (wct == 13)
            f1 = "Coreplus/Lanman1/Lanman2 Protocol\nDialectIndex=[d]\nSecMode=[w]\nMaxXMit=[d]\nMaxMux=[d]\nMaxVcs=[d]\nBlkMode=[w]\nSessionKey=[W]\nServerTime=[T1]TimeZone=[d]\nRes=[W]\nCryptKey=";
     }
 
     if (f1)
-       fdata(words + 1, f1, MIN(words + 1 + CVAL(words, 0) * 2, maxbuf));
-    else
-       print_data(words + 1, MIN(CVAL(words, 0) * 2,
-           PTR_DIFF(maxbuf, words + 1)));
-
-    if (f2)
-       fdata(data + 2, f2, MIN(data + 2 + SVAL(data, 0), maxbuf));
+       smb_fdata(words + 1, f1, SMBMIN(words + 1 + wct * 2, maxbuf),
+           unicodestr);
     else
-       print_data(data + 2, MIN(SVAL(data, 0), PTR_DIFF(maxbuf, data + 2)));
+       print_data(words + 1, SMBMIN(wct * 2, PTR_DIFF(maxbuf, words + 1)));
+
+    TCHECK2(*data, 2);
+    bcc = EXTRACT_LE_16BITS(data);
+    printf("smb_bcc=%u\n", bcc);
+    if (bcc > 0) {
+       if (f2)
+           smb_fdata(data + 2, f2, SMBMIN(data + 2 + EXTRACT_LE_16BITS(data),
+               maxbuf), unicodestr);
+       else
+           print_data(data + 2, SMBMIN(EXTRACT_LE_16BITS(data), PTR_DIFF(maxbuf, data + 2)));
+    }
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
 static void
-print_sesssetup(const uchar *words, const uchar *data, ...)
+print_sesssetup(const u_char *words, const u_char *data, const u_char *buf _U_, const u_char *maxbuf)
 {
-    int wcnt = CVAL(words, 0);
-    uchar *f1 = NULL, *f2 = NULL;
-    va_list ap;
-    uchar *buf, *maxbuf;
-
-    va_start(ap, data);
-    buf = va_arg(ap, uchar *);
-    maxbuf = va_arg(ap, uchar *);
-    va_end(ap);
+    u_int wct, bcc;
+    const char *f1 = NULL, *f2 = NULL;
 
+    TCHECK(words[0]);
+    wct = words[0];
     if (request) {
-       if (wcnt == 10)
+       if (wct == 10)
            f1 = "Com2=[w]\nOff2=[d]\nBufSize=[d]\nMpxMax=[d]\nVcNum=[d]\nSessionKey=[W]\nPassLen=[d]\nCryptLen=[d]\nCryptOff=[d]\nPass&Name=\n";
        else
            f1 = "Com2=[B]\nRes1=[B]\nOff2=[d]\nMaxBuffer=[d]\nMaxMpx=[d]\nVcNumber=[d]\nSessionKey=[W]\nCaseInsensitivePasswordLength=[d]\nCaseSensitivePasswordLength=[d]\nRes=[W]\nCapabilities=[W]\nPass1&Pass2&Account&Domain&OS&LanMan=\n";
     } else {
-       if (CVAL(words,0) == 3) {
-       f1 = "Com2=[w]\nOff2=[d]\nAction=[w]\n";
-       } else if (CVAL(words,0) == 13) {
-       f1 = "Com2=[B]\nRes=[B]\nOff2=[d]\nAction=[w]\n";
-       f2 = "NativeOS=[S]\nNativeLanMan=[S]\nPrimaryDomain=[S]\n";
+       if (wct == 3) {
+           f1 = "Com2=[w]\nOff2=[d]\nAction=[w]\n";
+       } else if (wct == 13) {
+           f1 = "Com2=[B]\nRes=[B]\nOff2=[d]\nAction=[w]\n";
+           f2 = "NativeOS=[S]\nNativeLanMan=[S]\nPrimaryDomain=[S]\n";
        }
     }
 
     if (f1)
-       fdata(words + 1, f1, MIN(words + 1 + CVAL(words, 0) * 2, maxbuf));
+       smb_fdata(words + 1, f1, SMBMIN(words + 1 + wct * 2, maxbuf),
+           unicodestr);
     else
-       print_data(words + 1, MIN(CVAL(words, 0) * 2,
-           PTR_DIFF(maxbuf, words + 1)));
+       print_data(words + 1, SMBMIN(wct * 2, PTR_DIFF(maxbuf, words + 1)));
+
+    TCHECK2(*data, 2);
+    bcc = EXTRACT_LE_16BITS(data);
+    printf("smb_bcc=%u\n", bcc);
+    if (bcc > 0) {
+       if (f2)
+           smb_fdata(data + 2, f2, SMBMIN(data + 2 + EXTRACT_LE_16BITS(data),
+               maxbuf), unicodestr);
+       else
+           print_data(data + 2, SMBMIN(EXTRACT_LE_16BITS(data), PTR_DIFF(maxbuf, data + 2)));
+    }
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
+}
 
-    if (f2)
-       fdata(data + 2, f2, MIN(data + 2 + SVAL(data, 0), maxbuf));
-    else
-       print_data(data + 2, MIN(SVAL(data, 0), PTR_DIFF(maxbuf, data+2)));
+static void
+print_lockingandx(const u_char *words, const u_char *data, const u_char *buf _U_, const u_char *maxbuf)
+{
+    u_int wct, bcc;
+    const u_char *maxwords;
+    const char *f1 = NULL, *f2 = NULL;
+
+    TCHECK(words[0]);
+    wct = words[0];
+    if (request) {
+       f1 = "Com2=[w]\nOff2=[d]\nHandle=[d]\nLockType=[w]\nTimeOut=[D]\nUnlockCount=[d]\nLockCount=[d]\n";
+       TCHECK(words[7]);
+       if (words[7] & 0x10)
+           f2 = "*Process=[d]\n[P2]Offset=[M]\nLength=[M]\n";
+       else
+           f2 = "*Process=[d]\nOffset=[D]\nLength=[D]\n";
+    } else {
+       f1 = "Com2=[w]\nOff2=[d]\n";
+    }
+
+    maxwords = SMBMIN(words + 1 + wct * 2, maxbuf);
+    if (wct)
+       smb_fdata(words + 1, f1, maxwords, unicodestr);
+
+    TCHECK2(*data, 2);
+    bcc = EXTRACT_LE_16BITS(data);
+    printf("smb_bcc=%u\n", bcc);
+    if (bcc > 0) {
+       if (f2)
+           smb_fdata(data + 2, f2, SMBMIN(data + 2 + EXTRACT_LE_16BITS(data),
+               maxbuf), unicodestr);
+       else
+           print_data(data + 2, SMBMIN(EXTRACT_LE_16BITS(data), PTR_DIFF(maxbuf, data + 2)));
+    }
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
 
-static struct smbfns smb_fns[] = {
+static const struct smbfns smb_fns[] = {
     { -1, "SMBunknown", 0, DEFDESCRIPT },
 
     { SMBtcon, "SMBtcon", 0,
@@ -515,7 +601,7 @@ static struct smbfns smb_fns[] = {
     { pSETDIR, "SMBsetdir", 0, { NULL, "Path=[Z]\n", NULL, NULL, NULL } },
 
     { SMBlseek, "SMBlseek", 0,
-       { "Handle=[d]\nMode=[w]\nOffset=[D]\n", "Offset=[D]\n", NULL, NULL } },
+       { "Handle=[d]\nMode=[w]\nOffset=[D]\n", "Offset=[D]\n", NULL, NULL, NULL } },
 
     { SMBflush, "SMBflush", 0, { "Handle=[d]\n", NULL, NULL, NULL, NULL } },
 
@@ -618,11 +704,6 @@ static struct smbfns smb_fns[] = {
          "Com2=[w]\nOff2=[d]\nCount=[d]\nRemaining=[d]\nRes=[W]\n",
          NULL, NULL } },
 
-    { SMBlockingX, "SMBlockingX", FLG_CHAIN,
-       { "Com2=[w]\nOff2=[d]\nHandle=[d]\nLockType=[w]\nTimeOut=[D]\nUnlockCount=[d]\nLockCount=[d]\n",
-         "*Process=[d]\nOffset=[D]\nLength=[D]\n",
-         "Com2=[w]\nOff2=[d]\n", NULL, NULL } },
-
     { SMBffirst, "SMBffirst", 0,
        { "Count=[d]\nAttrib=[A]\n",
          "Path=[Z]\nBlkType=[B]\nBlkLen=[d]\n|Res1=[B]\nMask=[s11]\nSrv1=[B]\nDirIndex=[d]\nSrv2=[w]\n",
@@ -677,7 +758,10 @@ static struct smbfns smb_fns[] = {
 
     { SMBtconX, "SMBtconX", FLG_CHAIN,
        { "Com2=[w]\nOff2=[d]\nFlags=[w]\nPassLen=[d]\nPasswd&Path&Device=\n",
-         NULL, "Com2=[w]\nOff2=[d]\n", "ServiceType=[S]\n", NULL } },
+         NULL, "Com2=[w]\nOff2=[d]\n", "ServiceType=[R]\n", NULL } },
+
+    { SMBlockingX, "SMBlockingX", FLG_CHAIN,
+       { NULL, NULL, NULL, NULL, print_lockingandx } },
 
     { SMBtrans2, "SMBtrans2", 0, { NULL, NULL, NULL, NULL, print_trans2 } },
 
@@ -690,10 +774,10 @@ static struct smbfns smb_fns[] = {
     { SMBnttranss, "SMBnttranss", 0, DEFDESCRIPT },
 
     { SMBntcreateX, "SMBntcreateX", FLG_CHAIN,
-       { "Com2=[w]\nOff2=[d]\nRes=[b]\nNameLen=[d]\nFlags=[W]\nRootDirectoryFid=[D]\nAccessMask=[W]\nAllocationSize=[L]\nExtFileAttributes=[W]\nShareAccess=[W]\nCreateDisposition=[W]\nCreateOptions=[W]\nImpersonationLevel=[W]\nSecurityFlags=[b]\n",
-         "Path=[S]\n",
+       { "Com2=[w]\nOff2=[d]\nRes=[b]\nNameLen=[ld]\nFlags=[W]\nRootDirectoryFid=[D]\nAccessMask=[W]\nAllocationSize=[L]\nExtFileAttributes=[W]\nShareAccess=[W]\nCreateDisposition=[W]\nCreateOptions=[W]\nImpersonationLevel=[W]\nSecurityFlags=[b]\n",
+         "Path=[C]\n",
          "Com2=[w]\nOff2=[d]\nOplockLevel=[b]\nFid=[d]\nCreateAction=[W]\nCreateTime=[T3]LastAccessTime=[T3]LastWriteTime=[T3]ChangeTime=[T3]ExtFileAttributes=[W]\nAllocationSize=[L]\nEndOfFile=[L]\nFileType=[w]\nDeviceState=[w]\nDirectory=[b]\n",
-         NULL } },
+         NULL, NULL } },
 
     { SMBntcancel, "SMBntcancel", 0, DEFDESCRIPT },
 
@@ -705,17 +789,26 @@ static struct smbfns smb_fns[] = {
  * print a SMB message
  */
 static void
-print_smb(const uchar *buf, const uchar *maxbuf)
+print_smb(const u_char *buf, const u_char *maxbuf)
 {
+    u_int16_t flags2;
+    int nterrcodes;
     int command;
-    const uchar *words, *data;
-    struct smbfns *fn;
-    char *fmt_smbheader =
+    u_int32_t nterror;
+    const u_char *words, *maxwords, *data;
+    const struct smbfns *fn;
+    const char *fmt_smbheader =
         "[P4]SMB Command   =  [B]\nError class   =  [BP1]\nError code    =  [d]\nFlags1        =  [B]\nFlags2        =  [B][P13]\nTree ID       =  [d]\nProc ID       =  [d]\nUID           =  [d]\nMID           =  [d]\nWord Count    =  [b]\n";
+    int smboffset;
 
-    request = (CVAL(buf, 9) & 0x80) ? 0 : 1;
+    TCHECK(buf[9]);
+    request = (buf[9] & 0x80) ? 0 : 1;
+    flags2 = EXTRACT_LE_16BITS(&buf[10]);
+    unicodestr = flags2 & 0x8000;
+    nterrcodes = flags2 & 0x4000;
+    startbuf = buf;
 
-    command = CVAL(buf, 4);
+    command = buf[4];
 
     fn = smbfind(command, smb_fns);
 
@@ -728,20 +821,30 @@ print_smb(const uchar *buf, const uchar *maxbuf)
        return;
 
     /* print out the header */
-    fdata(buf, fmt_smbheader, buf + 33);
+    smb_fdata(buf, fmt_smbheader, buf + 33, unicodestr);
 
-    if (CVAL(buf, 5)) {
-       int class = CVAL(buf, 5);
-       int num = SVAL(buf, 7);
-       printf("SMBError = %s\n", smb_errstr(class, num));
+    if (nterrcodes) {
+       nterror = EXTRACT_LE_32BITS(&buf[5]);
+       if (nterror)
+           printf("NTError = %s\n", nt_errstr(nterror));
+    } else {
+       if (buf[5])
+           printf("SMBError = %s\n", smb_errstr(buf[5], EXTRACT_LE_16BITS(&buf[7])));
     }
 
-    words = buf + 32;
-    data = words + 1 + CVAL(words,0)*2;
+    smboffset = 32;
 
-    while (words && data) {
-       char *f1, *f2;
-       int wct = CVAL(words, 0);
+    for (;;) {
+       const char *f1, *f2;
+       int wct;
+       u_int bcc;
+       int newsmboffset;
+
+       words = buf + smboffset;
+       TCHECK(words[0]);
+       wct = words[0];
+       data = words + 1 + wct * 2;
+       maxwords = SMBMIN(data, maxbuf);
 
        if (request) {
            f1 = fn->descript.req_f1;
@@ -752,49 +855,64 @@ print_smb(const uchar *buf, const uchar *maxbuf)
        }
 
        if (fn->descript.fn)
-           fn->descript.fn(words, data, buf, maxbuf);
+           (*fn->descript.fn)(words, data, buf, maxbuf);
        else {
-           if (f1) {
-               printf("smbvwv[]=\n");
-               fdata(words + 1, f1, words + 1 + wct * 2);
-           } else if (wct) {
-               int i;
-               int v;
-               printf("smbvwv[]=\n");
-               for (i = 0; i < wct; i++) {
-                   v = SVAL(words + 1, 2 * i);
-                   printf("smb_vwv[%d]=%d (0x%X)\n", i, v, v);
+           if (wct) {
+               if (f1)
+                   smb_fdata(words + 1, f1, words + 1 + wct * 2, unicodestr);
+               else {
+                   int i;
+                   int v;
+
+                   for (i = 0; &words[1 + 2 * i] < maxwords; i++) {
+                       TCHECK2(words[1 + 2 * i], 2);
+                       v = EXTRACT_LE_16BITS(words + 1 + 2 * i);
+                       printf("smb_vwv[%d]=%d (0x%X)\n", i, v, v);
+                   }
                }
            }
 
+           TCHECK2(*data, 2);
+           bcc = EXTRACT_LE_16BITS(data);
+           printf("smb_bcc=%u\n", bcc);
            if (f2) {
-               printf("smbbuf[]=\n");
-               fdata(data + 2, f2, maxbuf);
+               if (bcc > 0)
+                   smb_fdata(data + 2, f2, data + 2 + bcc, unicodestr);
            } else {
-               int bcc = SVAL(data, 0);
-               printf("smb_bcc=%d\n", bcc);
                if (bcc > 0) {
                    printf("smb_buf[]=\n");
-                   print_data(data + 2, MIN(bcc, PTR_DIFF(maxbuf, data + 2)));
+                   print_data(data + 2, SMBMIN(bcc, PTR_DIFF(maxbuf, data + 2)));
                }
            }
        }
 
-       if ((fn->flags & FLG_CHAIN) != 0&& CVAL(words, 0) &&
-           SVAL(words, 1) != 0xFF) {
-           command = SVAL(words, 1);
-           words = buf + SVAL(words, 3);
-           data = words + 1 + CVAL(words, 0) * 2;
+       if ((fn->flags & FLG_CHAIN) == 0)
+           break;
+       if (wct == 0)
+           break;
+       TCHECK(words[1]);
+       command = words[1];
+       if (command == 0xFF)
+           break;
+       TCHECK2(words[3], 2);
+       newsmboffset = EXTRACT_LE_16BITS(words + 3);
 
-           fn = smbfind(command, smb_fns);
+       fn = smbfind(command, smb_fns);
 
-           printf("\nSMB PACKET: %s (%s) (CHAINED)\n",
-               fn->name, request ? "REQUEST" : "REPLY");
-       } else
-           words = data = NULL;
+       printf("\nSMB PACKET: %s (%s) (CHAINED)\n",
+           fn->name, request ? "REQUEST" : "REPLY");
+       if (newsmboffset <= smboffset) {
+           printf("Bad andX offset: %u <= %u\n", newsmboffset, smboffset);
+           break;
+       }
+       smboffset = newsmboffset;
     }
 
     printf("\n");
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
 
@@ -802,89 +920,163 @@ print_smb(const uchar *buf, const uchar *maxbuf)
  * print a NBT packet received across tcp on port 139
  */
 void
-nbt_tcp_print(const uchar *data, int length)
+nbt_tcp_print(const u_char *data, int length)
 {
-    const uchar *maxbuf = data + length;
-    int flags = CVAL(data, 0);
-    int nbt_len = RSVAL(data, 2);
+    int caplen;
+    int type;
+    u_int nbt_len;
+    const u_char *maxbuf;
+
+    if (length < 4)
+       goto trunc;
+    if (snapend < data)
+       goto trunc;
+    caplen = snapend - data;
+    if (caplen < 4)
+       goto trunc;
+    maxbuf = data + caplen;
+    type = data[0];
+    nbt_len = EXTRACT_16BITS(data + 2);
+    length -= 4;
+    caplen -= 4;
 
     startbuf = data;
-    if (maxbuf <= data)
-       return;
-
-    if (vflag > 1)
-       printf ("\n>>> ");
-
-    printf("NBT Packet");
-
-    if (vflag < 2)
-       return;
-
-    printf("\n");
 
-    switch (flags) {
-    case 1:
-       printf("flags=0x%x\n", flags);
-    case 0:
-       data = fdata(data, "NBT Session Packet\nFlags=[rw]\nLength=[rd]\n",
-           data + 4);
-       if (data == NULL)
+    if (vflag < 2) {
+       printf(" NBT Session Packet: ");
+       switch (type) {
+       case 0x00:
+           printf("Session Message");
            break;
-       if (memcmp(data,"\377SMB",4) == 0) {
-           if (nbt_len > PTR_DIFF(maxbuf, data))
-           printf("WARNING: Short packet. Try increasing the snap length (%lu)\n",
-              (unsigned long)PTR_DIFF(maxbuf, data));
-           print_smb(data, maxbuf > data + nbt_len ? data + nbt_len : maxbuf);
-       } else
-           printf("Session packet:(raw data?)\n");
-       break;
 
        case 0x81:
-           data = fdata(data,
-               "NBT Session Request\nFlags=[rW]\nDestination=[n1]\nSource=[n1]\n",
-               maxbuf);
+           printf("Session Request");
            break;
 
        case 0x82:
-           data = fdata(data, "NBT Session Granted\nFlags=[rW]\n", maxbuf);
+           printf("Session Granted");
            break;
 
        case 0x83:
          {
-           int ecode = CVAL(data,4);
+           int ecode;
 
-           data = fdata(data, "NBT SessionReject\nFlags=[rW]\nReason=[B]\n",
-               maxbuf);
+           if (nbt_len < 4)
+               goto trunc;
+           if (length < 4)
+               goto trunc;
+           if (caplen < 4)
+               goto trunc;
+           ecode = data[4];
+
+           printf("Session Reject, ");
            switch (ecode) {
            case 0x80:
-               printf("Not listening on called name\n");
+               printf("Not listening on called name");
                break;
            case 0x81:
-               printf("Not listening for calling name\n");
+               printf("Not listening for calling name");
                break;
            case 0x82:
-               printf("Called name not present\n");
+               printf("Called name not present");
                break;
            case 0x83:
-               printf("Called name present, but insufficient resources\n");
+               printf("Called name present, but insufficient resources");
                break;
            default:
-               printf("Unspecified error 0x%X\n", ecode);
+               printf("Unspecified error 0x%X", ecode);
                break;
            }
          }
            break;
 
        case 0x85:
-           data = fdata(data, "NBT Session Keepalive\nFlags=[rW]\n", maxbuf);
+           printf("Session Keepalive");
            break;
 
        default:
-           printf("flags=0x%x\n", flags);
-           data = fdata(data, "NBT - Unknown packet type\nType=[rW]\n", maxbuf);
+           data = smb_fdata(data, "Unknown packet type [rB]", maxbuf, 0);
+           break;
+       }
+    } else {
+       printf ("\n>>> NBT Session Packet\n");
+       switch (type) {
+       case 0x00:
+           data = smb_fdata(data, "[P1]NBT Session Message\nFlags=[B]\nLength=[rd]\n",
+               data + 4, 0);
+           if (data == NULL)
+               break;
+           if (nbt_len >= 4 && caplen >= 4 && memcmp(data,"\377SMB",4) == 0) {
+               if ((int)nbt_len > caplen) {
+                   if ((int)nbt_len > length)
+                       printf("WARNING: Packet is continued in later TCP segments\n");
+                   else
+                       printf("WARNING: Short packet. Try increasing the snap length by %d\n",
+                           nbt_len - caplen);
+               }
+               print_smb(data, maxbuf > data + nbt_len ? data + nbt_len : maxbuf);
+           } else
+               printf("Session packet:(raw data or continuation?)\n");
+           break;
+
+       case 0x81:
+           data = smb_fdata(data,
+               "[P1]NBT Session Request\nFlags=[B]\nLength=[rd]\nDestination=[n1]\nSource=[n1]\n",
+               maxbuf, 0);
+           break;
+
+       case 0x82:
+           data = smb_fdata(data, "[P1]NBT Session Granted\nFlags=[B]\nLength=[rd]\n", maxbuf, 0);
+           break;
+
+       case 0x83:
+         {
+           const u_char *origdata;
+           int ecode;
+
+           origdata = data;
+           data = smb_fdata(data, "[P1]NBT SessionReject\nFlags=[B]\nLength=[rd]\nReason=[B]\n",
+               maxbuf, 0);
+           if (data == NULL)
+               break;
+           if (nbt_len >= 1 && caplen >= 1) {
+               ecode = origdata[4];
+               switch (ecode) {
+               case 0x80:
+                   printf("Not listening on called name\n");
+                   break;
+               case 0x81:
+                   printf("Not listening for calling name\n");
+                   break;
+               case 0x82:
+                   printf("Called name not present\n");
+                   break;
+               case 0x83:
+                   printf("Called name present, but insufficient resources\n");
+                   break;
+               default:
+                   printf("Unspecified error 0x%X\n", ecode);
+                   break;
+               }
+           }
+         }
+           break;
+
+       case 0x85:
+           data = smb_fdata(data, "[P1]NBT Session Keepalive\nFlags=[B]\nLength=[rd]\n", maxbuf, 0);
+           break;
+
+       default:
+           data = smb_fdata(data, "NBT - Unknown packet type\nType=[B]\n", maxbuf, 0);
+           break;
+       }
+       printf("\n");
+       fflush(stdout);
     }
-    printf("\n");
-    fflush(stdout);
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
 
@@ -892,22 +1084,25 @@ nbt_tcp_print(const uchar *data, int length)
  * print a NBT packet received across udp on port 137
  */
 void
-nbt_udp137_print(const uchar *data, int length)
+nbt_udp137_print(const u_char *data, int length)
 {
-    const uchar *maxbuf = data + length;
-    int name_trn_id = RSVAL(data, 0);
-    int response = (CVAL(data, 2) >> 7);
-    int opcode = (CVAL(data, 2) >> 3) & 0xF;
-    int nm_flags = ((CVAL(data, 2) & 0x7) << 4) + (CVAL(data, 3) >> 4);
-    int rcode = CVAL(data, 3) & 0xF;
-    int qdcount = RSVAL(data, 4);
-    int ancount = RSVAL(data, 6);
-    int nscount = RSVAL(data, 8);
-    int arcount = RSVAL(data, 10);
-    char *opcodestr;
-    const char *p;
+    const u_char *maxbuf = data + length;
+    int name_trn_id, response, opcode, nm_flags, rcode;
+    int qdcount, ancount, nscount, arcount;
+    const char *opcodestr;
+    const u_char *p;
     int total, i;
 
+    TCHECK2(data[10], 2);
+    name_trn_id = EXTRACT_16BITS(data);
+    response = (data[2] >> 7);
+    opcode = (data[2] >> 3) & 0xF;
+    nm_flags = ((data[2] & 0x7) << 4) + (data[3] >> 4);
+    rcode = data[3] & 0xF;
+    qdcount = EXTRACT_16BITS(data + 4);
+    ancount = EXTRACT_16BITS(data + 6);
+    nscount = EXTRACT_16BITS(data + 8);
+    arcount = EXTRACT_16BITS(data + 10);
     startbuf = data;
 
     if (maxbuf <= data)
@@ -964,12 +1159,13 @@ nbt_udp137_print(const uchar *data, int length)
 
     if (qdcount) {
        printf("QuestionRecords:\n");
-       for (i = 0; i < qdcount; i++)
-           p = fdata(p,
+       for (i = 0; i < qdcount; i++) {
+           p = smb_fdata(p,
                "|Name=[n1]\nQuestionType=[rw]\nQuestionClass=[rw]\n#",
-               maxbuf);
-       if (p == NULL)
-           goto out;
+               maxbuf, 0);
+           if (p == NULL)
+               goto out;
+       }
     }
 
     if (total) {
@@ -978,28 +1174,34 @@ nbt_udp137_print(const uchar *data, int length)
            int rdlen;
            int restype;
 
-           p = fdata(p, "Name=[n1]\n#", maxbuf);
+           p = smb_fdata(p, "Name=[n1]\n#", maxbuf, 0);
            if (p == NULL)
                goto out;
-           restype = RSVAL(p, 0);
-           p = fdata(p, "ResType=[rw]\nResClass=[rw]\nTTL=[rD]\n", p + 8);
+           restype = EXTRACT_16BITS(p);
+           p = smb_fdata(p, "ResType=[rw]\nResClass=[rw]\nTTL=[rD]\n", p + 8, 0);
            if (p == NULL)
                goto out;
-           rdlen = RSVAL(p, 0);
+           rdlen = EXTRACT_16BITS(p);
            printf("ResourceLength=%d\nResourceData=\n", rdlen);
            p += 2;
            if (rdlen == 6) {
-               p = fdata(p, "AddrType=[rw]\nAddress=[b.b.b.b]\n", p + rdlen);
+               p = smb_fdata(p, "AddrType=[rw]\nAddress=[b.b.b.b]\n", p + rdlen, 0);
                if (p == NULL)
                    goto out;
            } else {
                if (restype == 0x21) {
-                   int numnames = CVAL(p, 0);
-                   p = fdata(p, "NumNames=[B]\n", p + 1);
+                   int numnames;
+
+                   TCHECK(*p);
+                   numnames = p[0];
+                   p = smb_fdata(p, "NumNames=[B]\n", p + 1, 0);
                    if (p == NULL)
                        goto out;
                    while (numnames--) {
-                       p = fdata(p, "Name=[n2]\t#", maxbuf);
+                       p = smb_fdata(p, "Name=[n2]\t#", maxbuf, 0);
+                       if (p == NULL)
+                           goto out;
+                       TCHECK(*p);
                        if (p[0] & 0x80)
                            printf("<GROUP> ");
                        switch (p[0] & 0x60) {
@@ -1020,30 +1222,74 @@ nbt_udp137_print(const uchar *data, int length)
                        p += 2;
                    }
                } else {
-                   print_data(p, min(rdlen, length - ((const uchar *)p - data)));
+                   print_data(p, min(rdlen, length - (p - data)));
                    p += rdlen;
                }
            }
        }
     }
 
-    if ((uchar*)p < maxbuf)
-       fdata(p, "AdditionalData:\n", maxbuf);
+    if (p < maxbuf)
+       smb_fdata(p, "AdditionalData:\n", maxbuf, 0);
 
 out:
     printf("\n");
     fflush(stdout);
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
+/*
+ * Print an SMB-over-TCP packet received across tcp on port 445
+ */
+void
+smb_tcp_print (const u_char * data, int length)
+{
+    int caplen;
+    u_int smb_len;
+    const u_char *maxbuf;
+
+    if (length < 4)
+       goto trunc;
+    if (snapend < data)
+       goto trunc;
+    caplen = snapend - data;
+    if (caplen < 4)
+       goto trunc;
+    maxbuf = data + caplen;
+    smb_len = EXTRACT_24BITS(data + 1);
+    length -= 4;
+    caplen -= 4;
 
+    startbuf = data;
+    data += 4;
+
+    if (smb_len >= 4 && caplen >= 4 && memcmp(data,"\377SMB",4) == 0) {
+       if ((int)smb_len > caplen) {
+           if ((int)smb_len > length)
+               printf("WARNING: Packet is continued in later TCP segments\n");
+           else
+               printf("WARNING: Short packet. Try increasing the snap length by %d\n",
+                   smb_len - caplen);
+       }
+       print_smb(data, maxbuf > data + smb_len ? data + smb_len : maxbuf);
+    } else
+       printf("SMB-over-TCP packet:(raw data or continuation?)\n");
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
+}
 
 /*
  * print a NBT packet received across udp on port 138
  */
 void
-nbt_udp138_print(const uchar *data, int length)
+nbt_udp138_print(const u_char *data, int length)
 {
-    const uchar *maxbuf = data + length;
+    const u_char *maxbuf = data + length;
 
     if (maxbuf > snapend)
        maxbuf = snapend;
@@ -1056,13 +1302,19 @@ nbt_udp138_print(const uchar *data, int length)
        return;
     }
 
-    data = fdata(data,
+    data = smb_fdata(data,
        "\n>>> NBT UDP PACKET(138) Res=[rw] ID=[rw] IP=[b.b.b.b] Port=[rd] Length=[rd] Res2=[rw]\nSourceName=[n1]\nDestName=[n1]\n#",
-       maxbuf);
+       maxbuf, 0);
 
-    if (data != NULL)
-       print_smb(data, maxbuf);
+    if (data != NULL) {
+       /* If there isn't enough data for "\377SMB", don't check for it. */
+       if (&data[3] >= maxbuf)
+           goto out;
 
+       if (memcmp(data, "\377SMB",4) == 0)
+           print_smb(data, maxbuf);
+    }
+out:
     printf("\n");
     fflush(stdout);
 }
@@ -1071,21 +1323,75 @@ nbt_udp138_print(const uchar *data, int length)
 /*
    print netbeui frames
 */
+struct nbf_strings {
+       const char      *name;
+       const char      *nonverbose;
+       const char      *verbose;
+} nbf_strings[0x20] = {
+       { "Add Group Name Query", ", [P23]Name to add=[n2]#",
+         "[P5]ResponseCorrelator=[w]\n[P16]Name to add=[n2]\n" },
+       { "Add Name Query", ", [P23]Name to add=[n2]#",
+         "[P5]ResponseCorrelator=[w]\n[P16]Name to add=[n2]\n" },
+       { "Name In Conflict", NULL, NULL },
+       { "Status Query", NULL, NULL },
+       { NULL, NULL, NULL },   /* not used */
+       { NULL, NULL, NULL },   /* not used */
+       { NULL, NULL, NULL },   /* not used */
+       { "Terminate Trace", NULL, NULL },
+       { "Datagram", NULL,
+         "[P7]Destination=[n2]\nSource=[n2]\n" },
+       { "Broadcast Datagram", NULL,
+         "[P7]Destination=[n2]\nSource=[n2]\n" },
+       { "Name Query", ", [P7]Name=[n2]#",
+         "[P1]SessionNumber=[B]\nNameType=[B][P2]\nResponseCorrelator=[w]\nName=[n2]\nName of sender=[n2]\n" },
+       { NULL, NULL, NULL },   /* not used */
+       { NULL, NULL, NULL },   /* not used */
+       { "Add Name Response", ", [P1]GroupName=[w] [P4]Destination=[n2] Source=[n2]#",
+         "AddNameInProcess=[B]\nGroupName=[w]\nTransmitCorrelator=[w][P2]\nDestination=[n2]\nSource=[n2]\n" },
+       { "Name Recognized", NULL,
+         "[P1]Data2=[w]\nTransmitCorrelator=[w]\nResponseCorelator=[w]\nDestination=[n2]\nSource=[n2]\n" },
+       { "Status Response", NULL, NULL },
+       { NULL, NULL, NULL },   /* not used */
+       { NULL, NULL, NULL },   /* not used */
+       { NULL, NULL, NULL },   /* not used */
+       { "Terminate Trace", NULL, NULL },
+       { "Data Ack", NULL,
+         "[P3]TransmitCorrelator=[w][P2]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n" },
+       { "Data First/Middle", NULL,
+         "Flags=[{RECEIVE_CONTINUE|NO_ACK||PIGGYBACK_ACK_INCLUDED|}]\nResyncIndicator=[w][P2]\nResponseCorelator=[w]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n" },
+       { "Data Only/Last", NULL,
+         "Flags=[{|NO_ACK|PIGGYBACK_ACK_ALLOWED|PIGGYBACK_ACK_INCLUDED|}]\nResyncIndicator=[w][P2]\nResponseCorelator=[w]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n" },
+       { "Session Confirm", NULL,
+         "Data1=[B]\nData2=[w]\nTransmitCorrelator=[w]\nResponseCorelator=[w]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n" },
+       { "Session End", NULL,
+         "[P1]Data2=[w][P4]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n" },
+       { "Session Initialize", NULL,
+         "Data1=[B]\nData2=[w]\nTransmitCorrelator=[w]\nResponseCorelator=[w]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n" },
+       { "No Receive", NULL,
+         "Flags=[{|SEND_NO_ACK}]\nDataBytesAccepted=[b][P4]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n" },
+       { "Receive Outstanding", NULL,
+         "[P1]DataBytesAccepted=[b][P4]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n" },
+       { "Receive Continue", NULL,
+         "[P2]TransmitCorrelator=[w]\n[P2]RemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n" },
+       { NULL, NULL, NULL },   /* not used */
+       { NULL, NULL, NULL },   /* not used */
+       { "Session Alive", NULL, NULL }
+};
+
 void
-netbeui_print(u_short control, const uchar *data, int length)
+netbeui_print(u_short control, const u_char *data, int length)
 {
-    const uchar *maxbuf = data + length;
+    const u_char *maxbuf = data + length;
     int len;
     int command;
-    const uchar *data2;
+    const u_char *data2;
     int is_truncated = 0;
 
     if (maxbuf > snapend)
        maxbuf = snapend;
-    if (&data[7] >= maxbuf)
-       goto out;
-    len = SVAL(data,0);
-    command = CVAL(data,4);
+    TCHECK(data[4]);
+    len = EXTRACT_LE_16BITS(data);
+    command = data[4];
     data2 = data + len;
     if (data2 >= maxbuf) {
        data2 = maxbuf;
@@ -1095,69 +1401,37 @@ netbeui_print(u_short control, const uchar *data, int length)
     startbuf = data;
 
     if (vflag < 2) {
-       printf("NetBeui Packet");
-       return;
+       printf("NBF Packet: ");
+       data = smb_fdata(data, "[P5]#", maxbuf, 0);
+    } else {
+       printf("\n>>> NBF Packet\nType=0x%X ", control);
+       data = smb_fdata(data, "Length=[d] Signature=[w] Command=[B]\n#", maxbuf, 0);
     }
-
-    printf("\n>>> NetBeui Packet\nType=0x%X ", control);
-    data = fdata(data, "Length=[d] Signature=[w] Command=[B]\n#", maxbuf);
     if (data == NULL)
        goto out;
 
-    switch (command) {
-    case 0xA:
-       data = fdata(data, "NameQuery:[P1]\nSessionNumber=[B]\nNameType=[B][P2]\nResponseCorrelator=[w]\nDestination=[n2]\nSource=[n2]\n", data2);
-       break;
-
-    case 0x8:
-       data = fdata(data,
-           "NetbiosDataGram:[P7]\nDestination=[n2]\nSource=[n2]\n", data2);
-       break;
-
-    case 0xE:
-       data = fdata(data,
-           "NameRecognise:\n[P1]\nData2=[w]\nTransmitCorrelator=[w]\nResponseCorelator=[w]\nDestination=[n2]\nSource=[n2]\n",
-           data2);
-       break;
-
-    case 0x19:
-       data = fdata(data,
-           "SessionInitialise:\nData1=[B]\nData2=[w]\nTransmitCorrelator=[w]\nResponseCorelator=[w]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n",
-           data2);
-       break;
-
-    case 0x17:
-       data = fdata(data,
-           "SessionConfirm:\nData1=[B]\nData2=[w]\nTransmitCorrelator=[w]\nResponseCorelator=[w]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n",
-           data2);
-       break;
-
-    case 0x16:
-       data = fdata(data,
-           "NetbiosDataOnlyLast:\nFlags=[{|NO_ACK|PIGGYBACK_ACK_ALLOWED|PIGGYBACK_ACK_INCLUDED|}]\nResyncIndicator=[w][P2]\nResponseCorelator=[w]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n",
-           data2);
-       break;
-
-    case 0x14:
-       data = fdata(data,
-           "NetbiosDataAck:\n[P3]TransmitCorrelator=[w][P2]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n",
-           data2);
-       break;
-
-    case 0x18:
-       data = fdata(data,
-           "SessionEnd:\n[P1]Data2=[w][P4]\nRemoteSessionNumber=[B]\nLocalSessionNumber=[B]\n",
-           data2);
-       break;
+    if (command > 0x1f || nbf_strings[command].name == NULL) {
+       if (vflag < 2)
+           data = smb_fdata(data, "Unknown NBF Command#", data2, 0);
+       else
+           data = smb_fdata(data, "Unknown NBF Command\n", data2, 0);
+    } else {
+       if (vflag < 2) {
+           printf("%s", nbf_strings[command].name);
+           if (nbf_strings[command].nonverbose != NULL)
+               data = smb_fdata(data, nbf_strings[command].nonverbose, data2, 0);
+       } else {
+           printf("%s:\n", nbf_strings[command].name);
+           if (nbf_strings[command].verbose != NULL)
+               data = smb_fdata(data, nbf_strings[command].verbose, data2, 0);
+           else
+               printf("\n");
+       }
+    }
 
-    case 0x1f:
-       data = fdata(data, "SessionAlive\n", data2);
-       break;
+    if (vflag < 2)
+       return;
 
-    default:
-       data = fdata(data, "Unknown Netbios Command ", data2);
-       break;
-    }
     if (data == NULL)
        goto out;
 
@@ -1166,6 +1440,11 @@ netbeui_print(u_short control, const uchar *data, int length)
        goto out;
     }
 
+    /* If this isn't a command that would contain an SMB message, quit. */
+    if (command != 0x08 && command != 0x09 && command != 0x15 &&
+        command != 0x16)
+       goto out;
+
     /* If there isn't enough data for "\377SMB", don't look for it. */
     if (&data2[3] >= maxbuf)
        goto out;
@@ -1187,6 +1466,10 @@ netbeui_print(u_short control, const uchar *data, int length)
 
 out:
     printf("\n");
+    return;
+trunc:
+    printf("%s", tstr);
+    return;
 }
 
 
@@ -1194,14 +1477,14 @@ out:
  * print IPX-Netbios frames
  */
 void
-ipx_netbios_print(const uchar *data, u_int length)
+ipx_netbios_print(const u_char *data, u_int length)
 {
     /*
      * this is a hack till I work out how to parse the rest of the
      * NetBIOS-over-IPX stuff
      */
     int i;
-    const uchar *maxbuf;
+    const u_char *maxbuf;
 
     maxbuf = data + length;
     /* Don't go past the end of the captured data in the packet. */
@@ -1212,14 +1495,13 @@ ipx_netbios_print(const uchar *data, u_int length)
        if (&data[i + 4] > maxbuf)
            break;
        if (memcmp(&data[i], "\377SMB", 4) == 0) {
-           fdata(data, "\n>>> IPX transport ", &data[i]);
-           if (data != NULL)
-               print_smb(&data[i], maxbuf);
+           smb_fdata(data, "\n>>> IPX transport ", &data[i], 0);
+           print_smb(&data[i], maxbuf);
            printf("\n");
            fflush(stdout);
            break;
        }
     }
     if (i == 128)
-       fdata(data, "\n>>> Unknown IPX ", maxbuf);
+       smb_fdata(data, "\n>>> Unknown IPX ", maxbuf, 0);
 }