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

Mixed usage of `int`/`uint` with `int256`/`uint256`

Summary

There are two instances where uint has been used and everywhere else uint256 is used

Vulnerability Details

It is recommended to use one form consistently throughout all contracts. int256/uint256 are the preferred type names as they're used for function signatures

38 function deposit(uint _amount) external {
46 function withdraw(uint _amount) external {

https://github.com/Cyfrin/2023-07-beedle/blob/main/src/Staking.sol#L38
https://github.com/Cyfrin/2023-07-beedle/blob/main/src/Staking.sol#L46

Impact

Informational

Tools Used

Manual review

Recommendations

Use uint256 consistently across all contracts

Support

FAQs

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