We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
latestRoundData()
Medium Risk
https://github.com/Cyfrin/2023-07-foundry-defi-stablecoin/blob/main/src/libraries/OracleLib.sol#L26
This is the latestRoundData() problem, the contract call latestRoundData() but does not check the return value
This could lead to stale prices according to the Chainlink documentation: https://docs.chain.link/data-feeds/price-feeds/historical-data
Related report: code-423n4/2021-05-fairside-findings#70
vs code
we can refer to : sherlock-audit/2023-02-blueberry-judging#94
The text was updated successfully, but these errors were encountered:
We do check. The additional values are outdated and no longer relevant.
Sorry, something went wrong.
No branches or pull requests
latestRoundData()
problemSeverity
Medium Risk
Relevant GitHub Links
https://github.com/Cyfrin/2023-07-foundry-defi-stablecoin/blob/main/src/libraries/OracleLib.sol#L26
Summary
This is the
latestRoundData()
problem, the contract calllatestRoundData()
but does not check the return valueVulnerability Details
https://github.com/Cyfrin/2023-07-foundry-defi-stablecoin/blob/main/src/libraries/OracleLib.sol#L26
Impact
This could lead to stale prices according to the Chainlink documentation:
https://docs.chain.link/data-feeds/price-feeds/historical-data
Related report:
code-423n4/2021-05-fairside-findings#70
Tools Used
vs code
Recommendations
we can refer to :
sherlock-audit/2023-02-blueberry-judging#94
The text was updated successfully, but these errors were encountered: