Submission Details
Severity:
Calls will revert when `totalSupply()` returns zero
Summary
totalSupply() being zero will result in a division by zero,
causing the transaction to fail.
The function should instead special-case this scenario, and avoid reverting.
Vulnerability Details
There is 1 instance of this issue:
_shareRatio = shareAmt * SAFE_MULTIPLIER / IERC20(address(self.vault)).totalSupply() should check if
totalSupply()greater than zero before division.
Impact
Possibly dividing by 0 if the totalSupply ever returns a zero.
Tools Used
Manual Review
Recommendations
Checking if totalSupply() greater than zero before division.
Prize pool breakdown
Total prize
35,000 USDC
nSLOC
2,28915
USDC / LOC
33,000 USDC
2,000 USDC
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.