Skip to content

elliptic-curve: add weierstrass::point::Decompress trait#266

Merged
tarcieri merged 1 commit into
masterfrom
elliptic-curve/point-decompression-trait
Aug 23, 2020
Merged

elliptic-curve: add weierstrass::point::Decompress trait#266
tarcieri merged 1 commit into
masterfrom
elliptic-curve/point-decompression-trait

Conversation

@tarcieri

Copy link
Copy Markdown
Member

Adds a trait for decompressing Weierstrass points from a big endian encoded affine x-coordinate and a flag indicating whether or not the y-coordinate is odd.

Adds a trait for decompressing Weierstrass points from a big endian
encoded affine x-coordinate and a flag indicating whether or not the
y-coordinate is odd.
@tarcieri tarcieri requested review from fjarri and str4d August 22, 2020 19:39
@tarcieri

Copy link
Copy Markdown
Member Author

As discussed in #264, adds a trait for decompressing Weierstrass curve points intended to encapsulate what's presently tied to the SEC1 formats at a higher level:

https://github.com/RustCrypto/elliptic-curves/blob/68a83c8/k256/src/arithmetic.rs#L109-L128

This will also be useful for eliminating duplication in point decompression scenarios that aren't necessarily tied to the SEC1 formats, e.g.:

https://github.com/RustCrypto/elliptic-curves/blob/4ca0987/k256/src/ecdsa/recoverable.rs#L128-L144

@tarcieri tarcieri merged commit acfc369 into master Aug 23, 2020
@tarcieri tarcieri deleted the elliptic-curve/point-decompression-trait branch August 23, 2020 16:37
@tarcieri tarcieri mentioned this pull request Sep 11, 2020
dns2utf8 pushed a commit to dns2utf8/traits that referenced this pull request Jan 24, 2023
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.

1 participant