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

The deadline is not correctly initialized in th `InheritanceManager` contract

Summary

The deadline variable is not correctly initialized in InheritanceManager contract which could lead to beneficiaries claiming funds before deadline has passed.

Vulnerability Details

The deadlinevariable is set to 0 when contract is deployed which could lead to beneficiaries bypassing time lock and claiming funds before they are authorised to do so.

Impact

Beneficiaries could claim funds before deadline has passed.

Tools Used

Manual code review

Recommendations

Initialize the deadline in the in the constructor:

constructor() {
owner = msg.sender;
nft = new NFTFactory(address(this));
+ _setDeadline()
}
Updates

Lead Judging Commences

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

constructor does not initialize deadline

Appeal created

0xtimefliez Lead Judge 3 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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