Submission Details
Severity:
Missing Validation on chainlink data
Summary
The DSCEngine contract relies on the chainlink price feed, however there is no validation on the result which may bring the stale price.
Vulnerability Details
The contract relies on the following code for the price purpose:
function staleCheckLatestRoundData(AggregatorV3Interface priceFeed)
public
view
returns (uint80, int256, uint256, uint256, uint80)
{
(uint80 roundId, int256 answer, uint256 startedAt, uint256 updatedAt, uint80 answeredInRound) =
priceFeed.latestRoundData();
uint256 secondsSince = block.timestamp - updatedAt;
if (secondsSince > TIMEOUT) revert OracleLib__StalePrice();
return (roundId, answer, startedAt, updatedAt, answeredInRound);
}
However not all return value have been checked that answer should larger than 0
Impact
May bring invalid price when
Tools Used
Manual
Recommendations
Adding validation to avoid unexpected data
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.