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.
Affected Code:
receive() external payable {}
Secure Code:
event Deposit(address indexed sender, uint256 amount);
receive() external payable {
emit Deposit(msg.sender, msg.value);
}
Users cannot track deposits on-chain without actively monitoring contract balances.
Auditors and owners lack visibility into incoming funds, making debugging harder.
Manually
Modify the receive()
function to emit an event when ETH is deposited.
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.
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.