Sikkert include script på webside

[Scix]

Hei!

 

Jeg holder på å lager en webside. Siden jeg ikke er så enorm på sikkerhet, lurte jeg på om dette scriptet var sikkert? Fri for mulighet til å krasje siden, laste inn skadlig kode osv.

<?
$fy_pages = array("index", "main", "menu");
if (isset($_GET['page']) and ($_GET['page']!=""))	{
if (in_array(strtolower($_GET['page']), $fy_pages))	{
 echo "Kunne ikke laste filen";
 exit;
}
if (file_exists($_GET['page'] . ".php"))	{
 include $_GET['page'] . ".php";
}	else	{
 include 'hjem.php';
}
}	else	{
include 'hjem.php';
}
?>

Jeg har selv skrevet det, så det er sikkert fult med ting som kunne vært bedre. Hvis du ser noe, ikke vært redd for å si ifra

 

Edit: Fikset sikkerhetsfeil

Endret 23. september 2005 av Scix