Skip to content

ERC165Checker may revert instead of returning false

High
frangio published GHSA-qh9x-gcfh-pcrw Jul 20, 2022

Package

npm @openzeppelin/contracts (npm)

Affected versions

>=4.0.0 <4.7.1

Patched versions

4.7.1
npm @openzeppelin/contracts-upgradeable (npm)
>=4.0.0 <4.7.1
4.7.1

Description

Impact

ERC165Checker.supportsInterface is designed to always successfully return a boolean, and under no circumstance revert. However, an incorrect assumption about Solidity 0.8's abi.decode allows some cases to revert, given a target contract that doesn't implement EIP-165 as expected, specifically if it returns a value other than 0 or 1.

The contracts that may be affected are those that use ERC165Checker to check for support for an interface and then handle the lack of support in a way other than reverting.

Patches

The issue was patched in 4.7.1.

References

#3552

For more information

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

Severity

High

CVE ID

CVE-2022-31170

Weaknesses

No CWEs