Using address[] memory newPlayers as the parameter might make it impossible for users to enter the contract.
Consider not using any parameter in the function and using the global variables players in place of newPlayers or consider using parameters which users can easily understand.
Makes the entire contract useless.
Manual Review
The contest is live. Earn rewards by submitting a finding.
This is your time to appeal against judgements on your submissions.
Appeals are being carefully reviewed by our judges.