Submission Details
Impact:
Likelihood:
[L-2] `nonReentrant` should be the first modifier at `Game::withdrawPlatformFees`.
Root + Impact
[L-2] nonReentrant should be the first modifier at Game::withdrawPlatformFees.
Description
To protect against reentrancy in other modifiers, the nonReentrant modifier should be the first modifier in the list of modifiers.
Risk
Likelihood: Low
Impact: Low
Proof of Concept
The Game::withdrawPlatformFees function is defined as follows:
function withdrawPlatformFees() external onlyOwner nonReentrant {
Recommended Mitigation
Place the nonReentrant modifier as the first modifier in the list of modifiers of the Game::withdrawPlatformFees function.
- function withdrawPlatformFees() external onlyOwner nonReentrant {
+ function withdrawPlatformFees() external nonReentrant onlyOwner {
Rewards breakdown
nSLOC
181This 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.