TSender

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

Solidity TSender protocol reference differs from the Yul and Huff implementation

Summary

Solidity TSender reference contract differs from the Yul and Huff version.

Vulnerability Details

In the line 37 of TSender reference contract, the return value of ERC20 token is checked, while the TSender protocol documentation clearly states it doesn't. Yul and Huff contracts don't have this issue. I'm aware this issue is out of scope of the audit, but this may have unwanted consequences in the future.

Impact

Broken tests when comparing Solidity version to Yul and Huff ones.

Tools Used

Manual research

Recommendations

Don't verify the return value in lines 38-40.

Updates

Lead Judging Commences

inallhonesty Lead Judge about 1 year ago
Submission Judgement Published
Invalidated
Reason: Known issue

Support

FAQs

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