DatingDapp

First Flight #33
Beginner FriendlyFoundrySolidityNFT
100 EXP
View results
Submission Details
Severity: low
Invalid

Lack of Event Emission for ETH Deposits via Receive Function

Summary

The MultiSigWallet contract() allows ETH deposits via the receive() function, but it does not emit an event when funds are received. This makes it difficult to track incoming transactions, leading to reduced transparency and monitoring issues.

Vulnerability Details

Affected Code:

receive() external payable {}

Secure Code:

event Deposit(address indexed sender, uint256 amount);

receive() external payable {
emit Deposit(msg.sender, msg.value);
}

Impact

  • Users cannot track deposits on-chain without actively monitoring contract balances.

  • Auditors and owners lack visibility into incoming funds, making debugging harder.

Tools Used

Manually

Recommendations

Modify the receive() function to emit an event when ETH is deposited.

Updates

Appeal created

n0kto Lead Judge 4 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Informational or Gas

Please read the CodeHawks documentation to know which submissions are valid. If you disagree, provide a coded PoC and explain the real likelyhood and the detailed impact on the mainnet without any supposition (if, it could, etc) to prove your point.

Support

FAQs

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