Skip to content

Commit f16a709

Browse files
committed
Merge #1247: Apply Checks only in VERIFY mode.
4ebd828 Apply Checks only in VERIFY mode. (roconnor-blockstream) Pull request description: This is already done in `field_5x52_impl.h`. ACKs for top commit: sipa: ACK 4ebd828 jonasnick: ACK 4ebd828 Tree-SHA512: c24211e5219907e41e2c5792255734bd50ca5866a4863abbb3ec174ed92d1792dd10563a94c08e8fecd6cdf776a9c49ca87e8f9806a023d9081ecc0d55ae3e66
2 parents 70be3ca + 4ebd828 commit f16a709

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

src/field_10x26_impl.h

+4
Original file line numberDiff line numberDiff line change
@@ -1363,7 +1363,9 @@ static void secp256k1_fe_inv(secp256k1_fe *r, const secp256k1_fe *x) {
13631363
secp256k1_modinv32(&s, &secp256k1_const_modinfo_fe);
13641364
secp256k1_fe_from_signed30(r, &s);
13651365

1366+
#ifdef VERIFY
13661367
VERIFY_CHECK(secp256k1_fe_normalizes_to_zero(r) == secp256k1_fe_normalizes_to_zero(&tmp));
1368+
#endif
13671369
}
13681370

13691371
static void secp256k1_fe_inv_var(secp256k1_fe *r, const secp256k1_fe *x) {
@@ -1376,7 +1378,9 @@ static void secp256k1_fe_inv_var(secp256k1_fe *r, const secp256k1_fe *x) {
13761378
secp256k1_modinv32_var(&s, &secp256k1_const_modinfo_fe);
13771379
secp256k1_fe_from_signed30(r, &s);
13781380

1381+
#ifdef VERIFY
13791382
VERIFY_CHECK(secp256k1_fe_normalizes_to_zero(r) == secp256k1_fe_normalizes_to_zero(&tmp));
1383+
#endif
13801384
}
13811385

13821386
static int secp256k1_fe_is_square_var(const secp256k1_fe *x) {

0 commit comments

Comments
 (0)