Submission Details
Severity:
The Game Status Logic in `EggHuntGame::getGameStatus` could be simplified
Description: The EggHuntGame::getGameStatus function has redundant checks for block.timestamp and could be simplified. The Game not started yet status will never occur because gameActive and startTime are always set together.
Impact: The function is more complex than necessary
Recommended Mitigation: Simplify the logic to rely on gameActive and endTime.
function getGameStatus() external view returns (string memory) {
if (!gameActive) {
return "Game is not active";
} else if (block.timestamp > endTime) {
return "Game time elapsed";
} else {
return "Game is active";
}
}
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.