[git commit] shaNNNsum: accept one-space "HASH FILENAME" format for -c, closes 14866
Denys Vlasenko
vda.linux at googlemail.com
Mon Jul 11 12:36:39 UTC 2022
commit: https://git.busybox.net/busybox/commit/?id=3ad3aa6441ebaf817137051de2b74cb6b4379e7f
branch: https://git.busybox.net/busybox/commit/?id=refs/heads/master
function old new delta
md5_sha1_sum_main 496 501 +5
Signed-off-by: Denys Vlasenko <vda.linux at googlemail.com>
---
coreutils/md5_sha1_sum.c | 13 ++++++++-----
testsuite/sha1sum.tests | 10 ++++++++++
2 files changed, 18 insertions(+), 5 deletions(-)
diff --git a/coreutils/md5_sha1_sum.c b/coreutils/md5_sha1_sum.c
index 0e57673f1..b4bdc262c 100644
--- a/coreutils/md5_sha1_sum.c
+++ b/coreutils/md5_sha1_sum.c
@@ -301,9 +301,7 @@ int md5_sha1_sum_main(int argc UNUSED_PARAM, char **argv)
count_total++;
filename_ptr = strchr(line, ' ');
- if (filename_ptr == NULL
- || (filename_ptr[1] != ' ' && filename_ptr[1] != '*')
- ) {
+ if (!filename_ptr) {
if (flags & FLAG_WARN) {
bb_simple_error_msg("invalid format");
}
@@ -312,8 +310,13 @@ int md5_sha1_sum_main(int argc UNUSED_PARAM, char **argv)
free(line);
continue;
}
- *filename_ptr = '\0';
- filename_ptr += 2;
+ *filename_ptr++ = '\0';
+ /* coreutils 9.1 allows "HASH FILENAME" format,
+ * with only one space. Skip the 'correct'
+ * " " or " *" delimiter if it is there:
+ */
+ if (*filename_ptr == ' ' || *filename_ptr == '*')
+ filename_ptr++;
hash_value = hash_file(in_buf, filename_ptr, sha3_width);
diff --git a/testsuite/sha1sum.tests b/testsuite/sha1sum.tests
index a968fa87c..7ad1334c3 100755
--- a/testsuite/sha1sum.tests
+++ b/testsuite/sha1sum.tests
@@ -1,3 +1,13 @@
#!/bin/sh
+. ./testing.sh
+
+# testing "test name" "cmd" "expected result" "file input" "stdin"
+>EMPTY
+testing "sha1sum: one-space separated input for -c" \
+ 'echo "da39a3ee5e6b4b0d3255bfef95601890afd80709 EMPTY" | sha1sum -c' \
+ "EMPTY: OK\n" \
+ "" ""
+rm EMPTY
+
. ./md5sum.tests sha1sum d41337e834377140ae7f98460d71d908598ef04f
More information about the busybox-cvs
mailing list