Submission Details
Severity:
_isActivePlayer() can't be called
Summary
_isActivePlayer() is never called.
Vulnerability Details
_isActivePlayer() is intended to determine whether a player is participating in the raffle. However, it is marked as an internal function but is never actually called. This presents an issue, the users who wish to check whether they are part of the raffle are unable to do so because the function is inaccessible due to its internal visibility.
function _isActivePlayer() internal view returns (bool) {
for (uint256 i = 0; i < players.length; i++) {
if (players[i] == msg.sender) {
return true;
}
}
return false;
}
Impact
Function can't be called.
Tools Used
Manual review.
Recommendations
Set the function visibility to public.
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.