Uploader seguro

[BaByRoDrI]

Quiero desarrollar un sencillo y practico uploader de imagenes, quiero hacerlo lo más seguro posible, he aquí lo que he logrado:

Código PHP:

<?php
$dir = "/imagenes"; //recuerda que debe tener permisos de escritura ;)
$ext = array('image/jpeg', 'image/gif', 'image/png', 'image/bmp'); //Puedes agregar más extenciones
    foreach($_FILES as $archivo) {
        $temp  = $archivo['tmp_name'];
        $type = $archivo['type'];
        $name = $archivo['name'];
        $ruta = "$dir/$name";
        if(file_exists($temp)) {
            if(is_uploaded_file($temp)) {
                if(in_array($type,$ext)) {
                    if($type == 'image/jpeg') {
                        $image = imagecreatefromjpeg($temp);
                        imagejpeg($image, $ruta, 100);
                        echo "<img src='$ruta' />";
                    }
                    else if($type == 'image/png') {
                        $image = imagecreatefrompng($temp);
                        imagepng($image, $ruta, 100);
                        echo "<img src='$ruta' />";
                    }
                    else if($type == 'image/gif') {
                        $image = imagecreatefromgif($temp);
                        imagegif($image, $ruta, 100);
                        echo "<img src='$ruta' />";
                    }
                    else {
                        move_uploaded_file($temp, $ruta);
                        echo "<img src='$ruta' />";
                    }
                } else {
                    echo "Esto no es una imagen :(";
                }
            }
        }
    }
?>
 

Lamentablemente las imagenes GIF, JPG y PNG no las sube al servidor :(, como la librería GD soporta este tipo de imagenes las quiero re-construir para evitar que me vayan a colar por allí algún codigo raro :3.

No se realmente que esta mal, alguien me puede hechar la mano ?

PD. Según yo alguien pueda spoofear la cabecera y aun así podría saltarse la "seguridad" del script no?

[Triby]

A ver, supongamos que yo creo un archivo con "codigo malicioso en PHP", lo guardo como imagen.jpg, lo subo a un servidor y trato de abrirlo con el navegador... que pasara?

a) El servidor "piensa" que se trata de una imagen (por la extension) y lo envia directo al navegador, quien no puede interpretar compresion, tipo, tamaño, etc. y muestra un mensaje de error.
b) El servidor tiene una configuracion extraña que hace que los archivos con extensiones .jpg sean interpretados por PHP y ejecuta el codigo malicioso... yo pregunto, quien en su sano juicio configuraria un servidor para comportarse asi?

Bueno, todo esto es solo un resumen para que la cuestion de seguridad no te vuelva paranoico y, hay mejores formas de verificar que el archivo es realmente una imagen.

Regresa al manual de GD y lee acerca de la funcion getimagesize(), aunque los pasos a seguir son:

Código PHP:

// Extensiones de imagenes que deseas admitir
$extensiones = array('jpg', 'jpeg', 'gif', 'png');
 
// Extension real del archivo subido
$ext = strtolowr(end(explode('.', $name))); // En minusculas para facilitar las cosas
if( ! in_array($ext, $extensiones)) {
    die("Esto no es una imagen >:-(");
}
 
// La @ es para que no se muestre mensaje de error si:
// - El archivo NO existe
// - El archivo NO es una imagen valida
$imgData = @getimagesize($temp);
 
// Comparacion con triple signo igual (tipo de dato y valor)
if($imgData === false) {
    die("Esto no es una imagen >:-(");
}
 
// Aqui usas move_uploaded_file() o lo que prefieras