Skip to content

Commit 9554362

Browse files
committed
tests: call secp256k1_ecmult_multi_var with a non-NULL error callback
Function secp256k1_ecmult_multi_var expects to be called with a non-NULL error_callback parameter. Fix the invocation in test_ecmult_accumulate to do this. While at it, wrap the call in a CHECK macro to ensure it succeeds. Fixes: #1527
1 parent 7712a53 commit 9554362

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

src/tests.c

+2-2
Original file line numberDiff line numberDiff line change
@@ -5494,8 +5494,8 @@ static void test_ecmult_accumulate(secp256k1_sha256* acc, const secp256k1_scalar
54945494
secp256k1_ecmult_gen(&CTX->ecmult_gen_ctx, &rj1, x);
54955495
secp256k1_ecmult(&rj2, &gj, x, &secp256k1_scalar_zero);
54965496
secp256k1_ecmult(&rj3, &infj, &secp256k1_scalar_zero, x);
5497-
secp256k1_ecmult_multi_var(NULL, scratch, &rj4, x, NULL, NULL, 0);
5498-
secp256k1_ecmult_multi_var(NULL, scratch, &rj5, &secp256k1_scalar_zero, test_ecmult_accumulate_cb, (void*)x, 1);
5497+
CHECK(secp256k1_ecmult_multi_var(&CTX->error_callback, scratch, &rj4, x, NULL, NULL, 0));
5498+
CHECK(secp256k1_ecmult_multi_var(&CTX->error_callback, scratch, &rj5, &secp256k1_scalar_zero, test_ecmult_accumulate_cb, (void*)x, 1));
54995499
secp256k1_ecmult_const(&rj6, &secp256k1_ge_const_g, x);
55005500
secp256k1_ge_set_gej_var(&r, &rj1);
55015501
CHECK(secp256k1_gej_eq_ge_var(&rj2, &r));

0 commit comments

Comments
 (0)