On 16/04/2026 16:03, Bruno Haible wrote:
On Slackware 15 (32-bit x86), one test fails:FAIL: tests/cksum/b2sum
This is more fallout from a recent change to use idx_t. I.e., we're now assigning intmax_t to idx_t, hence the error mismatch on 32 bit. The attached should address this. cheers, Padraig
From 852c056889bb39207af3c8af80eeefd2e98b29f3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?P=C3=A1draig=20Brady?= <[email protected]> Date: Thu, 16 Apr 2026 20:37:28 +0100 Subject: [PATCH] cksum: fix --length validation on 32 bit platforms Fix an unreleased issue due to the recent change to using idx_t in commit v9.10-91-g02983e493 * src/cksum.c (main): Limit the possible return to the range supported by idx_t. Reported by Bruno Haible. --- src/cksum.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/cksum.c b/src/cksum.c index 79310a0d9..3a59e45ca 100644 --- a/src/cksum.c +++ b/src/cksum.c @@ -1626,7 +1626,7 @@ main (int argc, char **argv) #endif #if HASH_ALGO_BLAKE2 || HASH_ALGO_CKSUM case 'l': - digest_length = xnumtoimax (optarg, 10, 0, INTMAX_MAX, "", + digest_length = xnumtoimax (optarg, 10, 0, IDX_MAX, "", _("invalid length"), 0, XTOINT_MAX_QUIET); digest_length_str = optarg; -- 2.53.0
