From 3a788d773b13078ce8c87247e7f5f556646203b3 Mon Sep 17 00:00:00 2001 From: Dima Kogan Date: Sat, 14 Aug 2010 02:15:47 -0700 Subject: [PATCH] tailf: fixed timing issue that could cause duplicate data output The issue is that in roll_file() we fstat() to find the file size, then read() as much data as we can and then use the previously saved file size to mark our position. The bug occurs if we read past the file size reported by fstat() because more data has arrived while we were reading it. The attached patch uses the current file position as the location marker instead, with some extra logic to handle tailing truncated files. [kzak@redhat.com: - fix coding style] Signed-off-by: Dima Kogan Signed-off-by: Karel Zak --- text-utils/tailf.c | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-) diff --git a/text-utils/tailf.c b/text-utils/tailf.c index 75998ce7..dc18b2a6 100644 --- a/text-utils/tailf.c +++ b/text-utils/tailf.c @@ -88,6 +88,7 @@ roll_file(const char *filename, off_t *size) char buf[BUFSIZ]; int fd; struct stat st; + off_t pos; if (!(fd = open(filename, O_RDONLY))) err(EXIT_FAILURE, _("cannot open \"%s\" for read"), filename); @@ -111,8 +112,16 @@ roll_file(const char *filename, off_t *size) } fflush(stdout); } + + pos = lseek(fd, 0, SEEK_CUR); + + /* If we've successfully read something, use the file position, this + * avoids data duplication. If we read nothing or hit an error, reset + * to the reported size, this handles truncated files. + */ + *size = (pos != -1 && pos != *size) ? pos : st.st_size; + close(fd); - *size = st.st_size; } static void -- 2.39.5