aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBernhard Reutner-Fischer <rep.dot.nop@gmail.com>2007-01-27 22:11:28 +0000
committerBernhard Reutner-Fischer <rep.dot.nop@gmail.com>2007-01-27 22:11:28 +0000
commitcd75a96f0f9d446028cad7e4b9b9224e009752e1 (patch)
tree3d85b95f511d68e6e465a84f4465ecccb2226b7e
parent14813c5943ef6d3e07a8ccab1d29d9cb704cebd7 (diff)
downloadbusybox-cd75a96f0f9d446028cad7e4b9b9224e009752e1.tar.gz
- sum -r TODO should not print the filename as oposed to -s
Unfortunately, without rewriting sum, this bugfix adds 19 (!) bytes.
-rw-r--r--coreutils/sum.c37
1 files changed, 21 insertions, 16 deletions
diff --git a/coreutils/sum.c b/coreutils/sum.c
index df5804899..d62f2cfde 100644
--- a/coreutils/sum.c
+++ b/coreutils/sum.c
@@ -15,22 +15,21 @@
#include "busybox.h"
-enum { sysv_sum, bsd_sum };
+enum { SUM_BSD, PRINT_NAME, SUM_SYSV };
/* BSD: calculate and print the rotated checksum and the size in 1K blocks
The checksum varies depending on sizeof (int). */
/* SYSV: calculate and print the checksum and the size in 512-byte blocks */
/* Return 1 if successful. */
-static int sum_file(const char *file, int type, int print_name)
+static unsigned sum_file(const char *file, const unsigned type)
{
#define buf bb_common_bufsiz1
- int r, fd;
uintmax_t total_bytes = 0;
+ int fd = 0, r;
/* The sum of all the input bytes, modulo (UINT_MAX + 1). */
unsigned s = 0;
- fd = 0;
if (NOT_LONE_DASH(file)) {
fd = open(file, O_RDONLY);
if (fd == -1)
@@ -51,7 +50,7 @@ static int sum_file(const char *file, int type, int print_name)
}
total_bytes += bytes_read;
- if (type == sysv_sum) {
+ if (type >= SUM_SYSV) {
do s += buf[--bytes_read]; while (bytes_read);
} else {
r = 0;
@@ -63,8 +62,9 @@ static int sum_file(const char *file, int type, int print_name)
}
}
- if (!print_name) file = "";
- if (type == sysv_sum) {
+ if (type < PRINT_NAME)
+ file = "";
+ if (type >= SUM_SYSV) {
r = (s & 0xffff) + ((s & 0xffffffff) >> 16);
s = (r & 0xffff) + (r >> 16);
printf("%d %ju %s\n", s, (total_bytes+511)/512, file);
@@ -76,19 +76,24 @@ static int sum_file(const char *file, int type, int print_name)
int sum_main(int argc, char **argv)
{
- int n;
- int type = bsd_sum;
+ unsigned n;
+ unsigned type = SUM_BSD;
n = getopt32(argc, argv, "sr");
- if (n & 1) type = sysv_sum;
+ if (n & 1) type = SUM_SYSV;
/* give the bsd priority over sysv func */
- if (n & 2) type = bsd_sum;
+ if (n & 2) type = SUM_BSD;
- if (argc == optind)
- n = sum_file("-", type, 0);
- else
+ if (argc == optind) {
+ /* Do not print the name */
+ n = sum_file("-", type);
+ } else {
+ /* Need to print the name if either
+ - more than one file given
+ - doing sysv */
+ type += argc - 1 > optind || type == SUM_SYSV;
for (n = 1; optind < argc; optind++)
- n &= sum_file(argv[optind], type, 1);
-
+ n &= sum_file(argv[optind], type);
+ }
return !n;
}