Flow

Sablier
FoundryDeFi
20,000 USDC
View results
Submission Details
Severity: low
Valid

Valid ERC20s with no `decimals()` implementation are not supported

Vulnerability Details

As per the ERC20 standard, the decimals() function is OPTIONAL.

Thus, some valid ERC20 tokens may not have the decimals() function implemented. Thus reverting on the _create() function here:

uint8 tokenDecimals = IERC20Metadata(address(token)).decimals();

Impact

Expected supported ERC20 tokens, as per the contest docs Scope/Compatibility section and ERC20 specification are not supported.

Recommendations

Add to the list of supported tokens those that have the decimals() function implemented.

Updates

Lead Judging Commences

inallhonesty Lead Judge 9 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Tokens without the decimals() implementation

Appeal created

charlescheerful Submitter
9 months ago
inallhonesty Lead Judge
9 months ago
inallhonesty Lead Judge
9 months ago
inallhonesty Lead Judge 9 months ago
Submission Judgement Published
Validated
Assigned finding tags:

Tokens without the decimals() implementation

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.