Sicherheit beim Upload

[dedlfix]

Hi!

Aber, hmm, dann hat man im $lockdir einen Haufen "unnütze" 0-Byte-Dateien rumliegen, beziehungsweise muss sie erst einmal für die bereits vorhandenen $destdir-Dateien anlegen und darf sie nicht löschen. Besonders hübsch ist das nicht.
Man kann ja zuerst fopen(x) auf die Zieldatei machen, dann nochmal auf eine Lock-Datei, dann muf(), und danach die Lockdatei wieder löschen.

Wird auch nicht wesentlich hübscher.

Je länger wir drüber diskutieren, desto mehr komme ich aber zur Ansicht, dass die Funktion move_uploaded_file() eigentlich vielmehr einen dritten, optionalen Parameter BOOLEAN $overwrite bekommen sollte, mit dem man (default true) das Überschreiben der vorhandenen Datei aktiviert oder (false) einen Error erhält, wenn die Zieldatei schon vorhanden ist.

Sollte auch nicht allzu schwierig zu implementieren sein. Sagst du dem chris_se Bescheid, dass er mal eben einen Patch dafür entwickeln und einreichen soll, der kennt sich doch mit C und dem Innenleben von PHP aus :-)

Lo!