[RyDeR`]

for name; just use MAX_PLAYER_NAME.
Edit: also if you use a loop trough all players always check for connected players; otherwise it will do more then it's not needed.