Skip to content

Commit 6b32584

Browse files
Eliminate na_1 and na_lam state fields from ecmult_strauss_wnaf.
1 parent 7050dda commit 6b32584

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

src/ecmult_impl.h

+4-4
Original file line numberDiff line numberDiff line change
@@ -442,7 +442,6 @@ static int secp256k1_ecmult_wnaf(int *wnaf, int len, const secp256k1_scalar *a,
442442
}
443443

444444
struct secp256k1_strauss_point_state {
445-
secp256k1_scalar na_1, na_lam;
446445
int wnaf_na_1[129];
447446
int wnaf_na_lam[129];
448447
int bits_na_1;
@@ -472,16 +471,17 @@ static void secp256k1_ecmult_strauss_wnaf(const secp256k1_ecmult_context *ctx, c
472471
size_t no = 0;
473472

474473
for (np = 0; np < num; ++np) {
474+
secp256k1_scalar na_1, na_lam;
475475
if (secp256k1_scalar_is_zero(&na[np]) || secp256k1_gej_is_infinity(&a[np])) {
476476
continue;
477477
}
478478
state->ps[no].input_pos = np;
479479
/* split na into na_1 and na_lam (where na = na_1 + na_lam*lambda, and na_1 and na_lam are ~128 bit) */
480-
secp256k1_scalar_split_lambda(&state->ps[no].na_1, &state->ps[no].na_lam, &na[np]);
480+
secp256k1_scalar_split_lambda(&na_1, &na_lam, &na[np]);
481481

482482
/* build wnaf representation for na_1 and na_lam. */
483-
state->ps[no].bits_na_1 = secp256k1_ecmult_wnaf(state->ps[no].wnaf_na_1, 129, &state->ps[no].na_1, WINDOW_A);
484-
state->ps[no].bits_na_lam = secp256k1_ecmult_wnaf(state->ps[no].wnaf_na_lam, 129, &state->ps[no].na_lam, WINDOW_A);
483+
state->ps[no].bits_na_1 = secp256k1_ecmult_wnaf(state->ps[no].wnaf_na_1, 129, &na_1, WINDOW_A);
484+
state->ps[no].bits_na_lam = secp256k1_ecmult_wnaf(state->ps[no].wnaf_na_lam, 129, &na_lam, WINDOW_A);
485485
VERIFY_CHECK(state->ps[no].bits_na_1 <= 129);
486486
VERIFY_CHECK(state->ps[no].bits_na_lam <= 129);
487487
if (state->ps[no].bits_na_1 > bits) {

0 commit comments

Comments
 (0)