file_seek() used to be a wrapper around fseek() or gzseek(), both of
[metze/wireshark/wip.git] / wiretap / vms.c
index a77dfa453a92c6f4ec668881f38a59779a5e4f0f..8173e576cf07a8c734044af4ac434c99be54640a 100644 (file)
@@ -1,9 +1,9 @@
 /* vms.c
  *
- * $Id: vms.c,v 1.12 2002/03/25 21:15:54 guy Exp $
+ * $Id$
  *
  * Wiretap Library
- * Copyright (c) 2001 by Marc Milgram <mmilgram@arrayinc.com>
+ * Copyright (c) 2001 by Marc Milgram <ethereal@mmilgram.NOSPAMmail.net>
  *
  * This program is free software; you can redistribute it and/or
  * modify it under the terms of the GNU General Public License
 #include <string.h>
 #include <ctype.h>
 
-/* This module reads the output of the 'TCPIPTRACE' command in VMS
- * It was initially based on toshiba.c.
- */
+/* This module reads the output of the various VMS TCPIP trace utilities
+ * such as TCPIPTRACE, TCPTRACE and UCX$TRACE
+ *
+ * It was initially based on toshiba.c and refined with code from cosine.c
+
+--------------------------------------------------------------------------------
+   Example TCPIPTRACE TCPTRACE output data:
 
-/*
-   Example 'TCPIPTRACE' output data:
    TCPIPtrace full display RCV packet 8 at 10-JUL-2001 14:54:19.56
 
    IP Version = 4,  IHL = 5,  TOS = 00,   Total Length = 84 = ^x0054
    00000000   00000000   00000000   03000000    0030    ................
    06000000   01000000   A5860100   00000000    0040    ................
                                     00000000    0050    ....
+--------------------------------------------------------------------------------
+
+   Example UCX$TRACE output data:
+
+    UCX INTERnet trace RCV packet seq # = 1 at 14-MAY-2003 11:32:10.93 
+
+   IP Version = 4,  IHL = 5,  TOS = 00,   Total Length = 583 = ^x0247 
+   IP Identifier  = ^x702E,  Flags (0=0,DF=0,MF=0),  
+         Fragment Offset = 0 = ^x0000,   Calculated Offset = 0 = ^x0000 
+   IP TTL = 128 = ^x80,  Protocol = 17 = ^x11,  Header Checksum = ^x70EC 
+   IP Source Address      = 10.20.4.159 
+   IP Destination Address = 10.20.4.255 
+
+   UDP Source Port = 138,   UDP Destination Port = 138 
+   UDP Header and Datagram Length = 563 = ^x0233,   Checksum = ^xB913 
+
+   9F04140A   70EC1180   0000702E   47020045    0000    E..G.p.....p....
+   B1B80E11 | B9133302   8A008A00 | FF04140A    0010    .........3......
+   46484648   45200000   1D028A00   9F04140A    0020    ...........EHFHF
+   43414341   4341434D   454D4546   45454550    0030    PEEEFEMEMCACACAC
+
+--------------------------------------------------------------------------------
+
+   Alternate UCX$TRACE type output data:
+
+   TCPIP INTERnet trace RCV packet seq # = 1 at 23-OCT-1998 15:19:33.29
+
+   IP Version = 4,  IHL = 5,  TOS = 00,   Total Length = 217 = ^x00D9
+   IP Identifier  = ^x0065,  Flags (0=0,DF=0,MF=0),
+         Fragment Offset = 0 = ^x0000,   Calculated Offset = 0 = ^x0000
+   IP TTL = 32 = ^x20,  Protocol = 17 = ^x11,  Header Checksum = ^x8F6C
+   IP Source Address      = 16.20.168.93
+   IP Destination Address = 16.20.255.255
+
+   UDP Source Port = 138,   UDP Destination Port = 138
+   UDP Header and Datagram Length = 197 = ^x00C5,   Checksum = ^x0E77
+
+   5DA81410   8F6C1120   00000065   D9000045    0000    E...awe.....l....]
+            | 0E77C500   8A008A00 | FFFF1410    0010    ..........w.
+
+--------------------------------------------------------------------------------
+
+The only difference between the utilities is the Packet header line, primarily
+the utility identifier and the packet sequence formats.
+
+There appear to be 2 formats for packet seqencing
+
+Format 1:
+
+ ... packet nn at DD-MMM-YYYY hh:mm:ss.ss
+
+Format 2:
+
+ ... packet seq # = nn at DD-MMM-YYYY hh:mm:ss.ss
+
+If there are other formats then code will have to be written in parse_vms_rec_hdr()
+to handle them.
 
 --------------------------------------------------------------------------------
 
  */
 
-/* Magic text to check for VMS-ness of file */
-static const char vms_hdr_magic[]  =
-{ 'T', 'C', 'P', 'I', 'P', 't', 'r', 'a', 'c', 'e', ' '};
-#define VMS_HDR_MAGIC_SIZE  (sizeof vms_hdr_magic  / sizeof vms_hdr_magic[0])
+/* Magic text to check for VMS-ness of file using possible utility names
+ *
+ */
+#define VMS_HDR_MAGIC_STR1     "TCPIPtrace"
+#define VMS_HDR_MAGIC_STR2     "TCPtrace"
+#define VMS_HDR_MAGIC_STR3     "INTERnet trace"
 
 /* Magic text for start of packet */
-#define vms_rec_magic vms_hdr_magic
-#define VMS_REC_MAGIC_SIZE  (sizeof vms_rec_magic  / sizeof vms_rec_magic[0])
+#define VMS_REC_MAGIC_STR1     VMS_HDR_MAGIC_STR1
+#define VMS_REC_MAGIC_STR2     VMS_HDR_MAGIC_STR2
+#define VMS_REC_MAGIC_STR3     VMS_HDR_MAGIC_STR3
+
+#define VMS_HEADER_LINES_TO_CHECK    200
+#define VMS_LINE_LENGTH              240
 
-static gboolean vms_read(wtap *wth, int *err, long *data_offset);
-static gboolean vms_seek_read(wtap *wth, long seek_off,
-    union wtap_pseudo_header *pseudo_header, guint8 *pd, int len, int *err);
+static gboolean vms_read(wtap *wth, int *err, gchar **err_info,
+    gint64 *data_offset);
+static gboolean vms_seek_read(wtap *wth, gint64 seek_off,
+    union wtap_pseudo_header *pseudo_header, guint8 *pd, int len,
+    int *err, gchar **err_info);
 static gboolean parse_single_hex_dump_line(char* rec, guint8 *buf,
     long byte_offset, int in_off, int remaining_bytes);
 static gboolean parse_vms_hex_dump(FILE_T fh, int pkt_len, guint8* buf,
-    int *err);
-static int parse_vms_rec_hdr(wtap *wth, FILE_T fh, int *err);
-
+    int *err, gchar **err_info);
+static int parse_vms_rec_hdr(wtap *wth, FILE_T fh, int *err, gchar **err_info);
 
 #ifdef TCPIPTRACE_FRAGMENTS_HAVE_HEADER_LINE
 /* Seeks to the beginning of the next packet, and returns the
    byte offset.  Returns -1 on failure, and sets "*err" to the error. */
 static long vms_seek_next_packet(wtap *wth, int *err)
 {
-  int byte;
-  unsigned int level = 0;
   long cur_off;
-
-  while ((byte = file_getc(wth->fh)) != EOF) {
-    if ((level == 3) && (byte != vms_rec_magic[level]))
-      level += 2;  /* Accept TCPtrace as well as TCPIPtrace */
-    if (byte == vms_rec_magic[level]) {
-      level++;
-      if (level >= VMS_REC_MAGIC_SIZE) {
-        /* note: we're leaving file pointer right after the magic characters */
-        cur_off = file_tell(wth->fh);
-        if (cur_off == -1) {
-          /* Error. */
-          *err = file_error(wth->fh);
-          return -1;
-        }
-        return cur_off + 1;
+  char buf[VMS_LINE_LENGTH];
+  
+  while (1) {
+    cur_off = file_tell(wth->fh);
+    if (cur_off == -1) {
+      /* Error */
+      *err = file_error(wth->fh, err_info);
+      hdr = NULL;
+      return -1;
+    }
+    if (file_gets(buf, sizeof(buf), wth->fh) != NULL) {
+      if (strstr(buf, VMS_REC_MAGIC_STR1) ||
+         strstr(buf, VMS_REC_MAGIC_STR2) ||
+         strstr(buf, VMS_REC_MAGIC_STR2)) {
+                 g_strlcpy(hdr, buf,VMS_LINE_LENGTH);
+                 return cur_off;
       }
     } else {
-      level = 0;
+      if (file_eof(wth->fh)) {
+       /* We got an EOF. */
+       *err = 0;
+      } else {
+       /* We (presumably) got an error (there's no
+          equivalent to "ferror()" in zlib, alas,
+          so we don't have a wrapper to check for
+          an error). */
+       *err = file_error(wth->fh, err_info);
+      }
+      break;
     }
   }
-  if (file_eof(wth->fh)) {
-    /* We got an EOF. */
-    *err = 0;
-  } else {
-    /* We (presumably) got an error (there's no equivalent to "ferror()"
-       in zlib, alas, so we don't have a wrapper to check for an error). */
-    *err = file_error(wth->fh);
-  }
+  hdr = NULL;
   return -1;
 }
 #endif /* TCPIPTRACE_FRAGMENTS_HAVE_HEADER_LINE */
 
-#define VMS_HEADER_LINES_TO_CHECK    200
-#define VMS_LINE_LENGTH        240
-
 /* Look through the first part of a file to see if this is
  * a VMS trace file.
  *
  * Returns TRUE if it is, FALSE if it isn't or if we get an I/O error;
- * if we get an I/O error, "*err" will be set to a non-zero value.
+ * if we get an I/O error, "*err" will be set to a non-zero value and
+ * "*err_info will be set to null or an additional error string.
  *
  * Leaves file handle at begining of line that contains the VMS Magic
  * identifier.
  */
-static gboolean vms_check_file_type(wtap *wth, int *err)
+static gboolean vms_check_file_type(wtap *wth, int *err, gchar **err_info)
 {
-    char    buf[VMS_LINE_LENGTH];
-    int    line, byte;
-    unsigned int reclen, i, level;
-    long mpos;
-   
-    buf[VMS_LINE_LENGTH-1] = 0;
-
-    for (line = 0; line < VMS_HEADER_LINES_TO_CHECK; line++) {
-        mpos = file_tell(wth->fh);
-        if (mpos == -1) {
-            /* Error. */
-            *err = file_error(wth->fh);
-            return FALSE;
-        }
-        if (file_gets(buf, VMS_LINE_LENGTH, wth->fh) != NULL) {
-
-            reclen = strlen(buf);
-            if (reclen < VMS_HDR_MAGIC_SIZE)
-                continue;
-
-            level = 0;
-            for (i = 0; i < reclen; i++) {
-                byte = buf[i];
-               if ((level == 3) && (byte != vms_hdr_magic[level]))
-                   level += 2; /* Accept TCPIPtrace as well as TCPtrace */
-                if (byte == vms_hdr_magic[level]) {
-                    level++;
-                    if (level >= VMS_HDR_MAGIC_SIZE) {
-                        if (file_seek(wth->fh, mpos, SEEK_SET) == -1) {
-                            /* Error. */
-                            *err = file_error(wth->fh);
-                            return FALSE;
-                        }
-                        return TRUE;
-                    }
-                }
-                else
-                    level = 0;
-            }
-        }
-        else {
-            /* EOF or error. */
-            if (file_eof(wth->fh))
-                *err = 0;
-            else
-                *err = file_error(wth->fh);
-            return FALSE;
-        }
+  char buf[VMS_LINE_LENGTH];
+  guint        reclen, line;
+  gint64 mpos;
+  
+  buf[VMS_LINE_LENGTH-1] = '\0';
+  
+  for (line = 0; line < VMS_HEADER_LINES_TO_CHECK; line++) {
+    mpos = file_tell(wth->fh);
+    if (mpos == -1) {
+      /* Error. */
+      *err = file_error(wth->fh, err_info);
+      return FALSE;
+    }
+    if (file_gets(buf, VMS_LINE_LENGTH, wth->fh) != NULL) {
+      
+      reclen = (guint) strlen(buf);
+      if (reclen < strlen(VMS_HDR_MAGIC_STR1) ||
+         reclen < strlen(VMS_HDR_MAGIC_STR2) || 
+         reclen < strlen(VMS_HDR_MAGIC_STR3)) {
+       continue;
+      }
+      
+      if (strstr(buf, VMS_HDR_MAGIC_STR1) ||
+         strstr(buf, VMS_HDR_MAGIC_STR2) ||
+         strstr(buf, VMS_HDR_MAGIC_STR3)) {
+       /* Go back to the beginning of this line, so we will
+        * re-read it. */
+       if (file_seek(wth->fh, mpos, SEEK_SET, err) == -1) {
+         /* Error. */
+         return FALSE;
+       }
+       return TRUE;
+      }
+    } else {
+      /* EOF or error. */
+      if (file_eof(wth->fh))
+       *err = 0;
+      else
+       *err = file_error(wth->fh, err_info);
+      return FALSE;
     }
-    *err = 0;
-    return FALSE;
+  }
+  *err = 0;
+  return FALSE;
 }
 
 
-int vms_open(wtap *wth, int *err)
+int vms_open(wtap *wth, int *err, gchar **err_info)
 {
     /* Look for VMS header */
-    if (!vms_check_file_type(wth, err)) {
+    if (!vms_check_file_type(wth, err, err_info)) {
         if (*err == 0)
             return 0;
         else
             return -1;
     }
 
-    wth->data_offset = 0;
     wth->file_encap = WTAP_ENCAP_RAW_IP;
     wth->file_type = WTAP_FILE_VMS;
     wth->snapshot_length = 0; /* not known */
     wth->subtype_read = vms_read;
     wth->subtype_seek_read = vms_seek_read;
+       wth->tsprecision = WTAP_FILE_TSPREC_CSEC;
 
     return 1;
 }
 
-/* Find the next packet and parse it; called from wtap_loop(). */
-static gboolean vms_read(wtap *wth, int *err, long *data_offset)
+/* Find the next packet and parse it; called from wtap_read(). */
+static gboolean vms_read(wtap *wth, int *err, gchar **err_info,
+    gint64 *data_offset)
 {
-    long   offset = 0;
+    gint64   offset = 0;
     guint8    *buf;
     int    pkt_len;
 
@@ -229,11 +289,13 @@ static gboolean vms_read(wtap *wth, int *err, long *data_offset)
 #else
     offset = file_tell(wth->fh);
 #endif
-    if (offset < 1)
+    if (offset < 1) {
+        *err = file_error(wth->fh, err_info);
         return FALSE;
+    }
 
     /* Parse the header */
-    pkt_len = parse_vms_rec_hdr(wth, wth->fh, err);
+    pkt_len = parse_vms_rec_hdr(wth, wth->fh, err, err_info);
     if (pkt_len == -1)
        return FALSE;
 
@@ -242,36 +304,36 @@ static gboolean vms_read(wtap *wth, int *err, long *data_offset)
     buf = buffer_start_ptr(wth->frame_buffer);
 
     /* Convert the ASCII hex dump to binary data */
-    if (!parse_vms_hex_dump(wth->fh, pkt_len, buf, err))
+    if (!parse_vms_hex_dump(wth->fh, pkt_len, buf, err, err_info))
         return FALSE;
 
-    wth->data_offset = offset;
     *data_offset = offset;
     return TRUE;
 }
 
 /* Used to read packets in random-access fashion */
 static gboolean
-vms_seek_read (wtap *wth, long seek_off,
+vms_seek_read (wtap *wth, gint64 seek_off,
     union wtap_pseudo_header *pseudo_header _U_,
-    guint8 *pd, int len, int *err)
+    guint8 *pd, int len, int *err, gchar **err_info)
 {
     int    pkt_len;
 
-    if (file_seek(wth->random_fh, seek_off - 1, SEEK_SET) == -1) {
-        *err = file_error(wth->random_fh);
+    if (file_seek(wth->random_fh, seek_off - 1, SEEK_SET, err) == -1)
         return FALSE;
-    }
 
-    pkt_len = parse_vms_rec_hdr(NULL, wth->random_fh, err);
+    pkt_len = parse_vms_rec_hdr(NULL, wth->random_fh, err, err_info);
 
     if (pkt_len != len) {
-        if (pkt_len != -1)
-            *err = WTAP_ERR_BAD_RECORD;
+        if (pkt_len != -1) {
+            *err = WTAP_ERR_BAD_FILE;
+            *err_info = g_strdup_printf("vms: requested length %d doesn't match length %d",
+                len, pkt_len);
+        }
         return FALSE;
     }
 
-    return parse_vms_hex_dump(wth->random_fh, pkt_len, pd, err);
+    return parse_vms_hex_dump(wth->random_fh, pkt_len, pd, err, err_info);
 }
 
 /* isdumpline assumes that dump lines start with some non-alphanumerics
@@ -279,16 +341,16 @@ vms_seek_read (wtap *wth, long seek_off,
  * by 3 spaces.
  */
 static int
-isdumpline( guchar *line )
+isdumpline( gchar *line )
 {
     int i, j;
 
-    while (*line && !isalnum(*line))
+    while (*line && !isalnum((guchar)*line))
        line++;
 
     for (j=0; j<4; j++) {
        for (i=0; i<8; i++, line++)
-           if (! isxdigit(*line))
+           if (! isxdigit((guchar)*line))
                return FALSE;
 
        for (i=0; i<3; i++, line++)
@@ -296,33 +358,34 @@ isdumpline( guchar *line )
                return FALSE;
     }
 
-    return isspace(*line);
+    return isspace((guchar)*line);
 }
 
 /* Parses a packet record header. */
 static int
-parse_vms_rec_hdr(wtap *wth, FILE_T fh, int *err)
+parse_vms_rec_hdr(wtap *wth, FILE_T fh, int *err, gchar **err_info)
 {
     char   line[VMS_LINE_LENGTH + 1];
     int    num_items_scanned;
     int           pkt_len = 0;
     int           pktnum;
     int           csec = 101;
-    struct tm time;
+    struct tm tm;
     char mon[4] = {'J', 'A', 'N', 0};
-    guchar *p;
-    static guchar months[] = "JANFEBMARAPRMAYJUNJULAUGSEPOCTNOVDEC";
-
-    time.tm_year = 1970;
-    time.tm_hour = 1;
-    time.tm_min = 1;
-    time.tm_sec = 1;
+    gchar *p;
+    static const gchar months[] = "JANFEBMARAPRMAYJUNJULAUGSEPOCTNOVDEC";
 
+    tm.tm_year = 1970;
+    tm.tm_mon = 0;
+    tm.tm_mday = 1;
+    tm.tm_hour = 1;
+    tm.tm_min = 1;
+    tm.tm_sec = 1;
 
     /* Skip lines until one starts with a hex number */
     do {
         if (file_gets(line, VMS_LINE_LENGTH, fh) == NULL) {
-            *err = file_error(fh);
+            *err = file_error(fh, err_info);
            if ((*err == 0) && (csec != 101)) {
                *err = WTAP_ERR_SHORT_READ;
             }
@@ -330,27 +393,40 @@ parse_vms_rec_hdr(wtap *wth, FILE_T fh, int *err)
         }
        line[VMS_LINE_LENGTH] = '\0';
 
-       if ((csec == 101) && (p = strstr(line, "packet "))
+       if ((csec == 101) && (p = strstr(line, "packet ")) != NULL
            && (! strstr(line, "could not save "))) {
            /* Find text in line starting with "packet ". */
-           num_items_scanned = sscanf(p,
-                                      "packet %d at %d-%3s-%d %d:%d:%d.%d",
-                                      &pktnum, &time.tm_mday, mon,
-                                      &time.tm_year, &time.tm_hour,
-                                      &time.tm_min, &time.tm_sec, &csec);
 
+           /* First look for the Format 1 type sequencing */
+           num_items_scanned = sscanf(p,  
+                                      "packet %9d at %2d-%3s-%4d %2d:%2d:%2d.%9d",
+                                      &pktnum, &tm.tm_mday, mon,
+                                      &tm.tm_year, &tm.tm_hour,
+                                      &tm.tm_min, &tm.tm_sec, &csec);
+           /* Next look for the Format 2 type sequencing */
            if (num_items_scanned != 8) {
-               *err = WTAP_ERR_BAD_RECORD;
+             num_items_scanned = sscanf(p,
+                                        "packet seq # = %9d at %2d-%3s-%4d %2d:%2d:%2d.%9d",
+                                        &pktnum, &tm.tm_mday, mon,
+                                        &tm.tm_year, &tm.tm_hour,
+                                        &tm.tm_min, &tm.tm_sec, &csec);
+           }
+           /* if unknown format then exit with error        */
+           /* We will need to add code to handle new format */
+           if (num_items_scanned != 8) {
+               *err = WTAP_ERR_BAD_FILE;
+               *err_info = g_strdup_printf("vms: header line not valid");
                return -1;
            }
        }
         if ( (! pkt_len) && (p = strstr(line, "Length"))) {
             p += sizeof("Length ");
-            while (*p && ! isdigit(*p))
+            while (*p && ! isdigit((guchar)*p))
                 p++;
 
             if ( !*p ) {
-                *err = WTAP_ERR_BAD_RECORD;
+                *err = WTAP_ERR_BAD_FILE;
+               *err_info = g_strdup_printf("vms: Length field not valid");
                 return -1;
             }
 
@@ -362,15 +438,15 @@ parse_vms_rec_hdr(wtap *wth, FILE_T fh, int *err)
     if (wth) {
         p = strstr(months, mon);
         if (p)
-            time.tm_mon = (p - months) / 3;
-        time.tm_year -= 1900;
+            tm.tm_mon = (int) (p - months) / 3;
+        tm.tm_year -= 1900;
 
-        wth->phdr.ts.tv_sec = mktime(&time);
-
-        wth->phdr.ts.tv_usec = csec * 10000;
+       tm.tm_isdst = -1;
+        wth->phdr.presence_flags = WTAP_HAS_TS;
+        wth->phdr.ts.secs = mktime(&tm);
+        wth->phdr.ts.nsecs = csec * 10000000;
         wth->phdr.caplen = pkt_len;
         wth->phdr.len = pkt_len;
-        wth->phdr.pkt_encap = WTAP_ENCAP_RAW_IP;
     }
 
     return pkt_len;
@@ -378,15 +454,16 @@ parse_vms_rec_hdr(wtap *wth, FILE_T fh, int *err)
 
 /* Converts ASCII hex dump to binary data */
 static gboolean
-parse_vms_hex_dump(FILE_T fh, int pkt_len, guint8* buf, int *err)
+parse_vms_hex_dump(FILE_T fh, int pkt_len, guint8* buf, int *err,
+    gchar **err_info)
 {
-    guchar line[VMS_LINE_LENGTH + 1];
+    gchar line[VMS_LINE_LENGTH + 1];
     int    i;
     int    offset = 0;
 
     for (i = 0; i < pkt_len; i += 16) {
         if (file_gets(line, VMS_LINE_LENGTH, fh) == NULL) {
-            *err = file_error(fh);
+            *err = file_error(fh, err_info);
             if (*err == 0) {
                 *err = WTAP_ERR_SHORT_READ;
             }
@@ -396,7 +473,7 @@ parse_vms_hex_dump(FILE_T fh, int pkt_len, guint8* buf, int *err)
         if (i == 0) {
            while (! isdumpline(line)) { /* advance to start of hex data */
                if (file_gets(line, VMS_LINE_LENGTH, fh) == NULL) {
-                   *err = file_error(fh);
+                   *err = file_error(fh, err_info);
                    if (*err == 0) {
                        *err = WTAP_ERR_SHORT_READ;
                    }
@@ -404,18 +481,31 @@ parse_vms_hex_dump(FILE_T fh, int pkt_len, guint8* buf, int *err)
                }
                line[VMS_LINE_LENGTH] = '\0';
            }
-            while (line[offset] && !isxdigit(line[offset]))
+            while (line[offset] && !isxdigit((guchar)line[offset]))
                 offset++;
        }
        if (!parse_single_hex_dump_line(line, buf, i,
                                        offset, pkt_len - i)) {
-            *err = WTAP_ERR_BAD_RECORD;
+            *err = WTAP_ERR_BAD_FILE;
+           *err_info = g_strdup_printf("vms: hex dump not valid");
             return FALSE;
         }
     }
     /* Avoid TCPIPTRACE-W-BUFFERSFUL, TCPIPtrace could not save n packets.
-     * errors. */
-    file_gets(line, VMS_LINE_LENGTH, fh);
+     * errors.
+     *
+     * XXX - when we support packet drop report information in the
+     * Wiretap API, we should parse those lines and return "n" as
+     * a packet drop count. */
+    if (!file_gets(line, VMS_LINE_LENGTH, fh)) {
+        *err = file_error(fh, err_info);
+        if (*err == 0) {
+            /* There is no next line, so there's no "TCPIPtrace could not
+             * save n packets" line; not an error. */
+            return TRUE;
+        }
+        return FALSE;
+    }
     return TRUE;
 }
 
@@ -443,14 +533,14 @@ parse_single_hex_dump_line(char* rec, guint8 *buf, long byte_offset,
     int        i;
     char        *s;
     int        value;
-    static int offsets[16] = {39,37,35,33,28,26,24,22,17,15,13,11,6,4,2,0};
+    static const int offsets[16] = {39,37,35,33,28,26,24,22,17,15,13,11,6,4,2,0};
     char lbuf[3] = {0,0,0};
-   
+
 
     /* Get the byte_offset directly from the record */
     s = rec;
-    value = strtoul(s + 45 + in_off, NULL, 16);
-   
+    value = (int)strtoul(s + 45 + in_off, NULL, 16);   /* XXX - error check? */
+
     if (value != byte_offset) {
         return FALSE;
     }