Skip to content

eddilithium: fail verification if signature is wrong length - #633

Merged
bwesterb merged 1 commit into
mainfrom
bas/gw3
Jul 14, 2026
Merged

eddilithium: fail verification if signature is wrong length#633
bwesterb merged 1 commit into
mainfrom
bas/gw3

Conversation

@bwesterb

@bwesterb bwesterb commented Jul 14, 2026

Copy link
Copy Markdown
Member

Earlier a short signature would cause a panic, and padded data would be ignored on a long signature. This was intentional, but perhaps surprising.


Open in Devin Review

Earlier a short signature would cause a panic, and padded data would be
ignored on a long signature. This was intentional, but perhaps
surprising.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread sign/eddilithium2/eddilithium.go
@bwesterb
bwesterb merged commit 6ebb88b into main Jul 14, 2026
14 checks passed
@bwesterb
bwesterb deleted the bas/gw3 branch July 14, 2026 18:44
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