Skip to content

Commit 3050edd

Browse files
committed
fixup! check n_sigs in partial_sig_agg
1 parent e3ca4b7 commit 3050edd

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

src/modules/musig/session_impl.h

+1
Original file line numberDiff line numberDiff line change
@@ -735,6 +735,7 @@ int secp256k1_musig_partial_sig_agg(const secp256k1_context* ctx, unsigned char
735735
ARG_CHECK(sig64 != NULL);
736736
ARG_CHECK(session != NULL);
737737
ARG_CHECK(partial_sigs != NULL);
738+
ARG_CHECK(n_sigs > 0);
738739

739740
if (!secp256k1_musig_session_load(ctx, &session_i, session)) {
740741
return 0;

src/modules/musig/tests_impl.h

+2-1
Original file line numberDiff line numberDiff line change
@@ -524,7 +524,8 @@ void musig_api_tests(secp256k1_scratch_space *scratch) {
524524
CHECK(ecount == 4);
525525
CHECK(secp256k1_musig_partial_sig_agg(none, pre_sig, &session, invalid_partial_sig_ptr, 2) == 0);
526526
CHECK(ecount == 5);
527-
CHECK(secp256k1_musig_partial_sig_agg(none, pre_sig, &session, partial_sig_ptr, 0) == 1);
527+
CHECK(secp256k1_musig_partial_sig_agg(none, pre_sig, &session, partial_sig_ptr, 0) == 0);
528+
CHECK(ecount == 6);
528529
CHECK(secp256k1_musig_partial_sig_agg(none, pre_sig, &session, partial_sig_ptr, 1) == 1);
529530
CHECK(secp256k1_musig_partial_sig_agg(none, pre_sig, &session, partial_sig_ptr, 2) == 1);
530531

0 commit comments

Comments
 (0)