From a566f617d8f712a8f4d1fe7b6d9b9f6ebe79ba11 Mon Sep 17 00:00:00 2001 From: Guy Harris Date: Tue, 7 Oct 2014 16:18:47 -0700 Subject: 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 --- wiretap/netxray.c | 3 --- 1 file changed, 3 deletions(-) (limited to 'wiretap/netxray.c') 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 -- cgit v1.2.1