aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorElliott Hughes <enh@google.com>2018-12-19 08:58:36 -0800
committerRob Landley <rob@landley.net>2018-12-19 13:50:38 -0600
commit880e41fa961173652b4a414b1e6740f610421a21 (patch)
tree1c11ec1a08611e32a5b43725f584e827c94e2d41
parentb6c12d25bce1c923e44a5c6c897b9e8ffa801e2c (diff)
downloadtoybox-880e41fa961173652b4a414b1e6740f610421a21.tar.gz
md5sum/sha1sum: add --status and --check, fix --check with multiple files.
Annoyingly coreutils and busybox both have the --status functionality, but coreutils only accepts --status and busybox only accepts -s. Although all extant users known to me use --status I've supported both (a) for maximum compatibility and (b) because Rob hates longopts :-) Also, -c/--check don't take argument(s): they alter the interpretation of all the FILE... arguments. I removed a bunch of dead code here, but I couldn't switch us over to loopfiles_lines because I didn't want to regress on the "warn if any check file contains no hashes", and loopfiles_lines doesn't give you any way to recognize that you've moved on to the next file.
-rwxr-xr-xtests/md5sum.test9
-rw-r--r--toys/lsb/md5sum.c62
2 files changed, 37 insertions, 34 deletions
diff --git a/tests/md5sum.test b/tests/md5sum.test
index f159d1ed..cda9871f 100755
--- a/tests/md5sum.test
+++ b/tests/md5sum.test
@@ -28,3 +28,12 @@ testing "-c spaces" "md5sum -c input || echo ok" \
rm "te st" empty
testing "-c nolines" "md5sum -c input 2>/dev/null || echo ok" "ok\n" "" ""
+
+touch empty
+echo "d41d8cd98f00b204e9800998ecf8427e empty" > list
+echo "041d8cd98f00b204e9800998ecf8427e empty" > badlist
+testing "--status okay" "md5sum --status -c list ; echo \$?" "0\n" "" ""
+testing "--status fail" "md5sum --status -c badlist ; echo \$?" "1\n" "" ""
+testing "-c multiple" "md5sum -c list badlist --status ; echo \$?" "1\n" "" ""
+
+rm empty list badlist
diff --git a/toys/lsb/md5sum.c b/toys/lsb/md5sum.c
index 80f45a0e..14024184 100644
--- a/toys/lsb/md5sum.c
+++ b/toys/lsb/md5sum.c
@@ -12,9 +12,11 @@
* We optionally use openssl (or equivalent) to access assembly optimized
* versions of these functions, but provide a built-in version to reduce
* required dependencies.
+ *
+ * coreutils supports --status but not -s, busybox supports -s but not --status
-USE_MD5SUM(NEWTOY(md5sum, "bc*[!bc]", TOYFLAG_USR|TOYFLAG_BIN))
-USE_SHA1SUM(NEWTOY(sha1sum, "bc*[!bc]", TOYFLAG_USR|TOYFLAG_BIN))
+USE_MD5SUM(NEWTOY(md5sum, "bc(check)s(status)[!bc]", TOYFLAG_USR|TOYFLAG_BIN))
+USE_SHA1SUM(NEWTOY(sha1sum, "bc(check)s(status)[!bc]", TOYFLAG_USR|TOYFLAG_BIN))
USE_SHA224SUM(OLDTOY(sha224sum, sha1sum, TOYFLAG_USR|TOYFLAG_BIN))
USE_SHA256SUM(OLDTOY(sha256sum, sha1sum, TOYFLAG_USR|TOYFLAG_BIN))
USE_SHA384SUM(OLDTOY(sha384sum, sha1sum, TOYFLAG_USR|TOYFLAG_BIN))
@@ -24,26 +26,28 @@ config MD5SUM
bool "md5sum"
default y
help
- usage: md5sum [-b] [-c FILE] [FILE]...
+ usage: md5sum [-bcs] [FILE]...
Calculate md5 hash for each input file, reading from stdin if none.
Output one hash (32 hex digits) for each input file, followed by filename.
-b Brief (hash only, no filename)
- -c Check each line of FILE is the same hash+filename we'd output
+ -c Check each line of each FILE is the same hash+filename we'd output
+ -s No output, exit status 0 if all hashes match, 1 otherwise
config SHA1SUM
bool "sha1sum"
default y
help
- usage: sha?sum [-b] [-c FILE] [FILE]...
+ usage: sha?sum [-bcs] [FILE]...
- calculate sha hash for each input file, reading from stdin if none. Output
+ Calculate sha hash for each input file, reading from stdin if none. Output
one hash (40 hex digits for sha1, 56 for sha224, 64 for sha256, 96 for sha384,
and 128 for sha512) for each input file, followed by filename.
-b Brief (hash only, no filename)
- -c Check each line of FILE is the same hash+filename we'd output
+ -c Check each line of each FILE is the same hash+filename we'd output
+ -s No output, exit status 0 if all hashes match, 1 otherwise
config SHA224SUM
bool "sha224sum"
@@ -86,8 +90,6 @@ typedef int SHA512_CTX;
#endif
GLOBALS(
- struct arg_list *c;
-
int sawline;
// Crypto variables blanked after summing
@@ -338,13 +340,13 @@ static void do_builtin_hash(int fd, char *name)
static void do_hash(int fd, char *name)
{
if (CFG_TOYBOX_LIBCRYPTO) do_lib_hash(fd, name);
- else do_builtin_hash(fd,name);
+ else do_builtin_hash(fd, name);
if (name)
printf((toys.optflags & FLAG_b) ? "%s\n" : "%s %s\n", toybuf, name);
}
-static int do_c(char *line, size_t len)
+static int do_c_line(char *line)
{
int space = 0, fail = 0;
char *name;
@@ -366,55 +368,47 @@ static int do_c(char *line, size_t len)
*toybuf = 0;
} else do_hash(fd, 0);
if (strcasecmp(line, toybuf)) toys.exitval = fail = 1;
- printf("%s: %s\n", name, fail ? "FAILED" : "OK");
+ if (!FLAG(s)) printf("%s: %s\n", name, fail ? "FAILED" : "OK");
if (fd>0) close(fd);
}
return 0;
}
-// Open file, read each line, and call do_line(). Returns 0 if file existed
-// and we read it to the end, 1 if interrupted by callback, 2 of didn't exist
-// do_line returns 0 to free line, 1 to keep line, 2 to end loop
-int looplines(char *name, int trim, int (*do_line)(char *line, size_t len))
+// Used instead of loopfiles_line to report error on files containing no hashes.
+static void do_c_file(char *name)
{
FILE *fp = !strcmp(name, "-") ? stdin : fopen(name, "r");
- int rc = 0;
if (!fp) {
perror_msg_raw(name);
-
- return 2;
+ return;
}
+ TT.sawline = 0;
+
for (;;) {
char *line = 0;
ssize_t len;
if ((len = getline(&line, (void *)&len, fp))<1) break;
- if (line[len-1]=='\n') len--;
- if (trim) line[len] = 0;
- len = do_line(line, len);
- if (!len) free(line);
- if (len==2) {
- rc = 2;
- break;
- }
+ if (line[len-1]=='\n') line[len-1] = 0;
+ do_c_line(line);
+ free(line);
}
if (fp!=stdin) fclose(fp);
- return rc;
+ if (!TT.sawline) error_msg("%s: no lines", name);
}
void md5sum_main(void)
{
- struct arg_list *al;
+ char **arg;
- if (!TT.c) loopfiles(toys.optargs, do_hash);
- else for (al = TT.c; al; al = al->next) {
- TT.sawline = 0;
- looplines(al->arg, 1, do_c);
- if (!TT.sawline) error_msg("%s: no lines", al->arg);
+ if (FLAG(c)) for (arg = toys.optargs; *arg; arg++) do_c_file(*arg);
+ else {
+ if (FLAG(s)) error_exit("-s only with -c");
+ loopfiles(toys.optargs, do_hash);
}
}