From 17e575e3506e7b3f53ac621b0323321d199efa1d Mon Sep 17 00:00:00 2001 From: Steffen Eiden Date: Tue, 22 Oct 2024 10:13:05 +0200 Subject: [PATCH] rust/pv: Fix EC signature test The size of an EC-secp521r1 signature can be up to 139 bytes (if ASN1 encoded). There is no lower bound, although sizes lower than 137 are very unlikely but possible. Signed-off-by: Steffen Eiden --- rust/pv/src/crypto.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/rust/pv/src/crypto.rs b/rust/pv/src/crypto.rs index bb9903a9..d49c3367 100644 --- a/rust/pv/src/crypto.rs +++ b/rust/pv/src/crypto.rs @@ -350,7 +350,7 @@ mod tests { let data = "sample".as_bytes(); let sign = sign_msg(&ec_key, MessageDigest::sha512(), data).unwrap(); - assert!(sign.len() <= 139 && sign.len() >= 137); + assert!(sign.len() <= 139, "value is: {}", sign.len()); assert!(verify_signature(&ec_key, MessageDigest::sha512(), data, &sign).unwrap()); }