Skip to content

consensus/beacon: validate optimism params in Extra#766

Merged
nonsense merged 1 commit intooptimismfrom
nonsense/eip1559-val
Feb 17, 2026
Merged

consensus/beacon: validate optimism params in Extra#766
nonsense merged 1 commit intooptimismfrom
nonsense/eip1559-val

Conversation

@nonsense
Copy link
Copy Markdown
Contributor

No description provided.

@nonsense nonsense requested a review from a team as a code owner February 17, 2026 16:33
@nonsense nonsense merged commit 0ab188d into optimism Feb 17, 2026
8 checks passed
@nonsense nonsense deleted the nonsense/eip1559-val branch February 17, 2026 16:39
piersy pushed a commit to celo-org/op-geth that referenced this pull request Feb 18, 2026
piersy added a commit to celo-org/op-geth that referenced this pull request Feb 18, 2026
* crypto/ecies: fix ECIES invalid-curve handling (#33669)

Fix ECIES invalid-curve handling in RLPx handshake (reject invalid
ephemeral pubkeys early)
- Add curve validation in crypto/ecies.GenerateShared to reject invalid
public keys before ECDH.
- Update RLPx PoC test to assert invalid curve points fail with
ErrInvalidPublicKey.
 
Motivation / Context
RLPx handshake uses ECIES decryption on unauthenticated network input.
Prior to this change, an invalid-curve ephemeral public key would
proceed into ECDH and only fail at MAC verification, returning
ErrInvalidMessage. This allows an oracle on decrypt success/failure and
leaves the code path vulnerable to invalid-curve/small-subgroup attacks.
The fix enforces IsOnCurve validation up front.

* crypto/secp256k1: fix coordinate check

* version: release go-ethereum v1.16.9

* consensus/beacon: validate optimism params in Extra (ethereum-optimism#766)

---------

Co-authored-by: fengjian <445077+fengjian@users.noreply.github.com>
Co-authored-by: Felix Lange <fjl@twurst.com>
Co-authored-by: Anton Evangelatov <anton.evangelatov@oplabs.co>
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