[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#67490: [PATCH] tail: fix following /proc and /sys files when using a
From: |
dann frazier |
Subject: |
bug#67490: [PATCH] tail: fix following /proc and /sys files when using a 64K page size |
Date: |
Tue, 28 Nov 2023 17:38:52 -0700 |
On Mon, Nov 27, 2023 at 11:17:57AM -0800, Paul Eggert wrote:
> On 2023-11-27 08:24, dann frazier wrote:
> > + if (fstat (fd, &stats) != 0)
> > + {
> > + error (0, errno, _("cannot fstat %s"), quoteaf (pretty_filename));
> > + return false;
> > + }
> > +
> > + bufsize = ST_BLKSIZE (stats);
>
>
> If fstat fails, that's no reason to exit. Just use bufsize
> = BUFSIZ and keep going. fstat can fail for reasons
> unrelated to what 'tail' needs (e.g., time_t overflow).
Thanks for the suggestion - I'll make that change.
-dann