Skip to content

bign-genk: Initial implementation#1109

Merged
tarcieri merged 9 commits intoRustCrypto:masterfrom
makavity:master
Mar 27, 2026
Merged

bign-genk: Initial implementation#1109
tarcieri merged 9 commits intoRustCrypto:masterfrom
makavity:master

Conversation

@makavity
Copy link
Copy Markdown
Contributor

@makavity makavity commented Nov 21, 2025

Hello, implementation for bignp256 rework.
https://apmi.bsu.by/assets/files/std/bign-spec295.pdf
image

@makavity makavity changed the title stb340145: Initial implementation bign-genk: Initial implementation Nov 23, 2025
@tarcieri
Copy link
Copy Markdown
Member

Is this actually defined for any elliptic curves other than bignp256? If not, it might make sense to include it with the elliptic curve implementation, the same way we do secp256k1 Schnorr and SM2DSA.

@makavity
Copy link
Copy Markdown
Contributor Author

makavity commented Mar 22, 2026

It is also used for bignp384 and bignp512

Copy link
Copy Markdown
Member

@tarcieri tarcieri left a comment

Choose a reason for hiding this comment

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

This seems like largely a copy of rfc6979 which has an implementation I'm not particularly happy with (see #1026) and inherits its flaws / TODOs, but I guess we need it to make progress on bignp256

@tarcieri tarcieri merged commit 32e8094 into RustCrypto:master Mar 27, 2026
88 checks passed
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