From a93cfbbf95413f6e1e60e5e096c3fc262dc39a42 Mon Sep 17 00:00:00 2001 From: Guilherme Ferreira Date: Wed, 30 Aug 2023 18:19:58 -0300 Subject: [PATCH] change test case that should fail with s above upper limit --- crypto/crypto_test.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/crypto/crypto_test.go b/crypto/crypto_test.go index 157aee8d1d..4017b9cdbe 100644 --- a/crypto/crypto_test.go +++ b/crypto/crypto_test.go @@ -258,8 +258,8 @@ func TestValidateSignatureValues(t *testing.T) { check(false, 1, zero, one) check(false, 1, one, zero) - // correct sig with max r,s - check(true, 0, secp256k1nMinus1, secp256k1nMinus1) + // correct sig with max r and s above upper limit + check(false, 0, secp256k1nMinus1, secp256k1nMinus1) // correct v, combinations of incorrect r,s at upper limit check(false, 0, secp256k1N, secp256k1nMinus1) check(false, 0, secp256k1nMinus1, secp256k1N)