this is definitely not safe as it could lead to directory browsing and other unwanted usage
your best bet is to trap the $_GET, check it against a structure; switch or if/elseif, else which will handle the include
$page = $_GET['page'];
if (empty($page))
{
//include('page_home.htm')
}
elseif ($page == 'this')
{
//include('page_this.htm')
}
elseif ($page == 'that')
{
//include('page_that.htm')
}
else
{
//include('page_error.htm')
}
this works well if you KNOW which pages u have... you could store the valid page names in an array actually and just check for in_array instead of writing the elseif everytime