From b5c60fc7870a66d89de571e96596b0745edcd6d9 Mon Sep 17 00:00:00 2001 From: Denis Vlasenko Date: Sun, 27 Jan 2008 23:41:34 +0000 Subject: [PATCH] mkswap, readahead: stop using fdlength, it is reported to be unreliable --- libbb/xfuncs.c | 2 ++ miscutils/readahead.c | 11 +++++++++-- util-linux/mkswap.c | 6 ++++-- 3 files changed, 15 insertions(+), 4 deletions(-) diff --git a/libbb/xfuncs.c b/libbb/xfuncs.c index 25c36bd07..445e07717 100644 --- a/libbb/xfuncs.c +++ b/libbb/xfuncs.c @@ -475,6 +475,7 @@ void xsetuid(uid_t uid) } // Return how long the file at fd is, if there's any way to determine it. +#ifdef UNUSED off_t fdlength(int fd) { off_t bottom = 0, top = 0, pos; @@ -513,6 +514,7 @@ off_t fdlength(int fd) return pos + 1; } +#endif int bb_putchar(int ch) { diff --git a/miscutils/readahead.c b/miscutils/readahead.c index 7b375cfff..fb71ce85f 100644 --- a/miscutils/readahead.c +++ b/miscutils/readahead.c @@ -22,9 +22,16 @@ int readahead_main(int argc, char **argv) while (*++argv) { int fd = open_or_warn(*argv, O_RDONLY); if (fd >= 0) { - int r = readahead(fd, 0, fdlength(fd)); + off_t len; + int r; + + /* fdlength was reported to be unreliable - use seek */ + len = xlseek(fd, 0, SEEK_END); + xlseek(fd, 0, SEEK_SET); + r = readahead(fd, 0, len); close(fd); - if (r >= 0) continue; + if (r >= 0) + continue; } retval = EXIT_FAILURE; } diff --git a/util-linux/mkswap.c b/util-linux/mkswap.c index 8e1fbc384..f047cce26 100644 --- a/util-linux/mkswap.c +++ b/util-linux/mkswap.c @@ -64,9 +64,11 @@ int mkswap_main(int argc, char **argv) // Figure out how big the device is and announce our intentions. fd = xopen(argv[1], O_RDWR); - len = fdlength(fd); + /* fdlength was reported to be unreliable - use seek */ + len = xlseek(fd, 0, SEEK_END); + xlseek(fd, 0, SEEK_SET); pagesize = getpagesize(); - printf("Setting up swapspace version 1, size = %"OFF_FMT"d bytes\n", + printf("Setting up swapspace version 1, size = %"OFF_FMT"u bytes\n", len - pagesize); mkswap_selinux_setcontext(fd, argv[1]);