Submission Details
Severity:
```getActivePlayerIndex(address player)``` will mislead people into believeing they are ```players[0]```
Summary
getActivePlayerIndex(address player) will mislead people into believeing they are players[0]
Vulnerability Details
When a user (not a player) called getActivePlayerIndex(address player) function, he will get 0. This will mislead him into believing he is players[0]
Impact
mislead users into believing they are already players
Tools Used
manual
Recommendations
replace return 0 with revert NotActivePlayer();
Updates
Lead Judging Commences
Hamiltonite about 2 years ago
Submission Judgement Published Assigned finding tags:
getActivePlayerIndex can say a player is both entered at slot 0 and inactive
Rewards breakdown
nSLOC
143This 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.