Skip to content
This repository has been archived by the owner on Oct 28, 2024. It is now read-only.

Eddsa Broken Signature Verification #169

Open
dxt99 opened this issue Oct 28, 2024 · 0 comments
Open

Eddsa Broken Signature Verification #169

dxt99 opened this issue Oct 28, 2024 · 0 comments

Comments

@dxt99
Copy link

dxt99 commented Oct 28, 2024

In ethsnarks/eddsa.py, there seems to be a missing check on the signature verification step for the param s. This param should be validated, e.g. that it is in the range 0<s<l where l is the order of the curve. Otherwise, an attacker may forge signatures from a known plaintext-signature pair by simply crafting a different s value that is still equal to the original modulo the order of the curve, for example by simply crafting s' = s + l where l is the order of the curve.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant