Skip to content

Commit 8d967a6

Browse files
musig/test: Remove dead code
This avoids a compiler warning on clang-snapshot about &keypair being uninitialized.
1 parent 983711c commit 8d967a6

File tree

1 file changed

+2
-5
lines changed

1 file changed

+2
-5
lines changed

src/modules/musig/tests_impl.h

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -927,12 +927,9 @@ static void musig_test_vectors_signverify(void) {
927927
CHECK(secp256k1_musig_nonce_process(CTX, &session, &aggnonce, vector->msgs[c->msg_index], &keyagg_cache));
928928

929929
expected = c->error != MUSIG_SECNONCE;
930+
CHECK(!expected);
930931
musig_test_set_secnonce(&secnonce, vector->secnonces[c->secnonce_index], &pubkey);
931-
if (expected) {
932-
CHECK(secp256k1_musig_partial_sign(CTX, &partial_sig, &secnonce, &keypair, &keyagg_cache, &session));
933-
} else {
934-
CHECK_ILLEGAL(CTX, secp256k1_musig_partial_sign(CTX, &partial_sig, &secnonce, &keypair, &keyagg_cache, &session));
935-
}
932+
CHECK_ILLEGAL(CTX, secp256k1_musig_partial_sign(CTX, &partial_sig, &secnonce, &keypair, &keyagg_cache, &session));
936933
}
937934
for (i = 0; i < sizeof(vector->verify_fail_case)/sizeof(vector->verify_fail_case[0]); i++) {
938935
const struct musig_verify_fail_error_case *c = &vector->verify_fail_case[i];

0 commit comments

Comments
 (0)