Submission Details
Severity:
Host could set deadline multiple times
Summary
The setDeadline function fails to update the deadlineSet boolean after setting the deadline, allowing multiple deadline updates when it should only be settable once.
Vulnerability Details
deadlineSetremainsfalseafter setting deadlineCheck
if(deadlineSet)becomes ineffectiveAllows repeated deadline modifications
Impact
Host can set deadline multiple times despite intended one-time setting
Previous deadline values can be overwritten
Breaks the business logic of immutable deadlines
Tools Used
Manual Review
Recommendations
function setDeadline(uint256 _days) external onlyHost {
if(deadlineSet) {
revert DeadlineAlreadySet();
}
deadline = block.timestamp + _days * 1 days;
deadlineSet = true; //this needs to be done
emit DeadlineSet(deadline);
}
Updates
Lead Judging Commences
0xtimefliez 10 months ago
Submission Judgement Published Assigned finding tags:
deadline is never set to true
0xtimefliez 10 months ago
Submission Judgement Published Assigned finding tags:
deadline is never set to true
Rewards breakdown
nSLOC
129This 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.