crytic / crytic/building-secure-contracts

Add more risks to the token integration checklist

Open
#67 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Solidity
Stars
2.5k
Forks
391
PR merge metrics
No merged PRs in 30d

Description

There are a couple of risks that we can add:

- Risks associated with tokens that have external calls where the function id is controlled
- Decimals is not always 18
- Risks associated with rebalancing tokens

We have to review past audits to see what else is missing

Contributor guide

Open the contributing guide

Research direction

Start by locating the token integration checklist and reviewing past audits, as requested in the issue. Done means the checklist covers externally controlled function IDs, non-18 decimals, rebalancing-token risks, and any additional justified gaps.

Written by the indexing model from the issue text.

Assessment

Tech stack
solidity
Domain
blockchain, documentation, security
Issue type
Documentation
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.