cksum: Added check for sum length

[SarthakSingh31]

Fixes #6576

[cakebaker]

Can you please add a test so we don't regress in the future?

[sylvestre]

and it regressed a test (but maybe the test is incorrect)

--- TRY 3 STDERR:        coreutils::tests test_hashsum::test_check_b2sum_strict_check ---
thread 'test_hashsum::test_check_b2sum_strict_check' panicked at 'Expected stderr to be empty, but it's:
b2sum: WARNING: 7 lines are improperly formatted
', tests/by-util/test_hashsum.rs:932:10
stack backtrace:
   0: rust_begin_unwind
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/std/src/panicking.rs:578:5
   1: core::panicking::panic_fmt
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/core/src/panicking.rs:67:14
   2: tests::common::util::CmdResult::no_stderr
             at ./tests/common/util.rs:444:9
   3: tests::common::util::CmdResult::stdout_only
             at ./tests/common/util.rs:615:9
   4: tests::test_hashsum::test_check_b2sum_strict_check
             at ./tests/by-util/test_hashsum.rs:927:5
   5: tests::test_hashsum::test_check_b2sum_strict_check::{{closure}}
             at ./tests/by-util/test_hashsum.rs:908:36
   6: core::ops::function::FnOnce::call_once
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/core/src/ops/function.rs:250:5
   7: core::ops::function::FnOnce::call_once
             at /rustc/90c541806f23a127002de5b4038be731ba1458ca/library/core/src/ops/function.rs:250:5
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

[github-actions]

GNU testsuite comparison:

GNU test failed: tests/cksum/b2sum. tests/cksum/b2sum is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/cksum/cksum-base64. tests/cksum/cksum-base64 is passing on 'main'. Maybe you have to rebase?
GNU test failed: tests/cksum/cksum-c. tests/cksum/cksum-c is passing on 'main'. Maybe you have to rebase?
Skipping an intermittent issue tests/tail/inotify-dir-recreate (passes in this run but fails in the 'main' branch)

[SarthakSingh31]

@sylvestre The tests pass now.

I think the checkrum.rs and sum.rs need to be rewritten with more typing. One of the big issues is that everything is referred to as length.
Like this is 100% wrong

coreutils/src/uucore/src/lib/features/checksum.rs

Lines 238 to 239 in e33cecd

	create_fn: Box::new(move || Box::new(Blake2b::with_output_bytes(bits))),
	bits,

Here bits (which should be bits) are passed to Blake2b::with_output_bytes() which expects to take in bytes. After some investigation I found that the bits are actually bytes. So the assignment to HashAlgorithm::bits is wrong.

I have found that this issue effects other parts of the code aswell.

[github-actions]

GNU testsuite comparison:

GNU test failed: tests/rm/rm2. tests/rm/rm2 is passing on 'main'. Maybe you have to rebase?
Skip an intermittent issue tests/tail/inotify-dir-recreate (fails in this run but passes in the 'main' branch)

[github-actions]

GNU testsuite comparison:

Skip an intermittent issue tests/tail/inotify-dir-recreate (fails in this run but passes in the 'main' branch)