The Standard

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

No check for the sequencer being down in regards to querying 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 while querying these prices from Chainlink, protocol does not implement a check to see if the sequencer is down or up, which goes against the official docs on integrating prices from Chainlink on L2

Impact

Incorrect integration of chainlink price data, which would lead to bad data being ingested

Recommended Mitigation Steps

Check the L2 sequencer by using the chainlink sequencer up feed and ensure that the valid price is ingested.

Updates

Lead Judging Commences

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

Arbitrum-sequncer

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

Arbitrum-sequncer

Support

FAQs

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