Quick Php Question

[wdsguy]

I'm hoping someone can help me with a quick php question

I have the following on my signup

if($ip=getIp()){
if(!in_array($ip,$CFG['skipipcheck'])&&sign_up_check_ip($ip))
error_quit( "You already have an account.");
}


basically it grabs the users signup ip and then checks if it already exists in the system - if it does it will not let them signup.

How would I modify it to check against an array of ips? I want it to check if the users signup ip matches a couple of blacklisted ips and if it does i don't want them to be able to signup.


Thanks

[jwerd]

Originally Posted by wdsguy

I'm hoping someone can help me with a quick php question

I have the following on my signup

if($ip=getIp()){
if(!in_array($ip,$CFG['skipipcheck'])&&sign_up_check_ip($ip))
error_quit( "You already have an account.");
}


basically it grabs the users signup ip and then checks if it already exists in the system - if it does it will not let them signup.

How would I modify it to check against an array of ips? I want it to check if the users signup ip matches a couple of blacklisted ips and if it does i don't want them to be able to signup.


Thanks

I'm assuming if they are black listed, you don't want to mention to them that they "already have an account" so the easiest way would be an else if:

// above code yada,yada
} else if(in_array($ip, $blacklisted)) {
error_quit("Sorry bud, nice try.");
}

and blacklisted would be an array like $blacklisted = array('192.168.1.1', '192.168.1.2'); etc..

Good luck!

[wdsguy]

Originally Posted by lamerhooD

I'm assuming if they are black listed, you don't want to mention to them that they "already have an account" so the easiest way would be an else if:

// above code yada,yada
} else if(in_array($ip, $blacklisted)) {
error_quit("Sorry bud, nice try.");
}

and blacklisted would be an array like $blacklisted = array('192.168.1.1', '192.168.1.2'); etc..

Good luck!

thanks

[jwerd]

Originally Posted by wdsguy

thanks

np glad I could help!

[HighEnergy]

If you check it against dupe IPs you'll may possibly disallow anyone with a dynamic IP. I would check it against your blacklist only.