Submission Details
Impact:
Likelihood:
`notRegistered()::BriVault` error is unused
Root + Impact : notRegistered()::BriVault error is unused, increasing gas fee
Description
-
notRegistered()::BriVaulterror is not used anywhere in the code, which means there is no use of keeping this error. -
This extra public state variable is increasing the gas fee, hence is not optimized.
contract BriVault is ERC4626, Ownable {
...
@> error notRegistered();
...
}
Risk
Likelihood: Low
Impact: Low
Recommended Mitigation
It is best recommended to remove the error from the code completely, as it's not used anywhere in the code and also increasing the gas price unnecessarily.
contract BriVault is ERC4626, Ownable {
...
- error notRegistered();
...
}
Updates
Appeal created
bube 19 days ago
Submission Judgement Published Reason: Non-acceptable severity
Assigned finding tags:
Gas optimizations
Gas optimizations are invalid according to the CodeHawks documentation.
Rewards breakdown
nSLOC
193This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.