diff options
author | Denis Vlasenko <vda.linux@googlemail.com> | 2007-06-04 10:16:52 +0000 |
---|---|---|
committer | Denis Vlasenko <vda.linux@googlemail.com> | 2007-06-04 10:16:52 +0000 |
commit | 74324c86663f57a19c1de303ee8c8e5449db9ef2 (patch) | |
tree | 11f5da9de4212875ce5811be2e1050e076378c9a /coreutils/cksum.c | |
parent | 4e5f82c76f08614d0b69f9ec4a8baac303af15f6 (diff) | |
download | busybox-w32-74324c86663f57a19c1de303ee8c8e5449db9ef2.tar.gz busybox-w32-74324c86663f57a19c1de303ee8c8e5449db9ef2.tar.bz2 busybox-w32-74324c86663f57a19c1de303ee8c8e5449db9ef2.zip |
Audit bb_common_bufsiz usage, add script which looks for misuse.
tr: stop using globals needlessly.
code: -103 bytes
Diffstat (limited to 'coreutils/cksum.c')
-rw-r--r-- | coreutils/cksum.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/coreutils/cksum.c b/coreutils/cksum.c index 865bea0ee..987f5f32c 100644 --- a/coreutils/cksum.c +++ b/coreutils/cksum.c | |||
@@ -27,8 +27,9 @@ int cksum_main(int argc, char **argv) | |||
27 | crc = 0; | 27 | crc = 0; |
28 | length = 0; | 28 | length = 0; |
29 | 29 | ||
30 | while ((bytes_read = fread(bb_common_bufsiz1, 1, BUFSIZ, fp)) > 0) { | 30 | #define read_buf bb_common_bufsiz1 |
31 | cp = bb_common_bufsiz1; | 31 | while ((bytes_read = fread(read_buf, 1, BUFSIZ, fp)) > 0) { |
32 | cp = read_buf; | ||
32 | length += bytes_read; | 33 | length += bytes_read; |
33 | while (bytes_read--) | 34 | while (bytes_read--) |
34 | crc = (crc << 8) ^ crc32_table[((crc >> 24) ^ (*cp++)) & 0xffL]; | 35 | crc = (crc << 8) ^ crc32_table[((crc >> 24) ^ (*cp++)) & 0xffL]; |