Como validar solo imagenes en PHP

[el__tamer]

Saludos soy nuevo en el Foro. Hola

Tengo este siguiente código para subir imagenes. Funciona muy bien pero como le restrinjo que sean solo imagenes por que en realidad me sube cualquier archivo, pero yo quiero que sea solo imagenes.

?php
require_once("conexion.php");
$titulo=$_POST['titulo'];
$resumen=$_POST['resumen'];
$desarrollo=$_POST['desarrollo'];


$nombrefoto=$_FILES['imagen']['name'];
$ruta=$_FILES['imagen']['tmp_name'];
$destino = "fotos/".$nombrefoto;
//$fecha=$_POST['fecha'];
copy($ruta,$destino);


$query = @mysql_query('SELECT * FROM noticias WHERE imagen="'.$destino.'"');
if($existe = @mysql_fetch_array($query))
{
echo 'La Imagen Cuyo Nombre Ya Existe. Porfavor Cambie El Nombre A la Imagen.';
return false;
echo $destino;
exit;
}



if($existe = @mysql_fetch_array($query))
{
echo 'Este esta ocupado';
return false;
exit;
}



else{


mysql_query("insert into noticias(titulo,resumen,desarrollo,imagen,id_notd, fecha) values('$titulo','$resumen','$desarrollo','$destin o','1',CURDATE())");
header("Location:man_web.php");}
?>


Gracias

[enlinea777]

con eso

Código PHP:

$nombrefoto=strtolower($nombrefoto);
if(substr($nombrefoto,-4)!=".jpeg"){
echo "No es imagen";
die();
}
 

este es un ejemplo muy sencillo

[Alexis88]

También puedes tomar el tipo de archivo desde el array FILES, estableces los tipos de archivo permitidos y los comparas con el del archivo:

Código PHP:

$tipo = $_FILES["imagen"]["type"];
$admitidos = ["jpg", "jpeg", "gif", "bmp", "png"];
 
if (array_search(strtolower($tipo), $admitidos) !== FALSE) {
  //Procesas y subes el archivo
}
else
  echo "El archivo no es de un tipo válido";
 

También puedes restringir los tipos de archivo desde el input a través del atributo accept:

Código HTML:

<input type = "file" accept = "image/*" />
 

El único inconveniente está en que el usuario puede cambiar el filtrado desde el cuadro en el que elige el archivo que subirá.

Combinando ambas formas, será muy complicado que algún otro tipo de archivo pueda colarse.

Saludos

[el__tamer]

Iniciado por Alexis88

También puedes tomar el tipo de archivo desde el array [URL="http://php.net/manual/es/reserved.variables.files.php"][COLOR="Blue"][B]FILES[/B][/COLOR][/URL], estableces los tipos de archivo permitidos y los comparas con el del archivo:

Código PHP:

$tipo = $_FILES["imagen"]["type"];
$admitidos = ["jpg", "jpeg", "gif", "bmp", "png"];
 
if (array_search(strtolower($tipo), $admitidos) !== FALSE) {
  //Procesas y subes el archivo
}
else
  echo "El archivo no es de un tipo válido";
 

También puedes restringir los tipos de archivo desde el input a través del atributo [URL="http://www.w3schools.com/tags/att_input_accept.asp"][B][COLOR="Blue"]accept[/COLOR][/B][/URL]:

Código HTML:

<input type = "file" accept = "image/*" />
 

El único inconveniente está en que el usuario puede cambiar el filtrado desde el cuadro en el que elige el archivo que subirá.

Combinando ambas formas, será muy complicado que algún otro tipo de archivo pueda colarse.

Saludos

ME SALE ERROR EN LA LINEA
$admitidos = ["jpg", "jpeg", "gif", "bmp", "png"];

[el__tamer]

Iniciado por enlinea777

con eso

Código PHP:

$nombrefoto=strtolower($nombrefoto);
if(substr($nombrefoto,-4)!=".jpeg"){
echo "No es imagen";
die();
}
 

este es un ejemplo muy sencillo

me funciono
ahora tengo que ver como lo ago para que me acepte otras extensiones de imagenes.
haci quedo el codigo




<?php
require_once("conexion.php");


$titulo=$_POST['titulo'];
$resumen=$_POST['resumen'];
$desarrollo=$_POST['desarrollo'];

$nombrefoto=$_FILES['imagen']['name'];
$nombrefoto=strtolower($nombrefoto);

if(substr($nombrefoto,-4)!=".jpg"){
echo "La Imagen o Foto Que Ha Acabado de Subir No Pertenece A imagen O Foto";
exit();
}


else{


$ruta=$_FILES['imagen']['tmp_name'];
$destino = "fotos/".$nombrefoto;
//$fecha=$_POST['fecha'];
copy($ruta,$destino);

$destino2 = "../../cosmosportal/cosmoportalsupervisor/noticias/fotos/".$nombrefoto;

copy($ruta,$destino2);

$destino3 = "../../cosmosportal/CosmosPortalUsuario/noticias/fotos/".$nombrefoto;

copy($ruta,$destino3);




$query = @mysql_query('SELECT * FROM noticias WHERE imagen="'.$destino.'"');
if($existe = @mysql_fetch_array($query))
{
echo 'La Imagen Cuyo Nombre Ya Existe. Porfavor Cambie El Nombre A la Imagen.';
return false;
echo $destino;
exit;
}



if($existe = @mysql_fetch_array($query))
{
echo 'Este esta ocupado';
return false;
exit;
}



else{


mysql_query("insert into noticias(titulo,resumen,desarrollo,imagen,id_notd, fecha) values('$titulo','$resumen','$desarrollo','$destin o','1',CURDATE())");
header("Location:man_web.php");}}
?>


si me ayudaras xD slds

[Alexis88]

Iniciado por el__tamer

ME SALE ERROR EN LA LINEA
$admitidos = ["jpg", "jpeg", "gif", "bmp", "png"];

Probablemente sea porque tu versión de PHP es inferior a la 5.4 en donde los arrays pueden ser declarados de esa forma. Sustituye esa línea por esta:

Código PHP:

$admitidos = array("jpg", "jpeg", "gif", "bmp", "png");
 

Saludos

[nksn]

A ver gente, usen los "mimes"... que sucede si alguien sube un archivo.exe renombrado a archivo.jpg?

windows abre archivos por su mime y no extensión, dando puerta a posibles ataques/infecciones

Código PHP:

  // array de mimes
$acceptado = array('image/gif', 'image/jpeg', 'image/jpg');

// mime del archivo
$mime = mime_content_type($filepath);

// comprobación de tipos
if(!in_array($acceptado, $mime))
{
      echo 'no admitido';
} 
   

espero que les sirva

nota final: mime_content_type() esta deprecado, se debería usar finfo

[Alexis88]

Iniciado por nksn

nota final: mime_content_type() esta deprecado, se debería usar finfo

¿Y por qué haces el ejemplo con la función en desuso?

Aun así, es correcto comprobar el tipo por el valor MIME. el__tamer, dale una leída a esto.

Saludos

[moriorGames]

Tenía por ahí un código para subir archivos pero me he dado cuenta de que lo he perdido :(

De todas formas, he googleado un poco y he llegado a w3schools una página que uso bastante como referencia y he encontrado este código que me parece bastante correcto. Además, que tiene restricciones de peso, algo bastante interesante también :)

Código PHP:

  <?php
$allowedExts = array("gif", "jpeg", "jpg", "png");
$temp = explode(".", $_FILES["file"]["name"]);
$extension = end($temp);
if ((($_FILES["file"]["type"] == "image/gif")
|| ($_FILES["file"]["type"] == "image/jpeg")
|| ($_FILES["file"]["type"] == "image/jpg")
|| ($_FILES["file"]["type"] == "image/pjpeg")
|| ($_FILES["file"]["type"] == "image/x-png")
|| ($_FILES["file"]["type"] == "image/png"))
&& ($_FILES["file"]["size"] < 20000)
&& in_array($extension, $allowedExts))
  {
  if ($_FILES["file"]["error"] > 0)
    {
    echo "Error: " . $_FILES["file"]["error"] . "<br>";
    }
  else
    {
    echo "Upload: " . $_FILES["file"]["name"] . "<br>";
    echo "Type: " . $_FILES["file"]["type"] . "<br>";
    echo "Size: " . ($_FILES["file"]["size"] / 1024) . " kB<br>";
    echo "Stored in: " . $_FILES["file"]["tmp_name"];
    }
  }
else
  {
  echo "Invalid file";
  }