DeFiFoundry
60,000 USDC
View results
Submission Details
Severity: low
Valid

Funds Mistakenly Sent to createTradingAccountAndMulticall Function Will Be Stuck

Summary

The createTradingAccountAndMulticall function allows users to create a trading account and make multiple calls in a single transaction. This function is marked as payable, which means it can receive Ether. However, the function does not include any logic to handle or refund the received Ether, causing the funds to become irretrievable.

Vulnerability Details

The createTradingAccountAndMulticall function is marked as payable, but it lacks logic to handle the Ether sent to it. This results in any Ether sent by mistake being stuck in the contract, with no way for users to retrieve it.

Impact

Users funds will be stuck.

Tools Used

Manual Review

Recommendations

Remove payable keyword from createTradingAccountAndMulticallfunction

Updates

Lead Judging Commences

inallhonesty Lead Judge 10 months ago
Submission Judgement Published
Validated
Assigned finding tags:

`createTradingAccountAndMulticall` shouldn't be payable

Support

FAQs

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