First Flight #18: T-Swap

First Flight #18
Beginner FriendlyDeFiFoundry
100 EXP
View results
Submission Details
Severity: medium
Valid

TSwapToken.deposit does not check deadline

Summary

The deposit function has a deadline parameter but it is not enforced.

Vulnerability Details

https://github.com/agent3bood/2024-06-t-swap/blob/main/src/TSwapPool.sol#L117

Impact

Unexpected behavior by the protocol, when the user specify a deadline it should be enforced.

Tools Used

Recommendations

Add revertIfDeadlinePassed(deadline) modifier to the deposit function.

Updates

Appeal created

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Validated
Assigned finding tags:

`deposit` is missing deadline check causing transactions to complete even after the deadline

Support

FAQs

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