Part 2

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

`maxExecutionTime` Same For All Swaps

Summary

In MarketMakingEngineConfigurationBranch::configureUsdTokenSwapConfig maxExecutionTime is set. However, it is the same for all swaps and set only by the owner. Can put users in unfavourable scenarios when swapping if it set higher than they would like it to be.

Impact

Unfavourable scenarios for users and trust loss.

Tools Used

Manual Review

Recommendations

Let users choose their own maxExecutionTime on each of their swaps.

Updates

Lead Judging Commences

inallhonesty Lead Judge 6 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity

Support

FAQs

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