Esto realmente es seguro?

[Jose789]

Hola buenas, pues acabo de terminar una pequeña función para mi website que se trata de hacer un 'screenshot', realmente funciona perfectamente, no se si así es la mejor forma de hacerlo o directamente necesita algún tipo mas de verificación

Código PHP:

  
if(isset($_REQUEST['no']) AND (isset($_REQUEST['data']))){
    $upload_dir = "ss/";
    $img = $_REQUEST['data'];
    $img = str_replace('data:image/jpeg;base64,', '', $img);
    $img = str_replace(' ', '+', $img);
    $data = base64_decode($img);
    
    //var_dump($data);
    $file = $upload_dir.$_REQUEST['no'].".jpg";

    file_put_contents($file, $data);
}else{die('Error');}  
   

Gracias y un saludo.

[Eleazan]

No........

Código PHP:

   $file = $upload_dir.$_REQUEST['no'].".jpg";  
   

Por ejemplo... ¿Q hay en 'no'? Imagina, que me da a mi por enviar una imagen en data, y meto en no algo así como '../images/logo'

¿Qué pasaria? ;)

Y que escribas en un fichero, algo procedente de un request... pues, a mi, personalmente, no me hace gracia. No sé si podrían hacer algún tipo de code injection pero me aseguraría bien de que el contenido fuera el esperado ;)

[Jose789]

Iniciado por Eleazan

No........

Código PHP:

   $file = $upload_dir.$_REQUEST['no'].".jpg";  
   

Por ejemplo... ¿Q hay en 'no'? Imagina, que me da a mi por enviar una imagen en data, y meto en no algo así como '../images/logo'

¿Qué pasaria? ;)

Y que escribas en un fichero, algo procedente de un request... pues, a mi, personalmente, no me hace gracia. No sé si podrían hacer algún tipo de code injection pero me aseguraría bien de que el contenido fuera el esperado ;)

Gracias por comentar, he realizado una seria de cambios, pero no se si se hará falta mas clase de 'filtros'

Código PHP:

  if(isset($_POST['no']) AND (isset($_POST['data']))){
    
    $upload_dir = "ss/";
    $img = $_REQUEST['data'];
    $img = str_replace('data:image/jpeg;base64,', '', $img);
    $img = str_replace(' ', '+', $img);
    $data = base64_decode($img);
    
    $file = $upload_dir.$_REQUEST['no'].".jpg";
    $is_image = imagecreatefromstring($data);
    
    if ($is_image != false) {
        file_put_contents($file, $data);
    } else {
        die('Error');
    }
    
    file_put_contents($file, $data);
}else{die('Error');}  
   

[Jose789]

Alguna ayuda?=