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.

Give us feedback!