[SOLUCIONADO] Descarga de archivos mysql - archivo dañado

[Murallitz]

Buenos dias, estoy grabando documentos (docx, xlsx) en mysql (long bob)
Este es el código de subida:

Código PHP:

  $Doc_tmpname = $_FILES['documento']['tmp_name'];   
$Doc_Tipo = $_FILES['documento']['type'];  
$Doc_Nombre = $_FILES['documento']['name']; 
$Doc_Tamano = $_FILES['documento']['size']; 
$Doc_ext=pathinfo($Doc_Nombre, PATHINFO_EXTENSION); 
$accion ="1"; 
$fp = fopen($Doc_tmpname, 'r'); 
$contenido = fread($fp, filesize($Doc_tmpname)); 
$contenido = addslashes($contenido); 
fclose($fp); 
/*Grabamos Documento*/ 
 
if($Doc_ext=="png"||$Doc_ext=="PNG"||$Doc_ext=="JPG"||$Doc_ext=="jpg"||$Doc_ext=="jpeg"||$Doc_ext=="JPEG" 
        ||$Doc_ext=="pdf"||$Doc_ext=="PDF"||$Doc_ext=="doc"||$Doc_ext=="DOC"||$Doc_ext=="docx"||$Doc_ext=="DOCX" 
        ||$Doc_ext=="XLS"||$Doc_ext=="xls"||$Doc_ext=="XLSX"||$Doc_ext=="xlsx"||$Doc_ext=="xlsm"||$Doc_ext=="XLSM") 
 
 
{ 
$Activo=1; 
$grabadocumento = $mysqli->prepare("INSERT INTO tb_documento (Nombre,Tipodoc,Fecha,Activo,Data,ID_Empresa,Size) VALUES ('$Doc_Nombre','$Doc_Tipo', '$Fecha','$Activo','$contenido','$ID_Empresa','$Doc_Tamano')"); 
$grabadocumento->execute();  
$grabadocumento->close(); 
}  
   

y este es el de descarga

Código PHP:

  $ID_Documento = $_GET['ID_Documento']; 
$consulta = "SELECT Nombre, Tipodoc, Data, Size FROM tb_documento WHERE ID_Documento=$ID_Documento"; 
$resultado=$mysqli->query($consulta); 
$fila = $resultado->fetch_assoc(); 
$Doc_data = $fila['Data']; 
$Doc_nombre = $fila['Nombre']; 
$Doc_tipo = $fila['Tipodoc']; 
$Doc_size = $fila['Size']; 
 
         header("Content-length: $Doc_size"); 
         header("Content-type: $Doc_tipe"); 
         header("Content-Disposition: attachment; filename=$Doc_name"); 
       ob_clean(); 
         flush(); 
         echo $Doc_data; 
         $resultado->free(); 
         exit;  
   

El problema es que al descargarlo abre el archivo y sale este mensaje

"El archivo XXXXX.docx no puede abrirse porque hay problemas con el contenido"
Luego que se le da aceptar sale:
"Word encontro contenido no legible. ¿Desea recuperar el contenido de este documento?"
Le das aceptar y se abre, pero la idea es que esto no aparezca pues se ve mal.

Ya hice echo de los datos que trae desde la tabla y son estos:
$Doc_tipo= application/vnd.openxmlformats-officedocument.wordprocessingml.document
$Doc_size= 1530
$Doc_nombre= nombre.docx

Que podria ser?
uso PHP:7.0.15
y Mysql : 5.6.32

Saludos

[alvaro_trewhela]

Totalmente desaconsejado usar motores de bases de dato como gestor de archivos, están hechos para guardar, valga la redundancia datos y no archivos, te recomiendo que guardes en la bd la dir del archivo y listo, pero si continuas con blobs lo que estás haciendo mal es usar

header("Content-length: $Doc_size");

doc_size no te esta dando el tamaño real del archivo en vez de usar $doc_size usa strlen($Doc_data);


Otra cosa cuidado con ese sql injection estas pasando un $_GET directo a la sentencia sql, te van a destruir la base de datos en menos de un parpadeo.

[Murallitz]

Muchas gracias por los consejos de seguridad, los tedré en cuenta.
Esta mejor asi?
<?php
$grabadocumento = $mysqli->prepare("INSERT INTO tb_documento (Nombre,Tipodoc,Fecha,Activo,Data,ID_Empresa,Size) VALUES (?,?,?,?,?,?,?)");
$grabadocumento->bind_param("sssisii", $Doc_Nombre,$Doc_Tipo,$Fecha,$Activo, $contenido, $ID_Empresa,$Doc_Tamano);
$grabadocumento->execute();
$grabadocumento->close();
php?>

Sobre grabar en la bd, lamentablemente no puedo guardar en el servidor, debe ser directo a la BD.

Aun asi no funciona
Gracias

[alvaro_trewhela]

Te dejo una guía

file table

Código SQL:

CREATE TABLE files(
    id INT NOT NULL AUTO_INCREMENT,
    name VARCHAR(100) NOT NULL,
    cont BLOB, 
    PRIMARY KEY(id)
)
 

list.php

Código PHP:

<?php
 
$server = ["server"=>"localhost", "user"=>"root", "pass"=>"", "db"=>"test"];
 
$con = new mysqli($server["server"], $server["user"], $server["pass"], $server["db"]);
if (!$con->connect_errno) {
$q = $con->query("SELECT * FROM files");
    if($q->num_rows > 0){
    ?>
    <table>
        <tr>
            <td>Name</td>
            <td>Size</td>
            <td>Link</td>
        </tr>
        <?php
        while($file = $q->fetch_assoc()){
        echo "<tr>";
            echo "<td>".$file["name"]."</td>";
            echo "<td>".round(strlen($file["cont"])/1024, 2)." Kb</td>";
            echo "<td><a href=\"download.php?id=".$file["id"]."\">Download</a></td>";
        echo "</tr>";
        }
        ?>
    </table>
    <?php
    }
    else{
    echo "No files";
    }
}
else{
echo "DB ERROR";
}
?>
 

download.php

Código PHP:

<?php
 
$server = ["server"=>"localhost", "user"=>"root", "pass"=>"", "db"=>"test"];
 
function my_is_int($i){
    try{
    intval($i);
    return true;
    }
    catch(Exception $e){
    return false;
    }
}
 
$id = -1;
if(isset($_GET["id"]) && my_is_int($_GET["id"])) $id = intval($_GET["id"]);
    if($id != null){
    $con = new mysqli($server["server"], $server["user"], $server["pass"], $server["db"]);
        if (!$con->connect_errno) {
        $q = $con->query("SELECT * FROM files WHERE id=$id");
            if($q->num_rows > 0){
            $file = $q->fetch_assoc();
            header("Content-Type: application/octet-stream");
            header("Content-Transfer-Encoding: Binary"); 
            header("Content-disposition: attachment; filename=\"" .$file["name"]. "\"");
            header("Content-length: ".strlen($file["cont"]));
            echo $file["cont"];
            exit();
            }
        }
    }
echo 0; //send a 0 if some error...
?>
 

upload.php

Código PHP:

<?php
 
 
function myfread($file){
    if(file_exists($file)){
    $fo = fopen($file,"r");
    $cnt = fread($fo, filesize($file));
    fclose($fo);
    return $cnt;
    }
return "";
}
 
$server = ["server"=>"localhost", "user"=>"root", "pass"=>"", "db"=>"test"];
 
$con = new mysqli($server["server"], $server["user"], $server["pass"], $server["db"]);
if (!$con->connect_errno) {
    if(isset($_FILES["file"])){
    $f = $_FILES["file"];
    $st = $con->prepare("INSERT INTO files (name, cont) VALUES (?, ?)");
    $st->bind_param("ss", $f["name"], myfread($f["tmp_name"]));
        if ($st->execute())
            echo "uploaded";
        else
            echo "some error occurred";
    }
    else{
    ?>
    <form method="post" action="" enctype="multipart/form-data">
    <input type="file" name="file"/><input type="submit" value="upload" />
    </form>
    <?php
    }
}
else{
echo "DB ERROR";
}
?>