Skip to content

fuzzamoto-ir: implement BuildPayToBareMulti#129

Open
brunoerg wants to merge 1 commit intodergoegge:masterfrom
brunoerg:2026-02-bare-multisig
Open

fuzzamoto-ir: implement BuildPayToBareMulti#129
brunoerg wants to merge 1 commit intodergoegge:masterfrom
brunoerg:2026-02-bare-multisig

Conversation

@brunoerg
Copy link
Contributor

No description provided.

@brunoerg brunoerg marked this pull request as ready for review February 24, 2026 20:51
@brunoerg brunoerg force-pushed the 2026-02-bare-multisig branch from d0674f0 to 632cc61 Compare February 25, 2026 12:42
@brunoerg brunoerg marked this pull request as draft February 25, 2026 13:09
@brunoerg brunoerg force-pushed the 2026-02-bare-multisig branch from 632cc61 to 1ad8f95 Compare February 25, 2026 13:12
@brunoerg brunoerg marked this pull request as ready for review February 25, 2026 13:39
let mut sig_script_builder =
ScriptBuf::builder().push_opcode(bitcoin::opcodes::OP_0);

for sk_bytes in private_keys.iter().take(*required as usize) {
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will this ever result in a p2ms that has less signatures than required?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, because of let required_clamped = (*required).min(n).max(1); which is the used required here. So it can never produce fewer signatures than required.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants