Rock Paper Scissors

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

emit MoveRevealed(_gameId, msg.sender, move, game.currentTurn) will show the move

Summary

emit MoveRevealed(_gameId, msg.sender, move, game.currentTurn) will show the move ,causing the player's move been seen by the other player making losing.

Vulnerability Details

the event will show the move. the player who call revealMove() first will be seen the move,causing lose ether or Token.

emit MoveRevealed(_gameId, msg.sender, move, game.currentTurn);

Impact

the event will show the move. the player who call revealMove() first will be seen the move,causing lose ether or Token.

Tools Used

manual review

Recommendations

delete the move in the event.

- emit MoveRevealed(_gameId, msg.sender, move, game.currentTurn);
+ emit MoveRevealed(_gameId, msg.sender, game.currentTurn);
Updates

Appeal created

m3dython Lead Judge 2 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement
m3dython Lead Judge 2 months ago
Submission Judgement Published
Invalidated
Reason: Incorrect statement

Support

FAQs

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