Skip to content

Commit e3497bb

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 d79a6cc commit e3497bb

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;
@@ -4797,12 +4796,12 @@ static void test_ecmult_multi(secp256k1_scratch *scratch, secp256k1_ecmult_multi
47974796
testutil_random_ge_test(&pt[ncount]);
47984797
}
47994798

4800-
secp256k1_scalar_clear(&sc[0]);
4799+
secp256k1_scalar_set_int(&sc[0], 0);
48014800
CHECK(ecmult_multi(&CTX->error_callback, scratch, &r, &secp256k1_scalar_zero, ecmult_multi_callback, &data, 20));
4802-
secp256k1_scalar_clear(&sc[1]);
4803-
secp256k1_scalar_clear(&sc[2]);
4804-
secp256k1_scalar_clear(&sc[3]);
4805-
secp256k1_scalar_clear(&sc[4]);
4801+
secp256k1_scalar_set_int(&sc[1], 0);
4802+
secp256k1_scalar_set_int(&sc[2], 0);
4803+
secp256k1_scalar_set_int(&sc[3], 0);
4804+
secp256k1_scalar_set_int(&sc[4], 0);
48064805
CHECK(ecmult_multi(&CTX->error_callback, scratch, &r, &secp256k1_scalar_zero, ecmult_multi_callback, &data, 6));
48074806
CHECK(ecmult_multi(&CTX->error_callback, scratch, &r, &secp256k1_scalar_zero, ecmult_multi_callback, &data, 5));
48084807
CHECK(secp256k1_gej_is_infinity(&r));

0 commit comments

Comments
 (0)