Submission Details
Severity:
Timer Is Not Reset In InheritanceManager::removeBeneficiary Which Breaks Core Functionallity Of The Contract
Summary: The 90 days deadline reset isn't present in InheritanceManager::removeBeneficiary which breaks the core assumption of the contract
Vulnerability Details: The 90 days deadline is not set in InheritanceManager::removeBeneficiary which breaks the core assumption of the contract that all functions called by the owner will reset the 90 days deadline
//@audit --> Function called by owner doesn't reset the 90 days timer
function removeBeneficiary(address _beneficiary) external onlyOwner {
uint256 indexToRemove = _getBeneficiaryIndex(_beneficiary);
delete beneficiaries[indexToRemove];
}
Impact: This breaks the core functionality of the contract that every transaction carried out by the owner would reset the 90 days timer
Tools Used: Foundry
Recommendations: Implement the _setDeadline on InheritanceManager::removeBeneficiary
//@audit --> Added Deadline reset
_setDeadline();
Updates
Lead Judging Commences
0xtimefliez 9 months ago
Submission Judgement Published Assigned finding tags:
functions do not reset the deadline
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.