From 849a1c46cfa48ebaf4339a5a5b7f2ec9ddce5fb9 Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Sat, 11 Nov 1995 22:08:16 +0000 Subject: [PATCH] (skip): Cast fseek's offset argument to `long'. Erik Bennett reported that this is necessary on BSDI systems. And if offset doesn't fit in a long, then try using lseek instead. --- src/od.c | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/src/od.c b/src/od.c index 559626a196..0293b11a17 100644 --- a/src/od.c +++ b/src/od.c @@ -981,7 +981,13 @@ skip (off_t n_skip) } else { - if (fseek (in_stream, n_skip, SEEK_SET) == 0) + /* fseek may work on some streams for which lseek doesn't. + But fseek's offset argument is restricted to the range + of type `long'. So if N_SKIP is too large or if fseek + fails, try lseek. */ + if ((n_skip <= LONG_MAX + && fseek (in_stream, (long) n_skip, SEEK_SET) == 0) + || lseek (fileno (in_stream), n_skip, SEEK_SET) >= 0) { n_skip = 0; break; @@ -989,7 +995,8 @@ skip (off_t n_skip) } } - /* fseek didn't work or wasn't attempted; do it the slow way. */ + /* Seek didn't work or wasn't attempted; position the file pointer + by reading. */ for (j = n_skip / BUFSIZ; j >= 0; j--) { -- 2.47.3