Dateiupload Prüfung der Endung

supersass1

Ich habe folgenden Code für den Dateiaupload:

Code

if($_REQUEST['Send'])
{
        if($_FILES['Datei']['tmp_name'])
        {
move_uploaded_file($_FILES['Datei']['tmp_name'], "upload/".$_FILES['Datei']['name']);     
}}

das funktioniert auch, nun habe ich eine Mime-Prüfung, die auch funktioniert:

Code

if($_REQUEST['Send'])
{
        if($_FILES['Datei']['tmp_name'])
        {
                 $contentType = $_FILES['Datei']['type'];
                 if($contentType == "image/jpeg" || $contentType == "image/gif" || $contentType == "application/zip"  || $contentType == "image/png")


                 {

                         move_uploaded_file($_FILES['Datei']['tmp_name'], "upload/".$_FILES['Datei']['name']);

Alles anzeigen

nun möchte ich statt dieser eine Dateiendung vornehmen, da der Upload zum uploaden von SC2Replays (.SC2Replay) dient:

Code

if($_REQUEST['Send'])
{
        if($_FILES['Datei']['tmp_name'])
        {
                 $file_extensions = array('jpg', 'jpeg', 'jpe', 'gif', 'png', 'SC2Replay', 'sc2replay');


                 if (in_array(strtolower(getExtension($_FILES['name'])), $file_extensions))
                 {


                         move_uploaded_file($_FILES['Datei']['tmp_name'], "upload/".$_FILES['Datei']['name']);

Alles anzeigen

Leider funktioniert das nicht, warum?

threadi

Was macht denn die Funktion getExtension()?

supersass1

achso, sorry:

Code

function getExtension ($filename)
    {
        if(strrpos($filename, '.'))
             return substr($filename, strrpos($filename, '.')+1);
        return false;
    }

threadi

Da hätte ich dir gleich 2 Alternativen zu bieten:

Code

function getExtension($filename = "") {
 $pathinfo = pathinfo($filename);
 return $pathinfo['extension']; 
}

Code

function getExtension($filename = "") 
{ 
  return (false === ( $p = strrpos($filename, '.') ) ? '' : substr($filename, ++$p));
}

supersass1

Vielen Dank, aber leider klappt es nicht. Es kommt auch keine Fehlermeldung, er geht mit beiden deiner Varianten nicht in den if sondern in den else-Zweig.

threadi

Mit welcher Dateiendung versuchst Du es? Mit "SC2Replay" wird es nicht funktionieren, weil Du den Vergleichswert klein machst.

Btw. wäre der Weg über den Content-type der sicherere ..

Tobse

Vorsicht! Hier kann ein Hacker durch manuelles ändern des Dateinamens die datei irgendwo auf deinem Serverplatzieren und unschöne dinge anrichten.

PHP

function parseFilename($filename) {
    $pos=strpos($filename, dirname($filename));
    if (!$pos) $pos=0;
    return substr($filename, $pos);
}

Und threadi:
Daran liegt es glaub ich nicht, sondern:
@TS:

PHP

function getExtension ($filename) {
    $filename=parseFilename($filename);
    $dot=strrpos($file, ".");
    if ($dot) {
        return substr($file, $dot, strlen($file));
    }  else {
        return false;
    }
}
# Das hat bei mir IMMER geklappt, wobei das auch nichts anders tut als die Funktion vom TS

Von daher denke ich, es liegt eher am Inhalt von
$_FILES['Datei']['name']. Wie sieht denn der aus?

supersass1

Zitat von threadi

Btw. wäre der Weg über den Content-type der sicherere ..

Ich habe schon versucht ein Content-type selber zu definieren, aber das ist mir nicht gelungen:

Code

<configuration>


    <system.webServer>


        <staticContent>
            <mimeMap fileExtension=".SC2Replay" mimeType="image/sc2" />
            <mimeMap fileExtension=".sc2replay" mimeType="image/sc2" />
     </staticContent>


    </system.webServer>


</configuration>

Alles anzeigen

Zitat

Von daher denke ich, es liegt eher am Inhalt von
$_FILES['Datei']['name']. Wie sieht denn der aus?

Das ist nichts anderes als der Dateiname, z.B.: "Verschollener Tempel (4).SC2Replay"

Tobse:
Bei deinem Code ist laut Fehlermeldung parseFilename() nicht definiert

jojo87

evtl. solltest du in deiner Abfrage $_FILE['Datei']['name'] benutzen anstatt $_FILE['name']
Oder ist das nur ein Vertipper?

Tobse

Hehe, mal wieder alle blind gewesen. Ich wette, da liegt der hase im pfeffer.
@TS:
die Funktion gobt es standardmäsig auch nicht, die habe ich im codeschnipsel darüber definiert. Der muss natürlich dazu

supersass1

vielen Dank Jungs, jojo87 hat recht, es lag lediglich an $_FILE['Datei']['name']

Tags