Submission Details
Severity:
Zero Oracle price feed result
Summary
Function getTokenAmountFromUsd does not check for zero return value
Vulnerability Details
Function getTokenAmountFromUsd should affirm that the final calculated amount is more that zero before returning.
Impact
Liquidators risk getting zero collateral when the Chainlink oracle returns a zero price
Tools Used
Manual review
Recommendations
Ln 347 should first calculate the token amount then confirm that its more than zero.
uint256 tokenAmount = (usdAmountInWei * PRECISION) / (uint256(price) * ADDITIONAL_FEED_PRECISION);
require(tokenAmount > 0," Too low token amount");
Prize pool breakdown
Total prize
15,000 USDC
nSLOC
23664 USDC / LOC
14,000 USDC
1,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.