Skip to content

EC updates - #539

Merged
jgriffiths merged 3 commits into
masterfrom
ec_updates
Jul 31, 2026
Merged

EC updates#539
jgriffiths merged 3 commits into
masterfrom
ec_updates

Conversation

@jgriffiths

Copy link
Copy Markdown
Contributor

No description provided.

jgriffiths and others added 3 commits July 31, 2026 13:38
libsecp will happily parse and return such signatures without error,
however they can never be valid (in the overlong case, because libsecp
internally sets R to zero). Reject them up-front when parsing rather
than failing when attempting to use them later.
Allow both compress and decompress to work (as a no-op) on already
compressed/decompressed pubkeys.

Co-authored-by: odudex <odudex@proton.me>
@jgriffiths

jgriffiths commented Jul 31, 2026

Copy link
Copy Markdown
Contributor Author

@odudex FYI - I've included your review changes to the compress/decompress impl from #535 - thanks!

@jgriffiths
jgriffiths merged commit 03351cc into master Jul 31, 2026
5 checks passed
@jgriffiths
jgriffiths deleted the ec_updates branch July 31, 2026 06:29
@odudex

odudex commented Jul 31, 2026

Copy link
Copy Markdown
Contributor

@odudex FYI - I've included your review changes to the compress/decompress impl from #535 - thanks!

🫡 #535 rebased!

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