diff options
author | Guy Harris <guy@alum.mit.edu> | 2014-10-07 16:18:47 -0700 |
---|---|---|
committer | Guy Harris <guy@alum.mit.edu> | 2014-10-07 23:19:56 +0000 |
commit | a566f617d8f712a8f4d1fe7b6d9b9f6ebe79ba11 (patch) | |
tree | 1730a9b5c9cc7704273e4104a426e6987f9ef9b6 /wiretap/netxray.c | |
parent | 5e0e4fa9e4de39f9402e6472e6b25439f8464c24 (diff) | |
download | wireshark-a566f617d8f712a8f4d1fe7b6d9b9f6ebe79ba11.tar.gz |
No need for WTAP_ERR_CANT_READ.
Unlike the standard I/O routines, the code we introduced that supports
fast random seeking on gzipped files will always supply some specific
error code for read errors, so we don't need WTAP_ERR_CANT_READ.
Add WTAP_ERR_CANT_WRITE for writing, as we're still using the standard
I/O routines for that. Set errno to WTAP_ERR_CANT_WRITE before calling
fwrite() in wtap_dump_file_write(), so that it's used if fwrite() fails
without setting errno.
Change-Id: I6bf066a6838284a532737aa65fd0c9bb3639ad63
Reviewed-on: https://code.wireshark.org/review/4540
Reviewed-by: Guy Harris <guy@alum.mit.edu>
Diffstat (limited to 'wiretap/netxray.c')
-rw-r--r-- | wiretap/netxray.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/wiretap/netxray.c b/wiretap/netxray.c index fa8573839e..d6d37535a9 100644 --- a/wiretap/netxray.c +++ b/wiretap/netxray.c @@ -460,7 +460,6 @@ netxray_open(wtap *wth, int *err, gchar **err_info) /* Read in the string that should be at the start of a NetXRay * file */ - errno = WTAP_ERR_CANT_READ; if (!wtap_read_bytes(wth->fh, magic, MAGIC_SIZE, err, err_info)) { if (*err != WTAP_ERR_SHORT_READ) return -1; @@ -476,7 +475,6 @@ netxray_open(wtap *wth, int *err, gchar **err_info) } /* Read the rest of the header. */ - errno = WTAP_ERR_CANT_READ; if (!wtap_read_bytes(wth->fh, &hdr, sizeof hdr, err, err_info)) return -1; @@ -1139,7 +1137,6 @@ netxray_process_rec_header(wtap *wth, FILE_T fh, struct wtap_pkthdr *phdr, hdr_size = sizeof (struct netxrayrec_2_x_hdr); break; } - errno = WTAP_ERR_CANT_READ; if (!wtap_read_bytes_or_eof(fh, (void *)&hdr, hdr_size, err, err_info)) { /* * If *err is 0, we're at EOF. *err being 0 and a return |