Inconsistent Deadline Logic in `inherit()`
Summary
The inherit function sets isInherited = true for multiple beneficiaries but does not set a deadline, while it sets a deadline for a single beneficiary. This inconsistency could allow withdrawals to happen immediately for multiple beneficiaries.
Vulnerability Details
For one beneficiary, the deadline is set using
_setDeadline().For multiple beneficiaries, no deadline is set, which could allow withdrawals to happen immediately.
Impact
Inconsistent Behavior: Withdrawals for multiple beneficiaries could happen immediately, while withdrawals for a single beneficiary are delayed.
Security Risk: Funds could be withdrawn prematurely for multiple beneficiaries.
Recommendations
Set the deadline for all cases (single or multiple beneficiaries) in the
inheritfunction
Rewards breakdown
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.