GitHub Advisoryadvisory · npm

GHSA-cvj7-5f3c-9vg9 AttesterSlashing number overflow

HIGH Published 2022-05-24 Modified 2022-05-24 Official page ↗
Aliases
CVE-2022-29219
Ecosystem
npm
Packages
@chainsafe/lodestar
Severity
High
CVSS vector
CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:H/A:N
Weaknesses
CWE-190
Reviewed
2022-05-24

Affected packages

• npm @chainsafe/lodestar: < 0.36.0 → fixed in 0.36.0

Description

Impact Possible consensus split given maliciously-crafted `AttesterSlashing` or `ProposerSlashing` being included on-chain. Since we represent `uint64` values as native javascript `number`s, there is an issue when those variables with large (greater than 2^53) `uint64` values are included on chain. In those cases, Lodestar may view _valid_ `AttesterSlashing` or `ProposerSlashing` as _invalid_, due to rounding errors in large `number` values. This causes a consensus split, where Lodestar nodes are forked away from the main network. Similarly Lodestar may consider _invalid_ `ProposerSlashing` as _valid_, thus including in proposed blocks that will be considered invalid by the network. Patches https://github.com/ChainSafe/lodestar/pull/3977 Workarounds Use `BigInt` to represent `Slot` and `Epoch` values in `AttesterSlashing` and `ProposerSlashing` objects. `BigInt` is too slow to be used in all `Slot` and `Epoch` cases, so we will carefully use `BigInt` just where necessary for consensus.

Related

Alias · CVE (1)

  • CVE-2022-29219 Lodestar is a TypeScript implementation of the Ethereum Consensus specification. vulnerability

Weakness · CWE (1)

  • CWE-190 Integer Overflow or Wraparound weakness

References