Hi BusyBox maintainers,

We are looking at measuring system components and it will be ideal to
have sha384sum in BusyBox, since certain chips do not support sha512 hashes.
Here's a patch to add sha384sum applet and config.
It is my first time making changes to BusyBox. Please let me know what
you think about this patch.

Thanks,
Can

---
 NOFORK_NOEXEC.lst               |  1 +
 configs/TEST_nommu_defconfig    |  1 +
 configs/TEST_noprintf_defconfig |  1 +
 configs/TEST_rh9_defconfig      |  1 +
 configs/android2_defconfig      |  1 +
 configs/android_502_defconfig   |  1 +
 configs/android_defconfig       |  1 +
 configs/android_ndk_defconfig   |  1 +
 configs/cygwin_defconfig        |  1 +
 configs/freebsd_defconfig       |  1 +
 coreutils/md5_sha1_sum.c        | 35 ++++++++++++++++++++++++++++++---
 include/libbb.h                 |  1 +
 libbb/hash_md5_sha.c            | 31 ++++++++++++++++++++++++++++-
 testsuite/md5sum.tests          |  1 +
 testsuite/sha384sum.tests       |  3 +++
 15 files changed, 77 insertions(+), 4 deletions(-)
 create mode 100755 testsuite/sha384sum.tests

diff --git a/NOFORK_NOEXEC.lst b/NOFORK_NOEXEC.lst
index 055f9fb24..a000de45b 100644
--- a/NOFORK_NOEXEC.lst
+++ b/NOFORK_NOEXEC.lst
@@ -336,6 +336,7 @@ setuidgid - noexec. spawner
 sha1sum - noexec. runner
 sha256sum - noexec. runner
 sha3sum - noexec. runner
+sha384sum - noexec. runner
 sha512sum - noexec. runner
 showkey - interactive, longterm
 shred - runner
diff --git a/configs/TEST_nommu_defconfig b/configs/TEST_nommu_defconfig
index fa3e96326..7c5577ae8 100644
--- a/configs/TEST_nommu_defconfig
+++ b/configs/TEST_nommu_defconfig
@@ -220,6 +220,7 @@ CONFIG_FEATURE_RMDIR_LONG_OPTIONS=y
 CONFIG_SEQ=y
 CONFIG_SHA1SUM=y
 CONFIG_SHA256SUM=y
+CONFIG_SHA384SUM=y
 CONFIG_SHA512SUM=y
 CONFIG_SLEEP=y
 CONFIG_FEATURE_FANCY_SLEEP=y
diff --git a/configs/TEST_noprintf_defconfig b/configs/TEST_noprintf_defconfig
index 9b378fd55..91581b6e3 100644
--- a/configs/TEST_noprintf_defconfig
+++ b/configs/TEST_noprintf_defconfig
@@ -242,6 +242,7 @@ CONFIG_FALSE=y
 # CONFIG_SEQ is not set
 # CONFIG_SHA1SUM is not set
 # CONFIG_SHA256SUM is not set
+# CONFIG_SHA384SUM is not set
 # CONFIG_SHA512SUM is not set
 # CONFIG_SLEEP is not set
 # CONFIG_FEATURE_FANCY_SLEEP is not set
diff --git a/configs/TEST_rh9_defconfig b/configs/TEST_rh9_defconfig
index 4ac62b9da..c85e0c1d9 100644
--- a/configs/TEST_rh9_defconfig
+++ b/configs/TEST_rh9_defconfig
@@ -231,6 +231,7 @@ CONFIG_FEATURE_RMDIR_LONG_OPTIONS=y
 CONFIG_SEQ=y
 CONFIG_SHA1SUM=y
 CONFIG_SHA256SUM=y
+CONFIG_SHA384SUM=y
 CONFIG_SHA512SUM=y
 CONFIG_SLEEP=y
 CONFIG_FEATURE_FANCY_SLEEP=y
diff --git a/configs/android2_defconfig b/configs/android2_defconfig
index d4b8f1616..b71c03d0e 100644
--- a/configs/android2_defconfig
+++ b/configs/android2_defconfig
@@ -248,6 +248,7 @@ CONFIG_RMDIR=y
 CONFIG_SEQ=y
 CONFIG_SHA1SUM=y
 CONFIG_SHA256SUM=y
+CONFIG_SHA384SUM=y
 CONFIG_SHA512SUM=y
 CONFIG_SLEEP=y
 CONFIG_FEATURE_FANCY_SLEEP=y
diff --git a/configs/android_502_defconfig b/configs/android_502_defconfig
index 104e70f23..6d172a04f 100644
--- a/configs/android_502_defconfig
+++ b/configs/android_502_defconfig
@@ -344,6 +344,7 @@ CONFIG_FEATURE_RMDIR_LONG_OPTIONS=y
 CONFIG_SEQ=y
 CONFIG_SHA1SUM=y
 CONFIG_SHA256SUM=y
+CONFIG_SHA384SUM=y
 CONFIG_SHA512SUM=y
 CONFIG_SHA3SUM=y
 CONFIG_SLEEP=y
diff --git a/configs/android_defconfig b/configs/android_defconfig
index 92a66048a..cddb30421 100644
--- a/configs/android_defconfig
+++ b/configs/android_defconfig
@@ -272,6 +272,7 @@ CONFIG_RMDIR=y
 CONFIG_SEQ=y
 CONFIG_SHA1SUM=y
 CONFIG_SHA256SUM=y
+CONFIG_SHA384SUM=y
 CONFIG_SHA512SUM=y
 CONFIG_SLEEP=y
 CONFIG_FEATURE_FANCY_SLEEP=y
diff --git a/configs/android_ndk_defconfig b/configs/android_ndk_defconfig
index 425593454..86af28a92 100644
--- a/configs/android_ndk_defconfig
+++ b/configs/android_ndk_defconfig
@@ -269,6 +269,7 @@ CONFIG_RMDIR=y
 CONFIG_SEQ=y
 CONFIG_SHA1SUM=y
 CONFIG_SHA256SUM=y
+CONFIG_SHA384SUM=y
 CONFIG_SHA512SUM=y
 CONFIG_SHA3SUM=y
 CONFIG_SLEEP=y
diff --git a/configs/cygwin_defconfig b/configs/cygwin_defconfig
index 61e2c2463..46f778eb2 100644
--- a/configs/cygwin_defconfig
+++ b/configs/cygwin_defconfig
@@ -284,6 +284,7 @@ CONFIG_FEATURE_LS_COLOR_IS_DEFAULT=y
 CONFIG_MD5SUM=y
 CONFIG_SHA1SUM=y
 CONFIG_SHA256SUM=y
+CONFIG_SHA384SUM=y
 CONFIG_SHA512SUM=y
 CONFIG_SHA3SUM=y

diff --git a/configs/freebsd_defconfig b/configs/freebsd_defconfig
index 6cbd54895..8c7212b4c 100644
--- a/configs/freebsd_defconfig
+++ b/configs/freebsd_defconfig
@@ -245,6 +245,7 @@ CONFIG_RMDIR=y
 CONFIG_SEQ=y
 CONFIG_SHA1SUM=y
 CONFIG_SHA256SUM=y
+CONFIG_SHA384SUM=y
 CONFIG_SHA512SUM=y
 CONFIG_SLEEP=y
 CONFIG_FEATURE_FANCY_SLEEP=y
diff --git a/coreutils/md5_sha1_sum.c b/coreutils/md5_sha1_sum.c
index 978d328f1..70fe2fbf7 100644
--- a/coreutils/md5_sha1_sum.c
+++ b/coreutils/md5_sha1_sum.c
@@ -23,6 +23,12 @@
 //config: help
 //config: Compute and check SHA256 message digest
 //config:
+//config:config SHA384SUM
+//config: bool "sha384sum (7.3 kb)"
+//config: default y
+//config: help
+//config: Compute and check SHA384 message digest
+//config:
 //config:config SHA512SUM
 //config: bool "sha512sum (7.3 kb)"
 //config: default y
@@ -35,13 +41,13 @@
 //config: help
 //config: Compute and check SHA3 message digest
 //config:
-//config:comment "Common options for md5sum, sha1sum, sha256sum,
sha512sum, sha3sum"
-//config: depends on MD5SUM || SHA1SUM || SHA256SUM || SHA512SUM || SHA3SUM
+//config:comment "Common options for md5sum, sha1sum, sha256sum,
sha384sum, sha512sum, sha3sum"
+//config: depends on MD5SUM || SHA1SUM || SHA256SUM || SHA384SUM ||
SHA512SUM || SHA3SUM
 //config:
 //config:config FEATURE_MD5_SHA1_SUM_CHECK
 //config: bool "Enable -c, -s and -w options"
 //config: default y
-//config: depends on MD5SUM || SHA1SUM || SHA256SUM || SHA512SUM || SHA3SUM
+//config: depends on MD5SUM || SHA1SUM || SHA256SUM || SHA384SUM ||
SHA512SUM || SHA3SUM
 //config: help
 //config: Enabling the -c options allows files to be checked
 //config: against pre-calculated hash values.
@@ -51,11 +57,13 @@
 //applet:IF_SHA1SUM(APPLET_NOEXEC(sha1sum, md5_sha1_sum,
BB_DIR_USR_BIN, BB_SUID_DROP, sha1sum))
 //applet:IF_SHA3SUM(APPLET_NOEXEC(sha3sum, md5_sha1_sum,
BB_DIR_USR_BIN, BB_SUID_DROP, sha3sum))
 //applet:IF_SHA256SUM(APPLET_NOEXEC(sha256sum, md5_sha1_sum,
BB_DIR_USR_BIN, BB_SUID_DROP, sha256sum))
+//applet:IF_SHA384SUM(APPLET_NOEXEC(sha384sum, md5_sha1_sum,
BB_DIR_USR_BIN, BB_SUID_DROP, sha384sum))
 //applet:IF_SHA512SUM(APPLET_NOEXEC(sha512sum, md5_sha1_sum,
BB_DIR_USR_BIN, BB_SUID_DROP, sha512sum))

 //kbuild:lib-$(CONFIG_MD5SUM)    += md5_sha1_sum.o
 //kbuild:lib-$(CONFIG_SHA1SUM)   += md5_sha1_sum.o
 //kbuild:lib-$(CONFIG_SHA256SUM) += md5_sha1_sum.o
+//kbuild:lib-$(CONFIG_SHA384SUM) += md5_sha1_sum.o
 //kbuild:lib-$(CONFIG_SHA512SUM) += md5_sha1_sum.o
 //kbuild:lib-$(CONFIG_SHA3SUM)   += md5_sha1_sum.o

@@ -99,6 +107,16 @@
 //usage:     "\n -w Warn about improperly formatted checksum lines"
 //usage: )
 //usage:
+//usage:#define sha384sum_trivial_usage
+//usage: IF_FEATURE_MD5_SHA1_SUM_CHECK("[-c[sw]] ")"[FILE]..."
+//usage:#define sha384sum_full_usage "\n\n"
+//usage:       "Print" IF_FEATURE_MD5_SHA1_SUM_CHECK(" or check") "
SHA384 checksums"
+//usage: IF_FEATURE_MD5_SHA1_SUM_CHECK( "\n"
+//usage:     "\n -c Check sums against list in FILEs"
+//usage:     "\n -s Don't output anything, status code shows success"
+//usage:     "\n -w Warn about improperly formatted checksum lines"
+//usage: )
+//usage:
 //usage:#define sha512sum_trivial_usage
 //usage: IF_FEATURE_MD5_SHA1_SUM_CHECK("[-c[sw]] ")"[FILE]..."
 //usage:#define sha512sum_full_usage "\n\n"
@@ -134,6 +152,7 @@ enum {
  HASH_SHA1 = '1',
  HASH_SHA256 = '2',
  HASH_SHA3 = '3',
+ HASH_SHA384 = '8', /* 5th letter */
  HASH_SHA512 = '5',
 };

@@ -178,6 +197,10 @@ static uint8_t *hash_file(unsigned char *in_buf,
const char *filename, unsigned

  hash_algo = applet_name[3];

+ if (applet_name[4] == HASH_SHA384) {
+ hash_algo = HASH_SHA384;
+ }
+
  /* figure specific hash algorithms */
  if (ENABLE_MD5SUM && hash_algo == HASH_MD5) {
  md5_begin(&context.md5);
@@ -197,6 +220,12 @@ static uint8_t *hash_file(unsigned char *in_buf,
const char *filename, unsigned
  final = (void*)sha256_end;
  hash_len = 32;
  }
+ else if (ENABLE_SHA384SUM && hash_algo == HASH_SHA384) {
+ sha384_begin(&context.sha512);
+ update = (void*)sha512_hash;
+ final = (void*)sha512_end;
+ hash_len = 48;
+ }
  else if (ENABLE_SHA512SUM && hash_algo == HASH_SHA512) {
  sha512_begin(&context.sha512);
  update = (void*)sha512_hash;
diff --git a/include/libbb.h b/include/libbb.h
index 01cdb1bdc..88ad6c69b 100644
--- a/include/libbb.h
+++ b/include/libbb.h
@@ -2195,6 +2195,7 @@ unsigned sha1_end(sha1_ctx_t *ctx, void *resbuf)
FAST_FUNC;
 void sha256_begin(sha256_ctx_t *ctx) FAST_FUNC;
 #define sha256_hash md5_hash
 #define sha256_end  sha1_end
+void sha384_begin(sha512_ctx_t *ctx) FAST_FUNC;
 void sha512_begin(sha512_ctx_t *ctx) FAST_FUNC;
 void sha512_hash(sha512_ctx_t *ctx, const void *buffer, size_t len) FAST_FUNC;
 unsigned sha512_end(sha512_ctx_t *ctx, void *resbuf) FAST_FUNC;
diff --git a/libbb/hash_md5_sha.c b/libbb/hash_md5_sha.c
index 57a801459..28f28296e 100644
--- a/libbb/hash_md5_sha.c
+++ b/libbb/hash_md5_sha.c
@@ -11,7 +11,8 @@
 #define STR1(s) #s
 #define STR(s) STR1(s)

-#define NEED_SHA512 (ENABLE_SHA512SUM || ENABLE_USE_BB_CRYPT_SHA)
+#define NEED_SHA384 (ENABLE_SHA384SUM)
+#define NEED_SHA512 (ENABLE_SHA384SUM || ENABLE_SHA512SUM ||
ENABLE_USE_BB_CRYPT_SHA)

 #if ENABLE_SHA1_HWACCEL || ENABLE_SHA256_HWACCEL
 # if defined(__GNUC__) && (defined(__i386__) || defined(__x86_64__))
@@ -1215,6 +1216,20 @@ static const uint32_t init256[] ALIGN4 = {
  0x1f83d9ab,
  0x5be0cd19,
 };
+#if NEED_SHA384
+static const uint64_t init384[] ALIGN4 = {
+ 0,
+ 0,
+ 0xcbbb9d5dc1059ed8,
+ 0x629a292a367cd507,
+ 0x9159015a3070dd17,
+ 0x152fecd8f70e5939,
+ 0x67332667ffc00b31,
+ 0x8eb44a8768581511,
+ 0xdb0c2e0d64f98fa7,
+ 0x47b5481dbefa4fa4,
+};
+#endif /* NEED_SHA384 */
 #if NEED_SHA512
 static const uint32_t init512_lo[] ALIGN4 = {
  0,
@@ -1255,6 +1270,20 @@ void FAST_FUNC sha256_begin(sha256_ctx_t *ctx)
 #endif
 }

+#if NEED_SHA384
+/* Initialize structure containing state of computation.
+   (FIPS 180-2:5.3.3)  */
+void FAST_FUNC sha384_begin(sha512_ctx_t *ctx)
+{
+ int i;
+ /* Two extra iterations zero out ctx->total64[2] */
+ uint64_t *tp = ctx->total64;
+ for (i = 0; i < 8 + 2; i++)
+ tp[i] = init384[i];
+ /*ctx->total64[0] = ctx->total64[1] = 0; - already done */
+}
+#endif /* NEED_SHA384 */
+
 #if NEED_SHA512
 /* Initialize structure containing state of computation.
    (FIPS 180-2:5.3.3)  */
diff --git a/testsuite/md5sum.tests b/testsuite/md5sum.tests
index cca26dc64..634dcd4aa 100755
--- a/testsuite/md5sum.tests
+++ b/testsuite/md5sum.tests
@@ -9,6 +9,7 @@
 # efe30c482e0b687e0cca0612f42ca29b
 # d41337e834377140ae7f98460d71d908598ef04f
 # 8e1d3ed57ebc130f0f72508446559eeae06451ae6d61b1e8ce46370cfb8963c3
+# 
c01420bb6613d4bd00396a82033e59e81486cbb045ae0dd2b4c3332e581b3ce09fb1946d6e283acec685778ff205d485
 # 
fe413e0f177324d1353893ca0772ceba83fd41512ba63895a0eebb703ef9feac2fb4e92b2cb430b3bda41b46b0cb4ea8307190a5cc795157cfb680a9cd635d0f

 if ! test "$1"; then
diff --git a/testsuite/sha384sum.tests b/testsuite/sha384sum.tests
new file mode 100755
index 000000000..f1449d195
--- /dev/null
+++ b/testsuite/sha384sum.tests
@@ -0,0 +1,3 @@
+#!/bin/sh
+
+. ./md5sum.tests sha384sum
c01420bb6613d4bd00396a82033e59e81486cbb045ae0dd2b4c3332e581b3ce09fb1946d6e283acec685778ff205d485
--
2.45.2
_______________________________________________
busybox mailing list
[email protected]
https://lists.busybox.net/mailman/listinfo/busybox

Reply via email to