diff options
author | Guy Harris <guy@alum.mit.edu> | 2013-06-16 00:20:00 +0000 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2013-06-16 00:20:00 +0000 |
commit | 8c9edf12800bc6d68894dc457e7ebaf994429da8 (patch) | |
tree | ec6efefbd4e7f8227a7b96661f721ff4ba2986c3 /wiretap/packetlogger.c | |
parent | 3846abe34d6861c6ee0bba61fcd5baa4d213885c (diff) |
Have the seek-read routines take a Buffer rather than a guint8 pointer
as the "where to put the packet data" argument.
This lets more of the libwiretap code be common between the read and
seek-read code paths, and also allows for more flexibility in the "fill
in the data" path - we can expand the buffer as needed in both cases.
svn path=/trunk/; revision=49949
Diffstat (limited to 'wiretap/packetlogger.c')
-rw-r--r-- | wiretap/packetlogger.c | 35 |
1 files changed, 6 insertions, 29 deletions
diff --git a/wiretap/packetlogger.c b/wiretap/packetlogger.c index 12d8409f67..f72ece74d7 100644 --- a/wiretap/packetlogger.c +++ b/wiretap/packetlogger.c @@ -50,7 +50,7 @@ static gboolean packetlogger_read(wtap *wth, int *err, gchar **err_info, gint64 *data_offset); static gboolean packetlogger_seek_read(wtap *wth, gint64 seek_off, struct wtap_pkthdr *phdr, - guint8 *pd, int length, int *err, + Buffer *buf, int length, int *err, gchar **err_info); static gboolean packetlogger_read_header(packetlogger_header_t *pl_hdr, FILE_T fh, int *err, gchar **err_info); @@ -98,34 +98,19 @@ int packetlogger_open(wtap *wth, int *err, gchar **err_info) static gboolean packetlogger_read(wtap *wth, int *err, gchar **err_info, gint64 *data_offset) { - guint bytes_read; - *data_offset = file_tell(wth->fh); if(!packetlogger_process_header(wth->fh, &wth->phdr, err, err_info)) return FALSE; - buffer_assure_space(wth->frame_buffer, wth->phdr.caplen); - bytes_read = file_read(buffer_start_ptr(wth->frame_buffer), - wth->phdr.caplen, - wth->fh); - if(bytes_read != wth->phdr.caplen) { - *err = file_error(wth->fh, err_info); - if(*err == 0) - *err = WTAP_ERR_SHORT_READ; - - return FALSE; - } - - return TRUE; + return wtap_read_packet_bytes(wth->fh, wth->frame_buffer, + wth->phdr.caplen, err, err_info); } static gboolean packetlogger_seek_read(wtap *wth, gint64 seek_off, struct wtap_pkthdr *phdr, - guint8 *pd, int length, int *err, gchar **err_info) + Buffer *buf, int length, int *err, gchar **err_info) { - guint bytes_read; - if(file_seek(wth->random_fh, seek_off, SEEK_SET, err) == -1) return FALSE; @@ -142,16 +127,8 @@ packetlogger_seek_read(wtap *wth, gint64 seek_off, struct wtap_pkthdr *phdr, return FALSE; } - bytes_read = file_read(pd, phdr->caplen, wth->random_fh); - if(bytes_read != phdr->caplen) { - *err = file_error(wth->random_fh, err_info); - if(*err == 0) - *err = WTAP_ERR_SHORT_READ; - - return FALSE; - } - - return TRUE; + return wtap_read_packet_bytes(wth->random_fh, buf, phdr->caplen, + err, err_info); } static gboolean |