Submission Details
Severity:
Missing ETH deposit functionality
Summary
In the InheritanceManagercontract there is no normal way to deposit ETH to the contract even though it's clear that the contract should manage ETH.
Vulnerability Details
Because there is not a deposit, receive, or fallbackfunction there is no normal way to deposit ETH to the InheritanceManagercontract making it harder to receive ETH. Using the selfdestructfunction is not optimal for depositing funds to the contract and is probably not what is intended.
Tools Used
Manual code review
Recommendations
Add a receivefunction to the contract:
contract InheritanceManager is Trustee {
// code as is
+ receive() external payable {}
}
Updates
Lead Judging Commences
0xtimefliez 6 months ago
Submission Judgement Published Assigned finding tags:
Missing receive() or fallback() function
Rewards breakdown
nSLOC
211This contest has a flat reward structure with the following payouts:
100 EXP
20 EXP
2 EXP
Live
The contest is live. Earn rewards by submitting a finding.
Appeals
This is your time to appeal against judgements on your submissions.
Appeals Review
Appeals are being carefully reviewed by our judges.