¿Es este código seguro?

[Kwic]

En mi página web trabajo con includes para dinamizarla. El código que utilizo es el siguiente (no es mío):

Código PHP:

  <?
if(isset($_GET[seccion])){
if(file_exists($_GET[seccion].".php")){
include($_GET[seccion].".php");
}
}else{ 
include("news.php"); 
} 
?>   

Pero... ¿es el código seguro?

Obviamente la opción más segura es hacer un switch, pero claro, tendría que ir añadiendo una a una cada posibilidad y es mucho trabajo.

Como no me fio de que el código que uso sea seguro, acabo de hacer este:

Código PHP:

  <?php
 
$seccion = $_GET['seccion'];
$path = $_SERVER['DOCUMENT_ROOT'];
 
$include = $path . "/" . $seccion . ".php";
 
if (!$seccion) {
    include "news.php";
} else {
    if (file_exists ($include)) {
        include "$include";
    } else {
        include "404.php";
    }
}
 
?>   

A mí me parece más seguro este último pero... ¿qué opináis vosotros?

Saludos

[eft0]

IMHO: Lo más seguro es definir los archivos php en un arreglo y luego verificar si $seccion corresponde a alguno de ellos, solo entonces hacer el include.

[Kwic]

Sí, bueno, eso sería lo más seguro, pero son muchos archivos y es mucho trabajo, por eso quiero saber si los códigos que puse son seguros y cuál es más seguro ;)

Saludos

[eft0]

Entre los dos, me quedo con el segundo.