BriVault

First Flight #52
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Impact: low
Likelihood: low
Invalid

`stakedAmount::BriVault` is unused variable in the contract

Root + Impact : stakedAmount::BriVault is unused, hence increasing gas fee.

Description

  • stakedAmount::BriVault is not used anywhere in the whole contract, which means there is no use of keeping this variable.

  • This extra public state variable is increasing the gas fee, hence is not optimized.

contract BriVault is ERC4626, Ownable {
...
@> uint256 public stakedAmount;
...
}

Risk

Likelihood: Low

Impact: Low/Gas

Recommended Mitigation

  • It is simply recommended to remove the variable directly from the contract as it is not used anywhere in the contract.

contract BriVault is ERC4626, Ownable {
...
- uint256 public stakedAmount;
...
}
Updates

Appeal created

bube Lead Judge 19 days ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Gas optimizations

Gas optimizations are invalid according to the CodeHawks documentation.

Support

FAQs

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

Give us feedback!