MorpheusAI

MorpheusAI
Foundry
22,500 USDC
View results
Submission Details
Severity: medium
Invalid

Missing ETH Refund Logic and Inability to Receive ETH from in L1Sender

Summary

Vulnerability Details

According to the information provided in the Readme:

The alias of L1Sender.sol on Arbitrum will hold the ETH received as a gas refund from bridging tokens Context: L1Sender.sol, L2TokenReceiver.sol

The L1Sender currently lacks explicit refund logic for handling excess ETH in transactions, as well as the necessary receive() and fallback() functions for accepting direct Ether transfers. This limitation poses a significant risk in scenarios where excess ETH is expected to be refunded or other external contracts. Without these mechanisms, any excess ETH sent during a transaction might not be recoverable or could result in the loss of funds if the external contract attempts to refund ETH directly to the L1Sender contract.

Impact

Tools Used

Recommendations

Implement receive() and fallback() Functions: Add these functions to the L1Sender to ensure it can accept direct Ether transfers. This is crucial for receiving refunds from external contracts.
Add Refund Logic for Excess ETH: Include logic for handling and refunding excess ETH.

Updates

Lead Judging Commences

inallhonesty Lead Judge
over 1 year ago
inallhonesty Lead Judge over 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.