The Standard

The Standard
DeFiHardhat
20,000 USDC
View results
Submission Details
Severity: medium
Invalid

`latestRoundData` might return stale prices

Proof of Concept

Using this search command: https://github.com/search?q=repo%3ACyfrin%2F2023-12-the-standard%20latestRoundData&type=code, we can see that in multiple cases and for multiple instances the price of an asset is queried by the latestRoundData() , case with this is that there are currently no checks employed to ensure that the price returned is not stale

Impact

Wrong (stale) prices would be ingested in to protocol

Recommended Mitigation Steps

Follow the official documentation from Chainlink and apply the necessary checks to ensure that stale prices are not used.

Updates

Lead Judging Commences

hrishibhat Lead Judge over 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

Chainlink-price

hrishibhat Lead Judge over 1 year ago
Submission Judgement Published
Invalidated
Reason: Known issue
Assigned finding tags:

Chainlink-price

Support

FAQs

Can't find an answer? Chat with us on Discord, Twitter or Linkedin.