TempleGold

TempleDAO
Foundry
25,000 USDC
View results
Submission Details
Severity: low
Invalid

`TempleTeleporter::teleport` should allow user specify refund address

Summary

TempleTeleporter::teleport will refund to payable(msg.sender), this may not work as intended for some contract callers. If there is no sweep function, this would cause lock/loss of funds.

Vulnerability Details

TempleTeleporter::teleport will refund to payable(msg.sender).

receipt = _lzSend(dstEid, _payload, options, MessagingFee(msg.value, 0), payable(msg.sender));

However, this may cause issues if msg.sender is a contract which lacks the ability to sweep ETH. This would cause lock/loss of funds.

Impact

If the msg.sender is a contract, this may cause lock/loss of refunded funds.

Tools Used

Manual

Recommendations

Allow user specify the refund address.

Updates

Lead Judging Commences

inallhonesty Lead Judge 11 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.