Sikkerhed ved fil upload

Forfalsker man mime-typen, uploader en fil med navnet x.php, og sætter de bytes
forrest som får getimagesize til at svare korrekt, så har man en php-fil på din
server. Den ligger så godt nok udenfor webroden, og kan ikke udføres, men alligevel:
check også at extension er blandt dem du vil have.

Jep, det lyder som en ok idé at tilføje den del også.

Ellers kan man vel ikke gøre ret meget mere?

Joh, men kan derudover lægge dem til manuel kontrol, før de bliver brugt til noget ;)
Men derudover kan jeg umiddelbart ikke se mere.

Det skal de faktisk :)

Så siger jeg tak, smid et svar.

Nej tak, jeg samler slet ikke på point. Svar selv, og accepter eget svar.

Fair nok :)

Jeg poster lige min kode lidt senere, så kan du lige se den og andre kan måske drage nytte af den...

Her er min kode, du/andre er velkommen til at kommentere den og evt. komme med forslag til forbedringer.

-------------------------------------------------------------------------
if (isset($_POST['wantimage'])) //if user wants own image then...
    {
        /*****    image upload code begins    *****/
        if(($_FILES["upfil"]["size"] > 0) || ($_FILES['upfile']['name'] != "")) //if filesize above 0 or filename longer than zer0
        {
            $configuration["upload_dir"] = "/domains/www.luckystar.dk/memberprofileimages"; //where to put images
            $configuration["max_size"] = "2500"; //kilobytes
           
            // Move from where to...
            $from = $_FILES["upfil"]["tmp_name"];
            $to = $configuration["upload_dir"] . "/" . $_FILES["upfil"]["name"];
           
            // Filesize accepted?
            $file_size = filesize($from)/1024;
            if($file_size > $configuration["max_size"])
            {
              echo('<script type=\'text/javascript\'>alert(\'FEJL: Filen er for stor.\n\nMax størrelsen er '. $configuration["max_size"] .' kb,\nog dit billede fylder '. ceil($file_size) .' kb.\');</script>');
              exit();
            }
           
            //checks mime type of file
            $mimetype = array("image/gif", "image/jpeg", "image/pjpeg");
            if (in_array($_FILES['upfil']['type'], $mimetype))
            {
                //all good...
            }
            else
            {
                echo('<script type=\'text/javascript\'>alert(\'FEJL: Filen er ikke af typen .jpg eller .gif!\n\nFind venligst et billede af denne type,\neller undlad at uploade et profilbillede nu.\');</script>');
                exit();
            }
                       
            //Checks files original ending
            $file_extension = strrchr($_FILES["upfil"]["name"], ".");
            if ($file_extension == ".jpg" || $file_extension == ".gif" || $file_extension == ".jpeg")
            {
                $memberimage = time()."_".$_SESSION['memberid'].$file_extension;
            }
            else
            {
                echo('<script type=\'text/javascript\'>alert(\'FEJL: Filendelsen er ikke .jpg, .jpeg eller .gif!\n\nFind venligst et billede af denne type,\neller undlad at uploade et profilbillede nu.\');</script>');
                exit();
            }

            //Places the file in the specified upload directory....
            if(function_exists("move_uploaded_file"))
            {
              $filename = $configuration["upload_dir"] . "/" . time()."_".$_SESSION['memberid'].$file_extension;
              move_uploaded_file($_FILES['upfil']['tmp_name'], $filename);
            }
            else
            {
              copy($from, $to);
            }

            //Checks image height/width/type... If false, then it's not an image and will be deleted (unlinked)
            $result_array = getimagesize($filename);
            if($result_array == FALSE)
            {
                unlink($filename);
                echo('<script type=\'text/javascript\'>alert(\'FEJL: Filen er ikke et billede!\');</script>');
                exit();
            }
        }
        else //if no picture has been selected or filesize equals 0
        {
                echo('<script type=\'text/javascript\'>alert(\'FEJL: Du har ikke valgt et billede!\');</script>');
                exit();
        }
        /*****    image upload code ends    *****/
    }
    else //if user wants no image uploaded
    {
    $memberimage = "noimage.jpg"; //inserted to db
    }
-------------------------------------------------------------------------