05.12.2013, 06:56
Quote:
You should also save the name as it will change and it could be a valid admin name which you wouldn't want getting banned in your system. There is another problem with that solution as well, there could be many OnPlayerDisconnect()'s that are hooked before this callback then they might get called which isn't good because the player has not disconnected yet they've only spoofed an OnPlayerConnect().
Another potential issue if I'm reading this correctly. if(ce_fix[playerid]) OnPlayerDisconnect(playerid, 3); else ce_fix[playerid] = true; #if defined CE_OnPlayerConnect return CE_OnPlayerConnect(playerid); If i'm not mistaken that will allow the spoofed OnPlayerConnect() to always hook, that is not desirable it's not a real connection. Last point I'm pretty sure calling OnPlayerDisconnect() doesn't delete any per player stuff now this shouldn't matter but it could be an issue. Just clarify things for me if I got another wrong or am overlooking anything thanks |