else if ($_GET["port"] && $_GET["port2"]) {
$p1 = $_GET["port"];
$p2 = $_GET["port2"];
if ($p1 == $p2) {
$fp = @fsockopen($addr, $port, $errno, $errstr, 2);
sorry, but where is $port set?
should check to see if $_GET['port'] and $_GET['port2'] is numeric
if (is_numeric($_GET['port']) && is_numeric($_GET['port2'])) {
besides that, not bad....