Pádraig Brady <p...@draigbrady.com> writes:

> * src/digest.c (main): Don't saturate -l to BLAKE2B_MAX_LEN,
> so that the subsequent bounds check is performed.
> * tests/cksum/b2sum.sh: Add a test case.
> * NEWS: Mention the fix introduced in commit v9.5-71-gf2c84fe63
> ---
>  NEWS                 | 4 ++++
>  src/digest.c         | 2 +-
>  tests/cksum/b2sum.sh | 3 +++
>  3 files changed, 8 insertions(+), 1 deletion(-)
>
> diff --git a/NEWS b/NEWS
> index f2e7c9e6e..988cb96a8 100644
> --- a/NEWS
> +++ b/NEWS
> @@ -4,6 +4,10 @@ GNU coreutils NEWS                                    -*- 
> outline -*-
>  
>  ** Bug fixes
>  
> +  'b2sum' will diagnose --length values that are too big.
> +  Previously it would have silently assumed 512 for any larger values.
> +  [bug introduced in coreutils-9.6]

That doesn't seem like the worst behavior to me. But clearly was not the
intent, and I think the fixed behavior is better.

Thanks for the fix.

Collin

Reply via email to