Rock Paper Scissors

First Flight #38
Beginner FriendlySolidity
100 EXP
View results
Submission Details
Severity: low
Invalid

Unbounded Loops in Game Logic

Description: Throughout the contract where game turns are processed. The contract processes multiple turns in a game, but there's no upper limit on the number of turns that can be specified

Impact: An attacker could create a game with an extremely high number of turns, potentially causing out-of-gas errors

Proof of Concept:

// Attacker creates game with max turns
game.createGameWithEth{value: 0.01 ether}(type(uint256).max, 5 minutes);

Recommended Mitigation: Add a maximum turn limit in the constructor and validate in createGame functions

Updates

Appeal created

m3dython Lead Judge about 2 months ago
Submission Judgement Published
Invalidated
Reason: Non-acceptable severity
Assigned finding tags:

Informational

Code suggestions or observations that do not pose a direct security risk.

Gas Optimization

Support

FAQs

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