Refactor 802.11 radio flags.
[metze/wireshark/wip.git] / wiretap / erf.c
index 3bf9df54dead23da63a5d9b2ad0f5790ac456435..088845bd6b468e4fe1fe9ee9a41c23450e8b4093 100644 (file)
@@ -30,8 +30,6 @@
  * IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE)
  * ARISING IN ANY WAY OUT OF THE USE OF THIS SOFTWARE, EVEN IF ADVISED OF THE
  * POSSIBILITY OF SUCH DAMAGE.
- *
- * $Id$
  */
 
 /*
@@ -40,6 +38,7 @@
  * See
  *
  *      http://www.endace.com/support/EndaceRecordFormat.pdf
+ *      (mirror: https://bugs.wireshark.org/bugzilla/attachment.cgi?id=4333) (bug #4484)
  */
 
 #include "config.h"
 
 #include "wtap-int.h"
 #include "file_wrappers.h"
-#include "buffer.h"
 #include "pcap-encap.h"
-#include "atm.h"
 #include "erf.h"
 
-static int erf_read_header(FILE_T fh,
-                           struct wtap_pkthdr *phdr,
-                           erf_header_t *erf_header,
-                           int *err,
-                           gchar **err_info,
-                           guint32 *bytes_read,
-                           guint32 *packet_size);
+static gboolean erf_read_header(FILE_T fh,
+                                struct wtap_pkthdr *phdr,
+                                erf_header_t *erf_header,
+                                int *err,
+                                gchar **err_info,
+                                guint32 *bytes_read,
+                                guint32 *packet_size);
 static gboolean erf_read(wtap *wth, int *err, gchar **err_info,
                          gint64 *data_offset);
 static gboolean erf_seek_read(wtap *wth, gint64 seek_off,
-                              struct wtap_pkthdr *phdr, guint8 *pd,
-                              int length, int *err, gchar **err_info);
+                              struct wtap_pkthdr *phdr, Buffer *buf,
+                              int *err, gchar **err_info);
 
 static const struct {
   int erf_encap_value;
@@ -87,7 +84,7 @@ static const struct {
 
 #define NUM_ERF_ENCAPS (sizeof erf_to_wtap_map / sizeof erf_to_wtap_map[0])
 
-extern int erf_open(wtap *wth, int *err, gchar **err_info)
+extern wtap_open_return_val erf_open(wtap *wth, int *err, gchar **err_info)
 {
   int              i, n, records_for_erf_check = RECORDS_FOR_ERF_CHECK;
   int              valid_prev                  = 0;
@@ -100,7 +97,7 @@ extern int erf_open(wtap *wth, int *err, gchar **err_info)
   guint16          rlen;
   guint64          erf_ext_header;
   guint8           type;
-  size_t           r;
+  gboolean         r;
   gchar *          buffer;
 
   memset(&prevts, 0, sizeof(prevts));
@@ -120,21 +117,22 @@ extern int erf_open(wtap *wth, int *err, gchar **err_info)
 
   for (i = 0; i < records_for_erf_check; i++) {  /* records_for_erf_check */
 
-    r = file_read(&header,sizeof(header),wth->fh);
-
-    if (r == 0 ) break;
-    if (r != sizeof(header)) {
-      if ((*err = file_error(wth->fh, err_info)) != 0) {
-        return -1;
-      } else {
+    if (!wtap_read_bytes_or_eof(wth->fh,&header,sizeof(header),err,err_info)) {
+      if (*err == 0) {
+        /* EOF - all records have been successfully checked, accept the file */
+        break;
+      }
+      if (*err == WTAP_ERR_SHORT_READ) {
         /* ERF header too short accept the file,
            only if the very first records have been successfully checked */
         if (i < MIN_RECORDS_FOR_ERF_CHECK) {
-          return 0;
+          return WTAP_OPEN_NOT_MINE;
         } else {
           /* BREAK, the last record is too short, and will be ignored */
           break;
         }
+      } else {
+        return WTAP_OPEN_ERROR;
       }
     }
 
@@ -144,23 +142,22 @@ extern int erf_open(wtap *wth, int *err, gchar **err_info)
 
     /* Test valid rlen >= 16 */
     if (rlen < 16) {
-      return 0;
+      return WTAP_OPEN_NOT_MINE;
     }
 
     packet_size = rlen - (guint32)sizeof(header);
     if (packet_size > WTAP_MAX_PACKET_SIZE) {
       /*
        * Probably a corrupt capture file or a file that's not an ERF file
-       * but that passed earlier tests; don't blow up trying
-       * to allocate space for an immensely-large packet.
+       * but that passed earlier tests.
        */
-      return 0;
+      return WTAP_OPEN_NOT_MINE;
     }
 
     /* Skip PAD records, timestamps may not be set */
     if ((header.type & 0x7F) == ERF_TYPE_PAD) {
       if (file_seek(wth->fh, packet_size, SEEK_CUR, err) == -1) {
-        return -1;
+        return WTAP_OPEN_ERROR;
       }
       continue;
     }
@@ -168,24 +165,19 @@ extern int erf_open(wtap *wth, int *err, gchar **err_info)
     /* fail on invalid record type, decreasing timestamps or non-zero pad-bits */
     /* Not all types within this range are decoded, but it is a first filter */
     if ((header.type & 0x7F) == 0 || (header.type & 0x7F) > ERF_TYPE_MAX ) {
-      return 0;
-    }
-
-    /* The ERF_TYPE_MAX is the PAD record, but the last used type is ERF_TYPE_INFINIBAND_LINK */
-    if ((header.type & 0x7F) > ERF_TYPE_INFINIBAND_LINK) {
-      return 0;
+      return WTAP_OPEN_NOT_MINE;
     }
 
-    if ((ts = pletohll(&header.ts)) < prevts) {
+    if ((ts = pletoh64(&header.ts)) < prevts) {
       /* reassembled AALx records may not be in time order, also records are not in strict time order between physical interfaces, so allow 1 sec fudge */
       if ( ((prevts-ts)>>32) > 1 ) {
-        return 0;
+        return WTAP_OPEN_NOT_MINE;
       }
     }
 
     /* Check to see if timestamp increment is > 1 week */
     if ( (valid_prev) && (ts > prevts) && (((ts-prevts)>>32) > 3600*24*7) ) {
-      return 0;
+      return WTAP_OPEN_NOT_MINE;
     }
 
     memcpy(&prevts, &ts, sizeof(prevts));
@@ -193,9 +185,12 @@ extern int erf_open(wtap *wth, int *err, gchar **err_info)
     /* Read over the extension headers */
     type = header.type;
     while (type & 0x80){
-      if (file_read(&erf_ext_header, sizeof(erf_ext_header),wth->fh) != sizeof(erf_ext_header)) {
-        *err = file_error(wth->fh, err_info);
-        return -1;
+      if (!wtap_read_bytes(wth->fh,&erf_ext_header,sizeof(erf_ext_header),err,err_info)) {
+        if (*err == WTAP_ERR_SHORT_READ) {
+          /* Extension header missing, not an ERF file */
+          return WTAP_OPEN_NOT_MINE;
+        }
+        return WTAP_OPEN_ERROR;
       }
       packet_size -= (guint32)sizeof(erf_ext_header);
       memcpy(&type, &erf_ext_header, sizeof(type));
@@ -212,18 +207,25 @@ extern int erf_open(wtap *wth, int *err, gchar **err_info)
       case ERF_TYPE_MC_AAL2:
       case ERF_TYPE_COLOR_MC_HDLC_POS:
       case ERF_TYPE_AAL2: /* not an MC type but has a similar 'AAL2 ext' header */
-        if (file_read(&mc_hdr,sizeof(mc_hdr),wth->fh) != sizeof(mc_hdr)) {
-          *err = file_error(wth->fh, err_info);
-          return -1;
+        if (!wtap_read_bytes(wth->fh,&mc_hdr,sizeof(mc_hdr),err,err_info)) {
+          if (*err == WTAP_ERR_SHORT_READ) {
+            /* Subheader missing, not an ERF file */
+            return WTAP_OPEN_NOT_MINE;
+          }
+          return WTAP_OPEN_ERROR;
         }
         packet_size -= (guint32)sizeof(mc_hdr);
         break;
       case ERF_TYPE_ETH:
       case ERF_TYPE_COLOR_ETH:
       case ERF_TYPE_DSM_COLOR_ETH:
-        if (file_read(&eth_hdr,sizeof(eth_hdr),wth->fh) != sizeof(eth_hdr)) {
-          *err = file_error(wth->fh, err_info);
-          return -1;
+      case ERF_TYPE_COLOR_HASH_ETH:
+        if (!wtap_read_bytes(wth->fh,&eth_hdr,sizeof(eth_hdr),err,err_info)) {
+          if (*err == WTAP_ERR_SHORT_READ) {
+            /* Subheader missing, not an ERF file */
+            return WTAP_OPEN_NOT_MINE;
+          }
+          return WTAP_OPEN_ERROR;
         }
         packet_size -= (guint32)sizeof(eth_hdr);
         break;
@@ -235,20 +237,24 @@ extern int erf_open(wtap *wth, int *err, gchar **err_info)
        is reached whereas the record is truncated */
     if (packet_size > WTAP_MAX_PACKET_SIZE) {
       /*
-       * Probably a corrupt capture file; don't blow up trying
-       * to allocate space for an immensely-large packet.
+       * Probably a corrupt capture file or a file that's not an ERF file
+       * but that passed earlier tests.
        */
-      return 0;
+      return WTAP_OPEN_NOT_MINE;
     }
     buffer=(gchar *)g_malloc(packet_size);
-    r = file_read(buffer, packet_size, wth->fh);
+    r = wtap_read_bytes(wth->fh, buffer, packet_size, err, err_info);
     g_free(buffer);
 
-    if (r != packet_size) {
+    if (!r) {
+      if (*err != WTAP_ERR_SHORT_READ) {
+        /* A real error */
+        return WTAP_OPEN_ERROR;
+      }
       /* ERF record too short, accept the file,
          only if the very first records have been successfully checked */
       if (i < MIN_RECORDS_FOR_ERF_CHECK) {
-        return 0;
+        return WTAP_OPEN_NOT_MINE;
       }
     }
 
@@ -257,11 +263,11 @@ extern int erf_open(wtap *wth, int *err, gchar **err_info)
   } /* records_for_erf_check */
 
   if (file_seek(wth->fh, 0L, SEEK_SET, err) == -1) {   /* rewind */
-    return -1;
+    return WTAP_OPEN_ERROR;
   }
 
   /* This is an ERF file */
-  wth->file_type = WTAP_FILE_ERF;
+  wth->file_type_subtype = WTAP_FILE_TYPE_SUBTYPE_ERF;
   wth->snapshot_length = 0;     /* not available in header, only in frame */
 
   /*
@@ -271,11 +277,11 @@ extern int erf_open(wtap *wth, int *err, gchar **err_info)
 
   wth->subtype_read = erf_read;
   wth->subtype_seek_read = erf_seek_read;
-  wth->tsprecision = WTAP_FILE_TSPREC_NSEC;
+  wth->file_tsprec = WTAP_TSPREC_NSEC;
 
   erf_populate_interfaces(wth);
 
-  return 1;
+  return WTAP_OPEN_MINE;
 }
 
 /* Read the next packet */
@@ -294,10 +300,9 @@ static gboolean erf_read(wtap *wth, int *err, gchar **err_info,
       return FALSE;
     }
 
-    buffer_assure_space(wth->frame_buffer, packet_size);
-
-    wtap_file_read_expected_bytes(buffer_start_ptr(wth->frame_buffer),
-                                  (gint32)(packet_size), wth->fh, err, err_info);
+    if (!wtap_read_packet_bytes(wth->fh, wth->frame_buffer, packet_size,
+                                err, err_info))
+      return FALSE;
 
   } while ( erf_header.type == ERF_TYPE_PAD );
 
@@ -305,8 +310,8 @@ static gboolean erf_read(wtap *wth, int *err, gchar **err_info,
 }
 
 static gboolean erf_seek_read(wtap *wth, gint64 seek_off,
-                              struct wtap_pkthdr *phdr, guint8 *pd,
-                              int length _U_, int *err, gchar **err_info)
+                              struct wtap_pkthdr *phdr, Buffer *buf,
+                              int *err, gchar **err_info)
 {
   erf_header_t erf_header;
   guint32      packet_size;
@@ -320,19 +325,17 @@ static gboolean erf_seek_read(wtap *wth, gint64 seek_off,
       return FALSE;
   } while ( erf_header.type == ERF_TYPE_PAD );
 
-  wtap_file_read_expected_bytes(pd, (int)packet_size, wth->random_fh, err,
-                                err_info);
-
-  return TRUE;
+  return wtap_read_packet_bytes(wth->random_fh, buf, packet_size,
+                                err, err_info);
 }
 
-static int erf_read_header(FILE_T fh,
-                           struct wtap_pkthdr *phdr,
-                           erf_header_t *erf_header,
-                           int *err,
-                           gchar **err_info,
-                           guint32 *bytes_read,
-                           guint32 *packet_size)
+static gboolean erf_read_header(FILE_T fh,
+                                struct wtap_pkthdr *phdr,
+                                erf_header_t *erf_header,
+                                int *err,
+                                gchar **err_info,
+                                guint32 *bytes_read,
+                                guint32 *packet_size)
 {
   union wtap_pseudo_header *pseudo_header = &phdr->pseudo_header;
   guint32 mc_hdr;
@@ -344,8 +347,9 @@ static int erf_read_header(FILE_T fh,
   int     i       = 0;
   int     max     = sizeof(pseudo_header->erf.ehdr_list)/sizeof(struct erf_ehdr);
 
-  wtap_file_read_expected_bytes(erf_header, sizeof(*erf_header), fh, err,
-                                err_info);
+  if (!wtap_read_bytes_or_eof(fh, erf_header, sizeof(*erf_header), err, err_info)) {
+    return FALSE;
+  }
   if (bytes_read != NULL) {
     *bytes_read = sizeof(*erf_header);
   }
@@ -374,8 +378,9 @@ static int erf_read_header(FILE_T fh,
   }
 
   {
-    guint64 ts = pletohll(&erf_header->ts);
+    guint64 ts = pletoh64(&erf_header->ts);
 
+    phdr->rec_type = REC_TYPE_PACKET;
     phdr->presence_flags = WTAP_HAS_TS|WTAP_HAS_CAP_LEN|WTAP_HAS_INTERFACE_ID;
     phdr->ts.secs = (long) (ts >> 32);
     ts  = ((ts & 0xffffffff) * 1000 * 1000 * 1000);
@@ -390,7 +395,7 @@ static int erf_read_header(FILE_T fh,
 
   /* Copy the ERF pseudo header */
   memset(&pseudo_header->erf, 0, sizeof(pseudo_header->erf));
-  pseudo_header->erf.phdr.ts = pletohll(&erf_header->ts);
+  pseudo_header->erf.phdr.ts = pletoh64(&erf_header->ts);
   pseudo_header->erf.phdr.type = erf_header->type;
   pseudo_header->erf.phdr.flags = erf_header->flags;
   pseudo_header->erf.phdr.rlen = g_ntohs(erf_header->rlen);
@@ -400,13 +405,14 @@ static int erf_read_header(FILE_T fh,
   /* Copy the ERF extension header into the pseudo header */
   type = erf_header->type;
   while (type & 0x80){
-    wtap_file_read_expected_bytes(&erf_exhdr, sizeof(erf_exhdr), fh, err,
-                                  err_info);
+    if (!wtap_read_bytes(fh, &erf_exhdr, sizeof(erf_exhdr),
+                         err, err_info))
+      return FALSE;
     if (bytes_read != NULL)
       *bytes_read += (guint32)sizeof(erf_exhdr);
     *packet_size -=  (guint32)sizeof(erf_exhdr);
     skiplen += (guint32)sizeof(erf_exhdr);
-    erf_exhdr_sw = pntohll(erf_exhdr);
+    erf_exhdr_sw = pntoh64(erf_exhdr);
     if (i < max)
       memcpy(&pseudo_header->erf.ehdr_list[i].ehdr, &erf_exhdr_sw, sizeof(erf_exhdr_sw));
     type = erf_exhdr[0];
@@ -419,6 +425,7 @@ static int erf_read_header(FILE_T fh,
     case ERF_TYPE_RAW_LINK:
     case ERF_TYPE_INFINIBAND:
     case ERF_TYPE_INFINIBAND_LINK:
+    case ERF_TYPE_META:
 #if 0
       {
         phdr->len =  g_htons(erf_header->wlen);
@@ -431,6 +438,7 @@ static int erf_read_header(FILE_T fh,
     case ERF_TYPE_HDLC_POS:
     case ERF_TYPE_COLOR_HDLC_POS:
     case ERF_TYPE_DSM_COLOR_HDLC_POS:
+    case ERF_TYPE_COLOR_HASH_POS:
     case ERF_TYPE_ATM:
     case ERF_TYPE_AAL5:
       break;
@@ -438,8 +446,9 @@ static int erf_read_header(FILE_T fh,
     case ERF_TYPE_ETH:
     case ERF_TYPE_COLOR_ETH:
     case ERF_TYPE_DSM_COLOR_ETH:
-      wtap_file_read_expected_bytes(&eth_hdr, sizeof(eth_hdr), fh, err,
-                                    err_info);
+    case ERF_TYPE_COLOR_HASH_ETH:
+      if (!wtap_read_bytes(fh, &eth_hdr, sizeof(eth_hdr), err, err_info))
+        return FALSE;
       if (bytes_read != NULL)
         *bytes_read += (guint32)sizeof(eth_hdr);
       *packet_size -=  (guint32)sizeof(eth_hdr);
@@ -455,8 +464,8 @@ static int erf_read_header(FILE_T fh,
     case ERF_TYPE_MC_AAL2:
     case ERF_TYPE_COLOR_MC_HDLC_POS:
     case ERF_TYPE_AAL2: /* not an MC type but has a similar 'AAL2 ext' header */
-      wtap_file_read_expected_bytes(&mc_hdr, sizeof(mc_hdr), fh, err,
-                                    err_info);
+      if (!wtap_read_bytes(fh, &mc_hdr, sizeof(mc_hdr), err, err_info))
+        return FALSE;
       if (bytes_read != NULL)
         *bytes_read += (guint32)sizeof(mc_hdr);
       *packet_size -=  (guint32)sizeof(mc_hdr);
@@ -468,7 +477,7 @@ static int erf_read_header(FILE_T fh,
     case ERF_TYPE_TCP_FLOW_COUNTER:
       /* unsupported, continue with default: */
     default:
-      *err = WTAP_ERR_UNSUPPORTED_ENCAP;
+      *err = WTAP_ERR_UNSUPPORTED;
       *err_info = g_strdup_printf("erf: unknown record encapsulation %u",
                                   erf_header->type);
       return FALSE;
@@ -513,6 +522,7 @@ static gboolean erf_write_phdr(wtap_dumper *wdh, int encap, const union wtap_pse
   size_t size        = 0;
   size_t subhdr_size = 0;
   int    i           = 0;
+  guint8 has_more    = 0;
 
   switch(encap){
     case WTAP_ENCAP_ERF:
@@ -539,6 +549,7 @@ static gboolean erf_write_phdr(wtap_dumper *wdh, int encap, const union wtap_pse
         case ERF_TYPE_ETH:
         case ERF_TYPE_COLOR_ETH:
         case ERF_TYPE_DSM_COLOR_ETH:
+        case ERF_TYPE_COLOR_HASH_ETH:
           phtons(&erf_subhdr[0], pseudo_header->erf.subhdr.eth_hdr);
           subhdr_size += (int)sizeof(struct erf_eth_hdr);
           break;
@@ -555,15 +566,17 @@ static gboolean erf_write_phdr(wtap_dumper *wdh, int encap, const union wtap_pse
   wdh->bytes_dumped += size;
 
   /*write out up to MAX_ERF_EHDR extension headers*/
-  if((pseudo_header->erf.phdr.type & 0x80) != 0){  /*we have extension headers*/
+  has_more = pseudo_header->erf.phdr.type & 0x80;
+  if(has_more){  /*we have extension headers*/
     do{
       phtonll(ehdr+(i*8), pseudo_header->erf.ehdr_list[i].ehdr);
       if(i == MAX_ERF_EHDR-1) ehdr[i*8] = ehdr[i*8] & 0x7F;
+      has_more = ehdr[i*8] & 0x80;
       i++;
-    }while((ehdr[0] & 0x80) != 0 && i < MAX_ERF_EHDR);
-    if (!wtap_dump_file_write(wdh, ehdr, MAX_ERF_EHDR*i, err))
+    }while(has_more && i < MAX_ERF_EHDR);
+    if (!wtap_dump_file_write(wdh, ehdr, 8*i, err))
       return FALSE;
-    wdh->bytes_dumped += MAX_ERF_EHDR*i;
+    wdh->bytes_dumped += 8*i;
   }
 
   if(!wtap_dump_file_write(wdh, erf_subhdr, subhdr_size, err))
@@ -577,7 +590,8 @@ static gboolean erf_dump(
     wtap_dumper                    *wdh,
     const struct wtap_pkthdr       *phdr,
     const guint8                   *pd,
-    int                            *err)
+    int                            *err,
+    gchar                          **err_info _U_)
 {
   const union wtap_pseudo_header *pseudo_header = &phdr->pseudo_header;
   union wtap_pseudo_header other_phdr;
@@ -588,93 +602,106 @@ static gboolean erf_dump(
   gboolean must_add_crc = FALSE;
   guint32  crc32        = 0x00000000;
 
+  /* We can only write packet records. */
+  if (phdr->rec_type != REC_TYPE_PACKET) {
+    *err = WTAP_ERR_UNWRITABLE_REC_TYPE;
+    return FALSE;
+  }
+
+  /* Don't write anything bigger than we're willing to read. */
+  if(phdr->caplen > WTAP_MAX_PACKET_SIZE) {
+    *err = WTAP_ERR_PACKET_TOO_LARGE;
+    return FALSE;
+  }
+
   if(wdh->encap == WTAP_ENCAP_PER_PACKET){
     encap = phdr->pkt_encap;
   }else{
     encap = wdh->encap;
   }
 
-  switch(encap){
-    case WTAP_ENCAP_ERF:
-      alignbytes = wdh->bytes_dumped + pseudo_header->erf.phdr.rlen;
+  if(encap == WTAP_ENCAP_ERF){
+    /* We've been handed an ERF record, so there's not much to do here. */
+    alignbytes = wdh->bytes_dumped + pseudo_header->erf.phdr.rlen;
+
+    if(!erf_write_phdr(wdh, encap, pseudo_header, err)) return FALSE;
 
-      if(!erf_write_phdr(wdh, encap, pseudo_header, err)) return FALSE;
+    if(!wtap_dump_file_write(wdh, pd, phdr->caplen, err)) return FALSE;
+    wdh->bytes_dumped += phdr->caplen;
 
-      if(!wtap_dump_file_write(wdh, pd, phdr->caplen, err)) return FALSE;
-      wdh->bytes_dumped += phdr->caplen;
+    /*XXX: this pads the record to its original length, which is fine in most
+     * cases. However with >MAX_ERF_EHDR unnecessary padding will be added, and
+     * if the record was truncated this will be incorrectly treated as payload.
+     * More than 8 extension headers is unusual though, only the first 8 are
+     * written out anyway and fixing properly would require major refactor.*/
+    while(wdh->bytes_dumped < alignbytes){
+      if(!wtap_dump_file_write(wdh, "", 1, err)) return FALSE;
+      wdh->bytes_dumped++;
+    }
+    return TRUE;
+  }
 
-      while(wdh->bytes_dumped < alignbytes){
-        if(!wtap_dump_file_write(wdh, "", 1, err)) return FALSE;
-        wdh->bytes_dumped++;
+  /*generate a fake header in other_phdr using data that we know*/
+  /*covert time erf timestamp format*/
+  other_phdr.erf.phdr.ts = ((guint64) phdr->ts.secs << 32) + (((guint64) phdr->ts.nsecs <<32) / 1000 / 1000 / 1000);
+  other_phdr.erf.phdr.type = wtap_wtap_encap_to_erf_encap(encap);
+  other_phdr.erf.phdr.flags = 0x4;  /*vlen flag set because we're creating variable length records*/
+  other_phdr.erf.phdr.lctr = 0;
+  /*now we work out rlen, accounting for all the different headers and missing fcs(eth)*/
+  other_phdr.erf.phdr.rlen = phdr->caplen+16;
+  other_phdr.erf.phdr.wlen = phdr->len;
+  switch(other_phdr.erf.phdr.type){
+    case ERF_TYPE_ETH:
+      other_phdr.erf.phdr.rlen += 2;  /*2 bytes for erf eth_type*/
+      if (pseudo_header->eth.fcs_len != 4) {
+        /* Either this packet doesn't include the FCS
+           (pseudo_header->eth.fcs_len = 0), or we don't
+           know whether it has an FCS (= -1).  We have to
+           synthesize an FCS.*/
+         if(!(phdr->caplen < phdr->len)){ /*don't add FCS if packet has been snapped off*/
+          crc32 = crc32_ccitt_seed(pd, phdr->caplen, 0xFFFFFFFF);
+          other_phdr.erf.phdr.rlen += 4;  /*4 bytes for added checksum*/
+          other_phdr.erf.phdr.wlen += 4;
+          must_add_crc = TRUE;
+        }
       }
-      must_add_crc = TRUE; /* XXX - not if this came from an ERF file with an FCS! */
       break;
-    default:  /*deal with generic wtap format*/
-      /*generate a fake header in other_phdr using data that we know*/
-      /*covert time erf timestamp format*/
-      other_phdr.erf.phdr.ts = ((guint64) phdr->ts.secs << 32) + (((guint64) phdr->ts.nsecs <<32) / 1000 / 1000 / 1000);
-      other_phdr.erf.phdr.type = wtap_wtap_encap_to_erf_encap(encap);
-      other_phdr.erf.phdr.flags = 0x4;  /*vlen flag set because we're creating variable length records*/
-      other_phdr.erf.phdr.lctr = 0;
-      /*now we work out rlen, accounting for all the different headers and missing fcs(eth)*/
-      other_phdr.erf.phdr.rlen = phdr->caplen+16;
-      other_phdr.erf.phdr.wlen = phdr->len;
-      switch(other_phdr.erf.phdr.type){
-        case ERF_TYPE_ETH:
-          other_phdr.erf.phdr.rlen += 2;  /*2 bytes for erf eth_type*/
-          if (pseudo_header->eth.fcs_len != 4) {
-            /* Either this packet doesn't include the FCS
-               (pseudo_header->eth.fcs_len = 0), or we don't
-               know whether it has an FCS (= -1).  We have to
-               synthesize an FCS.*/
-
-            if(!(phdr->caplen < phdr->len)){ /*don't add FCS if packet has been snapped off*/
-              crc32 = crc32_ccitt_seed(pd, phdr->caplen, 0xFFFFFFFF);
-              other_phdr.erf.phdr.rlen += 4;  /*4 bytes for added checksum*/
-              other_phdr.erf.phdr.wlen += 4;
-              must_add_crc = TRUE;
-            }
-          }
-          break;
-        case ERF_TYPE_HDLC_POS:
-          /*we assume that it's missing a FCS checksum, make one up*/
-          if(!(phdr->caplen < phdr->len)){  /*unless of course, the packet has been snapped off*/
-            crc32 = crc32_ccitt_seed(pd, phdr->caplen, 0xFFFFFFFF);
-            other_phdr.erf.phdr.rlen += 4;  /*4 bytes for added checksum*/
-            other_phdr.erf.phdr.wlen += 4;
-            must_add_crc = TRUE; /* XXX - these never have an FCS? */
-          }
-          break;
-        default:
-          break;
-      }
-
-      alignbytes = (8 - (other_phdr.erf.phdr.rlen % 8)) % 8;  /*calculate how much padding will be required */
-      if(phdr->caplen < phdr->len){ /*if packet has been snapped, we need to round down what we output*/
-        round_down = (8 - alignbytes) % 8;
-        other_phdr.erf.phdr.rlen -= round_down;
-      }else{
-        other_phdr.erf.phdr.rlen += (gint16)alignbytes;
+    case ERF_TYPE_HDLC_POS:
+      /*we assume that it's missing a FCS checksum, make one up*/
+      if(!(phdr->caplen < phdr->len)){  /*unless of course, the packet has been snapped off*/
+        crc32 = crc32_ccitt_seed(pd, phdr->caplen, 0xFFFFFFFF);
+        other_phdr.erf.phdr.rlen += 4;  /*4 bytes for added checksum*/
+        other_phdr.erf.phdr.wlen += 4;
+        must_add_crc = TRUE; /* XXX - these never have an FCS? */
       }
+      break;
+    default:
+      break;
+  }
 
-      if(!erf_write_phdr(wdh, WTAP_ENCAP_ERF, &other_phdr, err)) return FALSE;
-      if(!wtap_dump_file_write(wdh, pd, phdr->caplen - round_down, err)) return FALSE;
-      wdh->bytes_dumped += phdr->caplen - round_down;
+  alignbytes = (8 - (other_phdr.erf.phdr.rlen % 8)) % 8;  /*calculate how much padding will be required */
+  if(phdr->caplen < phdr->len){ /*if packet has been snapped, we need to round down what we output*/
+    round_down = (8 - (guint)alignbytes) % 8;
+    other_phdr.erf.phdr.rlen -= round_down;
+  }else{
+    other_phdr.erf.phdr.rlen += (gint16)alignbytes;
+  }
 
-      /*add the 4 byte CRC if necessary*/
-      if(must_add_crc){
-        if(!wtap_dump_file_write(wdh, &crc32, 4, err)) return FALSE;
-        wdh->bytes_dumped += 4;
-      }
-      /*records should be 8byte aligned, so we add padding*/
-      if(round_down == 0){
-        for(i = (gint16)alignbytes; i > 0; i--){
-          if(!wtap_dump_file_write(wdh, "", 1, err)) return FALSE;
-          wdh->bytes_dumped++;
-        }
-      }
+  if(!erf_write_phdr(wdh, WTAP_ENCAP_ERF, &other_phdr, err)) return FALSE;
+  if(!wtap_dump_file_write(wdh, pd, phdr->caplen - round_down, err)) return FALSE;
+  wdh->bytes_dumped += phdr->caplen - round_down;
 
-      break;
+  /*add the 4 byte CRC if necessary*/
+  if(must_add_crc){
+    if(!wtap_dump_file_write(wdh, &crc32, 4, err)) return FALSE;
+    wdh->bytes_dumped += 4;
+  }
+  /*records should be 8byte aligned, so we add padding*/
+  if(round_down == 0){
+    for(i = (gint16)alignbytes; i > 0; i--){
+      if(!wtap_dump_file_write(wdh, "", 1, err)) return FALSE;
+      wdh->bytes_dumped++;
+    }
   }
 
   return TRUE;
@@ -687,7 +714,7 @@ int erf_dump_can_write_encap(int encap)
     return 0;
 
   if (wtap_wtap_encap_to_erf_encap(encap) == -1)
-    return WTAP_ERR_UNSUPPORTED_ENCAP;
+    return WTAP_ERR_UNWRITABLE_ENCAP;
 
   return 0;
 }
@@ -695,14 +722,13 @@ int erf_dump_can_write_encap(int encap)
 int erf_dump_open(wtap_dumper *wdh, int *err)
 {
   wdh->subtype_write = erf_dump;
-  wdh->subtype_close = NULL;
 
-  switch(wdh->file_type){
-    case WTAP_FILE_ERF:
-      wdh->tsprecision = WTAP_FILE_TSPREC_NSEC;
+  switch(wdh->file_type_subtype){
+    case WTAP_FILE_TYPE_SUBTYPE_ERF:
+      wdh->tsprecision = WTAP_TSPREC_NSEC;
       break;
     default:
-      *err = WTAP_ERR_UNSUPPORTED_FILE_TYPE;
+      *err = WTAP_ERR_UNWRITABLE_FILE_TYPE;
       return FALSE;
       break;
   }
@@ -718,10 +744,6 @@ int erf_populate_interfaces(wtap *wth)
   if (!wth)
     return -1;
 
-  if (!wth->interface_data) {
-    wth->interface_data = g_array_new(FALSE, FALSE, sizeof(wtapng_if_descr_t));
-  }
-
   memset(&int_data, 0, sizeof(int_data)); /* Zero all fields */
 
   int_data.wtap_encap = WTAP_ENCAP_ERF;
@@ -754,8 +776,20 @@ int erf_populate_interfaces(wtap *wth)
     int_data.if_description = g_strdup_printf("ERF Interface Id %d (Port %c)", i, 'A'+i);
 
     g_array_append_val(wth->interface_data, int_data);
-    wth->number_of_interfaces++;
   }
 
   return 0;
 }
+
+/*
+ * Editor modelines  -  http://www.wireshark.org/tools/modelines.html
+ *
+ * Local Variables:
+ * c-basic-offset: 2
+ * tab-width: 8
+ * indent-tabs-mode: nil
+ * End:
+ *
+ * vi: set shiftwidth=2 tabstop=8 expandtab:
+ * :indentSize=2:tabSize=8:noTabs=true:
+ */