DeFiFoundry
60,000 USDC
View results
Submission Details
Severity: low
Invalid

Storing other chains sequencerUptimeFeed is unnecessary

Description

Storing other chains sequencer uptime feed is unnecessary, as the only sequencer that matters is the one on which the protocol is deployed!

https://github.com/Cyfrin/2024-07-zaros/blob/main/src/perpetuals/leaves/GlobalConfiguration.sol#L54

mapping(uint256 chainId => address sequencerUptimeFeed) sequencerUptimeFeedByChainId;

This mapping can be reduced to just one address, storing the sequencerUptimeFeed of the chain on which the protocol is deployed.

Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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