the chainlinkOracle.sol contract specially the getChainlinkPrice function using the aggregator v2 and v3 to get/call the latestRoundData. the function should check for the min and max amount return to prevent some case happen, something like this:
<https://solodit.xyz/issues/missing-checks-for-chainlink-oracle-spearbit-connext-pdf>
<https://solodit.xyz/issues/m-16-chainlinkadapteroracle-will-return-the-wrong-price-for-asset-if-underlying-aggregator-hits-minanswer-sherlock-blueberry-blueberry-git>
if case like luna happen then the oracle will return the minimum price and not the crashed price.
the function getChainlinkPrice:
function getChainlinkPrice(
AggregatorV3Interface feed
) internal view returns (uint256) {
(, int256 answer, , uint256 updatedAt, ) = AggregatorV3Interface(feed)
.latestRoundData();
require(answer > 0, "Chainlink price cannot be lower than 0");
require(updatedAt != 0, "Round is in incompleted state");
// Chainlink USD-denominated feeds store answers at 8 decimals
uint256 decimalDelta = uint256(18).sub(feed.decimals());
// Ensure that we don't multiply the result by 0
if (decimalDelta > 0) {
return uint256(answer).mul(10 ** decimalDelta);
} else {
return uint256(answer);
}
}
the function did not check for the min and max price.
manual review
some check like this can be added to avoid returning of the min price or the max price in case of the price crashes.
require(answer < _maxPrice, "Upper price bound breached");
require(answer > _minPrice, "Lower price bound breached");
Oracle
The text was updated successfully, but these errors were encountered:
All reactions