isthisokay?

public OnRconLoginAttempt(ip[], password[], success)
{
	new pIP[18];
	foreach(Player,i)
	{
	    GetPlayerIp(i,pIP,sizeof(pIP));
	    if(pIP == ip)
	    {
	        GetName(i);
	        if(IsPlayerConnected(i))
	        {
		        if(success)
				{
					format(szString,sizeof(szString),"%s(%i) Has Login into Server AS RCON Admin");
					ISAY(szString);
				}
				else
				{
					format(szString,sizeof(szString),"Fail Rcon Attempt from %s(%i)[IP: %s]",ip);
					ISAY(szString);
				}
			}
	    }
	}

	return 1;
}