Descarga de archivos me dice que están corruptos

[marx-pola]

Hola a todos! Tengo un problema con este código para descargar archivos de mi web. Cuando descargo por ejemplo archivos pdf me dice que está corrupto. ¿Por qué puede ser?

Acá el código:
descargas.php

Código PHP:

      $the_array = Array();
    $handle = opendir('descargas/.');
 
echo "<table border='0' width='500'>";
  $numcolumnas = 2;
  if ($the_array > 1) {
      $i = 1;
 
    while (false !== ($file = readdir($handle))) {
       if ($file != "." && $file != "..") {
       $the_array[] = $file;
       }
    }
    closedir($handle);
    sort ($the_array);
    foreach($the_array as $val){
 
      $resto = ($i % $numcolumnas);
          if($resto == 1){ /*si es el primer elemento creamos una nueva fila*/
      echo "<tr>";
      }
     echo "<td height='27' class='arial-11pxNG'>";
       echo "<a href=\"file.php?file=$val\">$val</a><br>";
 
     echo "</td>";/*mostramos el valor del campo especificado*/
      if($resto == 0){/*cerramos la fila*/
      echo "</tr>";
      }
      $i++;
      }
     if($resto != 0){/*Si en la &uacute;ltima fila sobran columnas, creamos celdas vac&iacute;as*/
          for ($j = 0; $j < ($numcolumnas - $resto); $j++){
      echo "<td></td>";
      }
      echo "</tr>";
      }
  }
echo "</table>"; 
   

file.php

Código PHP:

  $files = $_GET['file'];
 
$file = "descargas/".$files;
 
$extensiones = array("jpg", "jpeg", "png", "pdf", "xls");
    $file = $_GET["file"];
    if(strpos($file,"/")!==false){
        die("No puedes navegar por otros directorios");
    }
    $ftmp = explode(".",$file);
    $fExt = strtolower($ftmp[count($ftmp)-1]);
 
    if(!in_array($fExt,$extensiones)){
        die("<b>ERROR!</b> no es posible descargar archivos con la extensión $fExt");
    }
 
header("Content-type: application/octet-stream");
header("Content-Disposition: attachment; filename=\"" . $file . "\"\n");
header("Content-Length: " . strlen ($file) );
readfile($file);
$fp=fopen("$file", "r");
fpassthru($fp); 
   

¿Alguna sugerencia del xq no bajan los archivos?

Muchas gracias y saludos.
Marx.

[maycolalvarez]

puedes prescindir del tag de cierre de PHP (?>) para evitar que caracteres invisibles se cuelen a la salida, además verifica que no exista nada antes de <?php

[marx-pola]

Iniciado por maycolalvarez

puedes prescindir del tag de cierre de PHP (?>) para evitar que caracteres invisibles se cuelen a la salida, además verifica que no exista nada antes de <?php

Ah, no... pero el archivo descargas.php tiene html antes y después de lo que yo les mostré. ¿Y entonces, está mal así?

[marx-pola]

¿Alguien me puede decir donde está el problema?

[marx-pola]

No hay caso... al archivo file.php le agregé este código y me devuelve el archivo con 0 (cero) bytes y además, los archivos jpg tampoco me los abre.

Código PHP:

  $extensiones = array("jpg", "jpeg", "png", "gif", "pdf");
    $f = $_GET["file"];
    if(strpos($f,"descargas/")!==false){
        die("No puedes navegar por otros directorios");
    }
    $ftmp = explode(".",$f);
    $fExt = strtolower($ftmp[count($ftmp)-1]);
 
    if(!in_array($fExt,$extensiones)){
        die("<b>ERROR!</b> no es posible descargar archivos con la extensión $fExt");
    }
 
    header("Content-type: application/octet-stream");
    header("Content-Disposition: attachment; filename=\"$f\"\n");
    $fp=fopen("$f", "r");
    fpassthru($fp); 
   

¿Puede ser que el error esté al principio con el código que escanea los archivos del servidor?
Seguro que es una pavada pero ¿no me pueden hechar una mano?
Gracias.

[marx-pola]

Bueno... avancé un poco más... ahora los archivos jpg me los baja perfecto y me muestra el tamaño. los archivos pdf también ya me muestra el tamaño pero cuando los bajo, NO ME LOS BAJA CON LA EXTENSION PDF, me los baja SIN EXTENSION.

Acá el cod. que modifiqué (era más un tema de rutas)

Código PHP:

  $extensiones = array("jpg", "jpeg", "png", "gif", "pdf");
    $f = $_GET["file"];
    $enlace = "descargas/".$f;
    if(strpos($f,"descargas/")!==false){
        die("No puedes navegar por otros directorios");
    }
    $ftmp = explode(".",$enlace);
    $fExt = strtolower($ftmp[count($ftmp)-1]);
 
    if(!in_array($fExt,$extensiones)){
        die("<b>ERROR!</b> no es posible descargar archivos con la extensión $fExt");
    }
 
    header ("Content-Disposition: attachment; filename=".$f);
    header ("Content-Type: application/octet-stream");
    header ("Content-Length: ".filesize($enlace));
    readfile($enlace);
    $fp=fopen("$enlace", "r");
    fpassthru($fp); 
   

¿Por qué los PDF no me los baja con su correspondiente extensión?

[marx-pola]

Listo... el error estaba en esta linea:

Código PHP:

  header("Content-type: application/octet-stream");
header("Content-Disposition: attachment; filename=\"$f\"\n"); 
   

Me faltaban las líneas de escape en filename.