crytic / crytic/slither

slither-check-erc: detect if the token allows transfer to zero

Open
#603 1 comment 1 reaction 0 assignees View on GitHub
Dominant language
Python
Stars
6.4k
Forks
1.1k
PR merge metrics
No merged PRs in 30d

Description

If the token reverts when the value is zero, it might break third-party integration.

Additionally, several tokens require the approval to be zero before calling approve with a non-zero value, to prevent the ERC20 race condition. We could also report it.

It's probably worth to update our [token integration](https://github.com/crytic/building-secure-contracts/blob/master/development-guidelines/token_integration.md) checklist with these items

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.