No need for read_new_line to return a packet offset.
[metze/wireshark/wip.git] / wiretap / iptrace.c
index ea1bef4998795f283430ce250dcec847ced6f44d..cba013b06bf5ac6530d33b7f3f2279065322e478 100644 (file)
@@ -1,6 +1,4 @@
 /* iptrace.c
- *
- * $Id$
  *
  * Wiretap Library
  * Copyright (c) 1998 by Gilbert Ramirez <gram@alumni.rice.edu>
@@ -26,7 +24,6 @@
 #include <string.h>
 #include "wtap-int.h"
 #include "file_wrappers.h"
-#include "buffer.h"
 #include "atm.h"
 #include "iptrace.h"
 
@@ -43,46 +40,42 @@ static gboolean iptrace_read_2_0(wtap *wth, int *err, gchar **err_info,
 static gboolean iptrace_seek_read_2_0(wtap *wth, gint64 seek_off,
     struct wtap_pkthdr *phdr, Buffer *buf, int *err, gchar **err_info);
 
-static int iptrace_read_rec_header(FILE_T fh, guint8 *header, int header_len,
-    int *err, gchar **err_info);
 static gboolean iptrace_read_rec_data(FILE_T fh, Buffer *buf,
     struct wtap_pkthdr *phdr, int *err, gchar **err_info);
 static void fill_in_pseudo_header(int encap,
     union wtap_pseudo_header *pseudo_header, guint8 *header);
 static int wtap_encap_ift(unsigned int  ift);
 
-int iptrace_open(wtap *wth, int *err, gchar **err_info)
+#define NAME_SIZE 11
+
+wtap_open_return_val iptrace_open(wtap *wth, int *err, gchar **err_info)
 {
-       int bytes_read;
-       char name[12];
-
-       errno = WTAP_ERR_CANT_READ;
-       bytes_read = file_read(name, 11, wth->fh);
-       if (bytes_read != 11) {
-               *err = file_error(wth->fh, err_info);
-               if (*err != 0 && *err != WTAP_ERR_SHORT_READ)
-                       return -1;
-               return 0;
+       char name[NAME_SIZE+1];
+
+       if (!wtap_read_bytes(wth->fh, name, NAME_SIZE, err, err_info)) {
+               if (*err != WTAP_ERR_SHORT_READ)
+                       return WTAP_OPEN_ERROR;
+               return WTAP_OPEN_NOT_MINE;
        }
-       name[11] = '\0';
+       name[NAME_SIZE] = '\0';
 
        if (strcmp(name, "iptrace 1.0") == 0) {
                wth->file_type_subtype = WTAP_FILE_TYPE_SUBTYPE_IPTRACE_1_0;
                wth->subtype_read = iptrace_read_1_0;
                wth->subtype_seek_read = iptrace_seek_read_1_0;
-               wth->tsprecision = WTAP_FILE_TSPREC_SEC;
+               wth->file_tsprec = WTAP_TSPREC_SEC;
        }
        else if (strcmp(name, "iptrace 2.0") == 0) {
                wth->file_type_subtype = WTAP_FILE_TYPE_SUBTYPE_IPTRACE_2_0;
                wth->subtype_read = iptrace_read_2_0;
                wth->subtype_seek_read = iptrace_seek_read_2_0;
-               wth->tsprecision = WTAP_FILE_TSPREC_NSEC;
+               wth->file_tsprec = WTAP_TSPREC_NSEC;
        }
        else {
-               return 0;
+               return WTAP_OPEN_NOT_MINE;
        }
 
-       return 1;
+       return WTAP_OPEN_MINE;
 }
 
 /***********************************************************
@@ -125,13 +118,10 @@ iptrace_read_rec_1_0(FILE_T fh, struct wtap_pkthdr *phdr, Buffer *buf,
     int *err, gchar **err_info)
 {
        guint8                  header[IPTRACE_1_0_PHDR_SIZE];
-       int                     ret;
        iptrace_1_0_phdr        pkt_hdr;
        guint32                 packet_size;
 
-       ret = iptrace_read_rec_header(fh, header, IPTRACE_1_0_PHDR_SIZE,
-           err, err_info);
-       if (ret <= 0) {
+       if (!wtap_read_bytes_or_eof(fh, header, sizeof header, err, err_info)) {
                /* Read error or EOF */
                return FALSE;
        }
@@ -144,7 +134,7 @@ iptrace_read_rec_1_0(FILE_T fh, struct wtap_pkthdr *phdr, Buffer *buf,
        pkt_hdr.if_type = header[28];
        phdr->pkt_encap = wtap_encap_ift(pkt_hdr.if_type);
        if (phdr->pkt_encap == WTAP_ENCAP_UNKNOWN) {
-               *err = WTAP_ERR_UNSUPPORTED_ENCAP;
+               *err = WTAP_ERR_UNSUPPORTED;
                *err_info = g_strdup_printf("iptrace: interface type IFT=0x%02x unknown or unsupported",
                    pkt_hdr.if_type);
                return FALSE;
@@ -188,20 +178,21 @@ iptrace_read_rec_1_0(FILE_T fh, struct wtap_pkthdr *phdr, Buffer *buf,
                /*
                 * Skip the padding.
                 */
-               if (!file_skip(fh, 3, err))
+               if (!wtap_read_bytes(fh, NULL, 3, err, err_info))
                        return FALSE;
        }
-       if (packet_size > WTAP_MAX_PACKET_SIZE) {
+       if (packet_size > WTAP_MAX_PACKET_SIZE_STANDARD) {
                /*
                 * Probably a corrupt capture file; don't blow up trying
                 * to allocate space for an immensely-large packet.
                 */
                *err = WTAP_ERR_BAD_FILE;
                *err_info = g_strdup_printf("iptrace: File has %u-byte packet, bigger than maximum of %u",
-                   packet_size, WTAP_MAX_PACKET_SIZE);
+                   packet_size, WTAP_MAX_PACKET_SIZE_STANDARD);
                return FALSE;
        }
 
+       phdr->rec_type = REC_TYPE_PACKET;
        phdr->presence_flags = WTAP_HAS_TS;
        phdr->len = packet_size;
        phdr->caplen = packet_size;
@@ -302,13 +293,10 @@ iptrace_read_rec_2_0(FILE_T fh, struct wtap_pkthdr *phdr, Buffer *buf,
     int *err, gchar **err_info)
 {
        guint8                  header[IPTRACE_2_0_PHDR_SIZE];
-       int                     ret;
        iptrace_2_0_phdr        pkt_hdr;
        guint32                 packet_size;
 
-       ret = iptrace_read_rec_header(fh, header, IPTRACE_2_0_PHDR_SIZE,
-           err, err_info);
-       if (ret <= 0) {
+       if (!wtap_read_bytes_or_eof(fh, header, sizeof header, err, err_info)) {
                /* Read error or EOF */
                return FALSE;
        }
@@ -338,7 +326,7 @@ iptrace_read_rec_2_0(FILE_T fh, struct wtap_pkthdr *phdr, Buffer *buf,
         * that we don't handle?
         */
        if (phdr->pkt_encap == WTAP_ENCAP_UNKNOWN) {
-               *err = WTAP_ERR_UNSUPPORTED_ENCAP;
+               *err = WTAP_ERR_UNSUPPORTED;
                *err_info = g_strdup_printf("iptrace: interface type IFT=0x%02x unknown or unsupported",
                    pkt_hdr.if_type);
                return FALSE;
@@ -383,20 +371,21 @@ iptrace_read_rec_2_0(FILE_T fh, struct wtap_pkthdr *phdr, Buffer *buf,
                /*
                 * Skip the padding.
                 */
-               if (!file_skip(fh, 3, err))
+               if (!wtap_read_bytes(fh, NULL, 3, err, err_info))
                        return FALSE;
        }
-       if (packet_size > WTAP_MAX_PACKET_SIZE) {
+       if (packet_size > WTAP_MAX_PACKET_SIZE_STANDARD) {
                /*
                 * Probably a corrupt capture file; don't blow up trying
                 * to allocate space for an immensely-large packet.
                 */
                *err = WTAP_ERR_BAD_FILE;
                *err_info = g_strdup_printf("iptrace: File has %u-byte packet, bigger than maximum of %u",
-                   packet_size, WTAP_MAX_PACKET_SIZE);
+                   packet_size, WTAP_MAX_PACKET_SIZE_STANDARD);
                return FALSE;
        }
 
+       phdr->rec_type = REC_TYPE_PACKET;
        phdr->presence_flags = WTAP_HAS_TS;
        phdr->len = packet_size;
        phdr->caplen = packet_size;
@@ -454,27 +443,6 @@ static gboolean iptrace_seek_read_2_0(wtap *wth, gint64 seek_off,
        return TRUE;
 }
 
-static int
-iptrace_read_rec_header(FILE_T fh, guint8 *header, int header_len, int *err,
-    gchar **err_info)
-{
-       int     bytes_read;
-
-       errno = WTAP_ERR_CANT_READ;
-       bytes_read = file_read(header, header_len, fh);
-       if (bytes_read != header_len) {
-               *err = file_error(fh, err_info);
-               if (*err != 0)
-                       return -1;
-               if (bytes_read != 0) {
-                       *err = WTAP_ERR_SHORT_READ;
-                       return -1;
-               }
-               return 0;
-       }
-       return 1;
-}
-
 static gboolean
 iptrace_read_rec_data(FILE_T fh, Buffer *buf, struct wtap_pkthdr *phdr,
     int *err, gchar **err_info)
@@ -487,8 +455,7 @@ iptrace_read_rec_data(FILE_T fh, Buffer *buf, struct wtap_pkthdr *phdr,
                 * Attempt to guess from the packet data, the VPI,
                 * and the VCI information about the type of traffic.
                 */
-               atm_guess_traffic_type(buffer_start_ptr(buf), phdr->caplen,
-                   &phdr->pseudo_header);
+               atm_guess_traffic_type(phdr, ws_buffer_start_ptr(buf));
        }
 
        return TRUE;
@@ -636,3 +603,16 @@ wtap_encap_ift(unsigned int  ift)
                }
        }
 }
+
+/*
+ * Editor modelines  -  http://www.wireshark.org/tools/modelines.html
+ *
+ * Local variables:
+ * c-basic-offset: 8
+ * tab-width: 8
+ * indent-tabs-mode: t
+ * End:
+ *
+ * vi: set shiftwidth=8 tabstop=8 noexpandtab:
+ * :indentSize=8:tabSize=8:noTabs=false:
+ */