Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump Ethers 5.0.12 to 5.0.30 for CVE-2020-28498 #142

Closed
wants to merge 2 commits into from
Closed

Bump Ethers 5.0.12 to 5.0.30 for CVE-2020-28498 #142

wants to merge 2 commits into from

Conversation

breakabort
Copy link
Contributor

Ethers < 5.0.30 used Elliptic.js 6.5.3 which did not check to ensure that the public key passed in to ECDH is a point that actually exists on the curve.
This opens up for a twist attack which could be used to reveal the private key of a party in an ECDH operation over a number of occurances.
ethers-io/ethers.js#1284

https://github.com/christianlundkvist/blog/blob/master/2020_05_26_secp256k1_twist_attacks/secp256k1_twist_attacks.md
CVE: CVE-2020-28498

breakabort and others added 2 commits February 9, 2021 16:11
Ethers < 5.0.30 used Elliptic.js 6.5.3 which did not check to ensure that the public key passed in to ECDH is a point that actually exists on the curve.
This opens up for a twist attack which could be used to reveal the private key of a party in an ECDH operation over a number of occurances.
ethers-io/ethers.js#1284

https://github.com/christianlundkvist/blog/blob/master/2020_05_26_secp256k1_twist_attacks/secp256k1_twist_attacks.md
CVE: CVE-2020-28498
@rauljordan
Copy link
Contributor

Hi @somebodyrandom ran an npm update and pushed the code to master, thanks for finding

@rauljordan rauljordan closed this Feb 12, 2021
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