Submission Details
Severity:
Users will spent more Gas if ```amount == 0```
Summary
In ZENO.solif user call redeemAll function when balance is zero. Then user will spent more gas on calling reedemAllfunction . Because there is no checking for amount==0 or not. If that is present there then user will spent less gas when their amout is zero.
Vulnerability Details
function redeemAll() external nonReentrant {
if (!isRedeemable()) {
revert BondNotRedeemable();
}
+ if(amount ==0 ) revert AmountIsZero;
uint256 amount = balanceOf(msg.sender);
totalZENORedeemed += amount;
_burn(msg.sender, amount);
USDC.safeTransfer(msg.sender, amount);
}
Impact
User will spent more gas when amount==0and call redeemAllFunction.
Tools Used
Maual Review
Recommendations
Prize pool breakdown
Total prize
77,280 USDC
nSLOC
3,86420 USDC / LOC
74,000
3,280
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.