Tadle

Tadle
DeFiFoundry
27,750 USDC
View results
Submission Details
Severity: low
Invalid

`TokenManager::withdraw` transfer can revert due to 2300 gas limit

Summary

Inside TokenManager::withdraw, `transfer` is limited to 2300 gas, so there is a possibility that the receiver can revert

Vulnerability Details

Withdraw function transfers ETH to the msg.sender, if the token is a wrapped native eth token. The protocol use `transfer` function , which is limited to 2300 gas. If msg.sender consumes more gas when receiving the ETH, this will always revert

Impact

Some withdrawal can revert dot to lack of gas

Tools Used

Manual review

Recommendations

Use call function with success check, instead of transfer

Updates

Lead Judging Commences

0xnevi Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Known issue
Assigned finding tags:

[invalid] finding-TokenManager-withdraw-transfer-2300-gas

Invalid, known issues [Medium-2](https://github.com/Cyfrin/2024-08-tadle/issues/1)

Support

FAQs

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