GHSA-3fp5-2xwh-fxm6

Source

https://github.com/advisories/GHSA-3fp5-2xwh-fxm6

Import Source

https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2024/04/GHSA-3fp5-2xwh-fxm6/GHSA-3fp5-2xwh-fxm6.json

JSON Data

https://api.osv.dev/v1/vulns/GHSA-3fp5-2xwh-fxm6

Aliases

Published

2024-04-10T22:04:30Z

Modified

2024-06-05T16:43:15.355785Z

Severity

9.1 (Critical) CVSS_V3 - CVSS:3.1/AV:N/AC:L/PR:N/UI:N/S:U/C:N/I:H/A:H CVSS Calculator

Summary

Evmos transaction execution not accounting for all state transition after interaction with precompiles

Details

Context

stateObject: represents the state of an account and is used to store its updates during a state transition. This is accomplished using two in memory Storage variables: originStorage and dirtyStorage
StateDB: it is the general interface to retrieve accounts and holds a map of stateObjects.

Impact

An external contributor, @iczc, discovered a way to mint arbitrary tokens due to the possibility to have two different states not in sync during the execution of a transaction. The exploit is based on the fact that to sync the Cosmos SDK state and the EVM one, we rely on the stateDB.Commit() method. When we call this method, we iterate though all the dirtyStorage and, if and only if it is different than the originStorage, we set the new state. Setting the new state means we update the Cosmos SDK KVStore.

Below, are described the steps to perform the attack:

User send a tx to a smart contract (SC) that is calling a precompile.
The SC perform a state transition of its state from A to B.
The SC call the precompile.
The SC perform a state transition of its state from B to A (revert of the previous).
Once the transaction is executed, and the final Commit is performed, the state A will not be committed to the store because A is the same as originStorage.

If the tx is executed correctly, this is what happens at the store level:

Initial state A is loaded from the KVStore and the dirtyStorage is set to B.
Before running the precompile, the dirtyStorage is committed to the KVStore without changing the originStorage.
Now, since we have a dirtyStorage, it is updated to the previous value A without changing the originStorage.

Since the tx executed correctly, the evm calls the commit to persist the dirtyStorage. However, since dirtyStorage is equal to originStorage, nothing will be changed.

To summarize, if a contract storage state that is the same before and after a transaction, but is changed during the transaction and can call an external contract after the change, it can be exploited to make the transaction similar to non-atomic. The vulnerability is critical since this could lead to drain of funds through creative SC interactions.

Severity

Based on ImmuneFi Severity Classification System the severity was evaluated to Critical since the attack could have lead to direct loss of funds.

Patches

The issue has been patched in versions >=V17.0.0.

For more information

If you have any questions or comments about this advisory:

Reach out to the Core Team in Discord Open a discussion in evmos/evmos Email us at security@evmos.org for security questions

Database specific

{
    "nvd_published_at": "2024-04-19T15:15:50Z",
    "cwe_ids": [
        "CWE-662"
    ],
    "severity": "CRITICAL",
    "github_reviewed": true,
    "github_reviewed_at": "2024-04-10T22:04:30Z"
}

References

Affected packages

Go / github.com/evmos/evmos/v16

Package

Name: github.com/evmos/evmos/v16; View open source insights on deps.dev
Purl: pkg:golang/github.com/evmos/evmos/v16

Affected ranges

Type: SEMVER
Events: Introduced

0Unknown introduced version / All previous versions are affected

Fixed

17.0.0

Database specific

{
    "last_known_affected_version_range": "<= 16.0.4"
}

Go / github.com/evmos/evmos/v7

Package

Name: github.com/evmos/evmos/v7; View open source insights on deps.dev
Purl: pkg:golang/github.com/evmos/evmos/v7

Affected ranges

Type: SEMVER
Events: Introduced

0Unknown introduced version / All previous versions are affected

Last affected

7.0.0

Go / github.com/evmos/evmos/v6

Package

Name: github.com/evmos/evmos/v6; View open source insights on deps.dev
Purl: pkg:golang/github.com/evmos/evmos/v6

Affected ranges

Type: SEMVER
Events: Introduced

0Unknown introduced version / All previous versions are affected

Last affected

6.0.4

Go / github.com/evmos/evmos/v5

Package

Name: github.com/evmos/evmos/v5; View open source insights on deps.dev
Purl: pkg:golang/github.com/evmos/evmos/v5

Affected ranges

Type: SEMVER
Events: Introduced

0Unknown introduced version / All previous versions are affected

Last affected

5.0.0

Go / github.com/tharsis/evmos

Package

Name: github.com/tharsis/evmos; View open source insights on deps.dev
Purl: pkg:golang/github.com/tharsis/evmos

Affected ranges

Type: SEMVER
Events: Introduced

0Unknown introduced version / All previous versions are affected

Last affected

1.1.3

Go / github.com/tharsis/evmos/v2

Package

Name: github.com/tharsis/evmos/v2; View open source insights on deps.dev
Purl: pkg:golang/github.com/tharsis/evmos/v2

Affected ranges

Type: SEMVER
Events: Introduced

0Unknown introduced version / All previous versions are affected

Last affected

2.0.2

Go / github.com/tharsis/evmos/v3

Package

Name: github.com/tharsis/evmos/v3; View open source insights on deps.dev
Purl: pkg:golang/github.com/tharsis/evmos/v3

Affected ranges

Type: SEMVER
Events: Introduced

0Unknown introduced version / All previous versions are affected

Last affected

3.0.3

Go / github.com/tharsis/evmos/v4

Package

Name: github.com/tharsis/evmos/v4; View open source insights on deps.dev
Purl: pkg:golang/github.com/tharsis/evmos/v4

Affected ranges

Type: SEMVER
Events: Introduced

0Unknown introduced version / All previous versions are affected

Last affected

4.0.2

Go / github.com/tharsis/evmos/v5

Package

Name: github.com/tharsis/evmos/v5; View open source insights on deps.dev
Purl: pkg:golang/github.com/tharsis/evmos/v5

Affected ranges

Type: SEMVER
Events: Introduced

0Unknown introduced version / All previous versions are affected

Last affected

5.0.1