20,000 USDC
View results
Submission Details
Severity: gas
Valid

Implicit use of uint

Summary

uint without defining if uint8, uint32 uint256 was intended

Vulnerability Details

Staking.sol line 38 -> function deposit(uint _amount) external { // uint defaults to uint256 was that the intention?
Staking.sol line 46 -> function withdraw(uint _amount) external { //maybe intention was to use uint16 to limit amount size?
With implicit use of uint questions above are valid as its not clear if the default of uint256 was desired or smaller uints

Impact

Informational: This impacts code readability, code interpretability, may even introduce errors as there may be misunderstanding if smaller uint were intended. It is best practise to explicitly state uint as uint256 if that is intended desire

Tools Used

Manual Analyis

Recommendations

Explicitly use uint as uint256

Support

FAQs

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

Give us feedback!