GHSA-m6w8-fq7v-ph4m

Source

https://github.com/advisories/GHSA-m6w8-fq7v-ph4m

Import Source

https://github.com/github/advisory-database/blob/main/advisories/github-reviewed/2022/01/GHSA-m6w8-fq7v-ph4m/GHSA-m6w8-fq7v-ph4m.json

JSON Data

https://api.osv.dev/v1/vulns/GHSA-m6w8-fq7v-ph4m

Published

2022-01-13T16:09:36Z

Modified

2022-01-12T21:38:27Z

Summary

GovernorCompatibilityBravo incorrect ABI encoding may lead to unexpected behavior

Details

Impact

The GovernorCompatibilityBravo module may lead to the creation of governance proposals that execute function calls with incorrect arguments due to bad ABI encoding. This happens if the proposal is created using explicit function signatures, e.g. a proposal to invoke the function foo(uint256) is created as propose([target], [0], ["foo(uint256)"], ["0x00..01"]). If the function selector is provided as part of the encoded proposal data the issue is not present, e.g. the same proposal is created as propose([target], [0], ["0x2fbebd3800..01"]), where 2fbebd38 is the function selector.

We've assessed the instances of this contract found on chain, and did not find any occurrence of this bug in the past. Proposal creation through Tally or OpenZeppelin Defender is not affected. The core Governor contract on its own is not affected.

Patches

A fix is included in version v4.4.2 of @openzeppelin/contracts and @openzeppelin/contracts-upgradeable.

Workarounds

Do not create proposals using explicit function signatures. Instead, use the propose function without the signatures argument, and create the proposal using the fully ABI-encoded function call including the function selector in the calldatas argument as explained above.

References

https://github.com/OpenZeppelin/openzeppelin-contracts/issues/3099

Credits

This issue was identified and reported by @GeraldHost.

For more information

If you have any questions, comments, or need assistance regarding this advisory, email us at security@openzeppelin.com.

To submit security reports please use our bug bounty on Immunefi.

Database specific

{
    "severity": "MODERATE",
    "nvd_published_at": null,
    "github_reviewed_at": "2022-01-11T23:50:55Z",
    "github_reviewed": true,
    "cwe_ids": []
}

References

Affected packages

npm / @openzeppelin/contracts

Package

Name: @openzeppelin/contracts; View open source insights on deps.dev
Purl: pkg:npm/%40openzeppelin/contracts

Affected ranges

Type: SEMVER
Events: Introduced

4.3.0

Fixed

4.4.2

npm / @openzeppelin/contracts-upgradeable

Package

Name: @openzeppelin/contracts-upgradeable; View open source insights on deps.dev
Purl: pkg:npm/%40openzeppelin/contracts-upgradeable

Affected ranges

Type: SEMVER
Events: Introduced

4.3.0

Fixed

4.4.2