Part 2

Zaros
PerpetualsDEXFoundrySolidity
70,000 USDC
View results
Submission Details
Severity: medium
Invalid

Gas Limit Issues in checkUpkeep()

Summary

https://github.com/Cyfrin/2025-01-zaros-part-2/blob/main/src/external/chainlink/keepers/fee-conversion-keeper/FeeConversionKeeper

Vulnerability Details

Loops over all markets and all assets per market to check fees.

Impact

If too many assets exist, this could exceed the block gas limit, causing failures.

Tools Used

Manual Review

Recommendations

Implement pagination (e.g., process a fixed number of assets per call).

Use batch processing instead of handling all assets in a single transaction.

Updates

Lead Judging Commences

inallhonesty Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Lack of quality

Support

FAQs

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