Skip to content

Fix bad numbering in BIP340Signer comment#2340

Open
liamgilligan wants to merge 1 commit into
bcgit:mainfrom
liamgilligan:fix-bip340signer-comments
Open

Fix bad numbering in BIP340Signer comment#2340
liamgilligan wants to merge 1 commit into
bcgit:mainfrom
liamgilligan:fix-bip340signer-comments

Conversation

@liamgilligan

Copy link
Copy Markdown

Updates step numbering to match the actual number of steps being performed, as outlined in bip-0340.mediawiki. Note that these steps:

- Let rand = hashBIP0340/nonce(t || bytes(P) || m).
- Let k' = int(rand) mod n.

are folded into Step 6.

@ligefeiBouncycastle ligefeiBouncycastle self-requested a review July 1, 2026 00:16
@dghgit

dghgit commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

Thanks for noticing this. Merged with one small revision. I think the 9-10, 11-12 are okay, the off by one that occurred at step 1 got accounted for... Let me know if you think I'm confused.

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