Beginner FriendlySolidity
100 EXP
View results
Submission Details
Severity: medium
Valid

removeBeneficiary does not resets the 90 days timer.

Summary

removeBeneficiary in InheritanceManage.sol breaks the protocol's invariant .

Vulnerability Details

removeBeneficiary in InheritanceManage.sol does not reset the 90 days timer after removing beneficiary from "beneficiaries" mapping.

Impact

Breaking of Invariants.

Tools Used

Manual review.

Recommendations

removeBeneficiary must call " _setDeadline()" after removing beneficiary.

function removeBeneficiary(address _beneficiary) external onlyOwner {
uint256 indexToRemove = _getBeneficiaryIndex(_beneficiary);
delete beneficiaries[indexToRemove];
+ _setDeadline();
}
Updates

Lead Judging Commences

0xtimefliez Lead Judge 5 months ago
Submission Judgement Published
Validated
Assigned finding tags:

functions do not reset the deadline

Support

FAQs

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