Submission Details
Severity:
Low-1 Game Can be ended prematurely
Summary
Every game set for certain duration more then 60 second and protocol intended this way however owner can stop the game anytime.
Vulnerability Details
function endGame() external onlyOwner {
require(gameActive, "Game not active");
gameActive = false;
emit GameEnded(block.timestamp);
}
Theres no check in endGame function for duration check and its against the protocol behaviour.
Impact
Unwanted small amount of gaming time expected .
Tools Used
Manuel review.
Recommendations
Set check for minumum duration check in function to protocol act as expected.
Updates
Lead Judging Commences
m3dython 3 months ago
Submission Judgement Published Reason: Design choice
Assigned finding tags:
Trusted Owner
Owner is trusted and is not expected to interact in ways that would compromise security
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.