Tom: Upload

Beitrag lesen

Hello,

if(copy($_FILES["file"]["tmp_name"], $_FILES["file"]["name"])) {

Das ist prinzipiell gefährlich. Hier fehlt die explizite Pfadangabe für das persistente Abspeichern und außerdem wird einer Usereingabe $_FILES["file"]["name"] blind vertraut.

Ersten sollen die Files doch wohl kaum im aktuellen Arbeitsverzeichnis [siehe ->pwd()] gepeichert werden und auch nicht in einem beliebigen, vom Benutzer bestimmbaren Pfad, oder?

http://www.php.net/manual/de/function.basename.php

und außerdem sollte man als erstes fragen:

if(isset($_FILES[$uploadfiledname]['error'] and $_FILES[$uploadfiledname]['error']===0))
     {
         ## alles OK, es kann weitergearbeitet werden

}

Liebe Grüße aus Syburg bei Dortmund

Tom vom Berg

--
Nur selber lernen macht schlau
http://bergpost.annerschbarrich.de