Rock Paper Scissors

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

Function revealMove() parameter `_move` can be seen on chain

Summary

Function revealMove() parameter _move can be seen on chain ,causing the first player to call revealMove will be lose.

Vulnerability Details

revealMove() use _move as the parameter ,which can be seen on transaction. So the first player to call this function will lose.

function revealMove(uint256 _gameId, uint8 _move, bytes32 _salt) external {}

Impact

the first player to call revealMove will lose Token or Ether.

Tools Used

manual review.

Recommendations

Use other way to record _move .

Updates

Appeal created

m3dython Lead Judge 2 months ago
Submission Judgement Published
Invalidated
Reason: Too generic
m3dython Lead Judge 2 months ago
Submission Judgement Published
Invalidated
Reason: Too generic

Support

FAQs

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