diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2006-10-31 22:46:08 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2006-10-31 22:46:08 +0000 |
commit | b141b9b512f0faa01624bf384046439abc9b2850 (patch) | |
tree | 3a83675d2871e1e74168e2ffef9e9f4bed509b17 | |
parent | 4126b1f5c6983b7c2dd4f92d635ab762d861c2d6 (diff) | |
download | busybox-w32-b141b9b512f0faa01624bf384046439abc9b2850.tar.gz busybox-w32-b141b9b512f0faa01624bf384046439abc9b2850.tar.bz2 busybox-w32-b141b9b512f0faa01624bf384046439abc9b2850.zip |
reads: fix bug 1078
-rw-r--r-- | libbb/read.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/libbb/read.c b/libbb/read.c index 1c2945f45..b3648b4d7 100644 --- a/libbb/read.c +++ b/libbb/read.c | |||
@@ -86,7 +86,8 @@ char *reads(int fd, char *buffer, size_t size) | |||
86 | if (p) { | 86 | if (p) { |
87 | off_t offset; | 87 | off_t offset; |
88 | *p++ = '\0'; | 88 | *p++ = '\0'; |
89 | offset = (p-buffer) - size; | 89 | // avoid incorrect (unsigned) widening |
90 | offset = (off_t)(p-buffer) - (off_t)size; | ||
90 | // set fd position the right after the \n | 91 | // set fd position the right after the \n |
91 | if (offset && lseek(fd, offset, SEEK_CUR) == (off_t)-1) | 92 | if (offset && lseek(fd, offset, SEEK_CUR) == (off_t)-1) |
92 | return NULL; | 93 | return NULL; |