Skip to content

Commit f532b9d

Browse files
real-or-randomisle2983sipa
authored andcommitted
Separate between clearing memory and setting to zero in tests
Co-Authored-By: isle2983 <isle2983@yahoo.com> Co-Authored-By: Pieter Wuille <pieter.wuille@gmail.com>
1 parent a2721b4 commit f532b9d

File tree

2 files changed

+7
-8
lines changed

2 files changed

+7
-8
lines changed

src/bench_ecmult.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ static void bench_ecmult_teardown_helper(bench_data* data, size_t* seckey_offset
7171
secp256k1_scalar sum_scalars;
7272

7373
secp256k1_gej_set_infinity(&sum_output);
74-
secp256k1_scalar_clear(&sum_scalars);
74+
secp256k1_scalar_set_int(&sum_scalars, 0);
7575
for (i = 0; i < iters; ++i) {
7676
secp256k1_gej_add_var(&sum_output, &sum_output, &data->output[i], NULL);
7777
if (scalar_gen_offset != NULL) {

src/tests.c

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -3671,8 +3671,7 @@ static void test_ge(void) {
36713671
secp256k1_fe zfi2, zfi3;
36723672

36733673
secp256k1_gej_set_infinity(&gej[0]);
3674-
secp256k1_ge_clear(&ge[0]);
3675-
secp256k1_ge_set_gej_var(&ge[0], &gej[0]);
3674+
secp256k1_ge_set_infinity(&ge[0]);
36763675
for (i = 0; i < runs; i++) {
36773676
int j, k;
36783677
secp256k1_ge g;
@@ -4768,12 +4767,12 @@ static void test_ecmult_multi(secp256k1_scratch *scratch, secp256k1_ecmult_multi
47684767
testutil_random_ge_test(&pt[ncount]);
47694768
}
47704769

4771-
secp256k1_scalar_clear(&sc[0]);
4770+
secp256k1_scalar_set_int(&sc[0], 0);
47724771
CHECK(ecmult_multi(&CTX->error_callback, scratch, &r, &secp256k1_scalar_zero, ecmult_multi_callback, &data, 20));
4773-
secp256k1_scalar_clear(&sc[1]);
4774-
secp256k1_scalar_clear(&sc[2]);
4775-
secp256k1_scalar_clear(&sc[3]);
4776-
secp256k1_scalar_clear(&sc[4]);
4772+
secp256k1_scalar_set_int(&sc[1], 0);
4773+
secp256k1_scalar_set_int(&sc[2], 0);
4774+
secp256k1_scalar_set_int(&sc[3], 0);
4775+
secp256k1_scalar_set_int(&sc[4], 0);
47774776
CHECK(ecmult_multi(&CTX->error_callback, scratch, &r, &secp256k1_scalar_zero, ecmult_multi_callback, &data, 6));
47784777
CHECK(ecmult_multi(&CTX->error_callback, scratch, &r, &secp256k1_scalar_zero, ecmult_multi_callback, &data, 5));
47794778
CHECK(secp256k1_gej_is_infinity(&r));

0 commit comments

Comments
 (0)