change test case that should fail with s above upper limit

dominant-strategies · Aug 30, 2023 · a93cfbb · a93cfbb
1 parent 1eb811c
commit a93cfbb
Showing 1 changed file with 2 additions and 2 deletions.
diff --git 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)