Submission Details
Severity:
_isActivePlayer() - Internal function not being called internally at all.
Summary
_isActivePlayer() - Internal function not being called internally at all.
Vulnerability Details
/// @notice this function will return true if the msg.sender is an active player
function _isActivePlayer() internal view returns (bool) {
Impact
Absence of intended functionality.
OR
Unnecessary additional gas costs during contract deployment.
Tools Used
VSC.
Recommendations
If it's meant to be external function, it should be changed to external modifier, but I suspect the intention was for it to be called internally only.
It's not being used in any other functions within the contract. Should either be removed or its intended implementation/use case fully implemented or it should be commented out.
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.