Submission Details
Severity:
`GMXReader::svTokenValue()`, `GMXReader::valueToShares()`, `GMXReader::delta` variable initialisation
Summary
No need to initialize variables to 0 separately in solidity
Vulnerability Details
GMXReader::svTokenValue()
@> if (equityValue_ == 0 || totalSupply_ == 0) return SAFE_MULTIPLIER;
,
GMXReader::valueToShares()
@> if (_sharesSupply == 0 || currentEquity == 0) return value;
, GMXReader::delta
@> if (_tokenAAmt == 0 && _tokenADebtAmt == 0) return 0;
@> if (equityValue_ == 0) return 0;
Tools Used
Manual Review
Recommendations
Write the variable name only solidity initializes variables to 0 by default
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.