Hallo allemaal,

Alweer worstel ik met een script waarmee ik bezig ben, ditmaal lijkt het script niet te uploaden terwijl ik de fout niet zie!

Dit is wat ik nu heb (niet de hele pagina, die enkel per pm):

PHP Code:


    define ("MAX_SIZE","2048"); 
 function getExtension($str) {
         $i = strrpos($str,".");
         if (!$i) { return ""; }
         $l = strlen($str) - $i;
         $ext = substr($str,$i+1,$l);
         return $ext;
 }

 $errors=0;
     $image=$_FILES['image']['name'];
     if ($image) 
     {
         $filename = stripslashes($_FILES['image']['name']);
          $extension = getExtension($filename);
         $extension = strtolower($extension);
 if (($extension != "jpg") && ($extension != "jpeg") && ($extension != "png") && ($extension != "gif")) 
         {
             echo 'Dit formaat wordt niet ondersteund!';
             $errors=1;
         }
         else
         {
 $size=filesize($_FILES['image']['tmp_name']);

if ($size > MAX_SIZE*1024)
{
    echo 'De foto is te groot!';
    $errors=1;
}

$image_name=time().'.'.$extension;
$newname="images/bedrijfsfoto/".$image_name;
$copied = copy($_FILES['image']['tmp_name'], $newname);
if (!$copied) 
{
    echo 'Er ging iets fout, probeer het nogmaals!';
    $errors=1;
}}}

 if(!$errors) 
 {    
 define("IMG_NAME","$newname");
 
 $foto_url = 'http://www.pasteacode.com/bso/'.IMG_NAME.'';
 
    if(
    $query_insert = mysql_query("INSERT INTO ads
    (
    naam,
    naam_op_site,
    adres,
    postcode,
    woonplaats,
    provincie,
    url,
    beschrijving,
    telefoon,
    halen_brengen,
    sport,
    verl_opvang,
    eigen_vervoer,
    mogelijkheden,
    plaatsen,
    groepen,
    email,
    foto_url
    ) 
    
    VALUES(
    '".$naam."',
    '".$naam_op_site."',
    '".$adres."',
    '".$postcode."',
    '".$woonplaats."',
    '".$provincie."',
    '".$url."',
    '".$beschrijving."',
    '".$telefoon."',
    '".$halen_brengen."',
    '".$sport."',
    '".$verl_opvang."',
    '".$eigen_vervoer."',
    '".$mogelijkheden."',
    '".$plaatsen."',
    '".$groepen."',
    '".$email."',
    '".$foto_url."'
    )
    ")
    
    ){
        echo'<br /><br />Uw advertentie is succesvol geplaatst en <a href="http://www.pasteacode.com/bso/advertenties.php" style="color: #000; font-weight: bold; text-decoration: none;">hier</a> te bekijken.';
    }else{
        echo'<br /><br />Er ging iets mis, loop alle velden na en probeer het nogmaals.';
    }
    
 }
    
} 


Hij upload het plaatje niet en zet dus ook niet de goede url in de database. Iemand die me hiermee kan helpen, in ruil voor een positieve feedback?

Alvast heel erg bedankt voor jullie tijd!

Gr.
Jos

Heeft u deze regel mee in de form tag gezet?
Anders zal deze geen images uploaden.

enctype="multipart/form-data"

vb. <form action="" enctype="multipart/form-data" method="">

Ja, dit staat er in:

PHP Code:

<form name="newad" method="post" enctype="multipart/form-data"> 
     <input type="file" name="image"> 


Aanvullend bericht:
Inmiddels is het verholpen, dank aan degene die mij hebben geholpen.

Origineel gepost door Jos van M

Aanvullend bericht:
Inmiddels is het verholpen, dank aan degene die mij hebben geholpen.

Het zou mooi zijn als je vertelde wat de fout was voor mensen die er ook mee worstelen. Een veel voorkomende fout met uploadscripts is dat mensen de map waar de fotos in moeten komen te staan niet chmodden zodat het script geen rechten heeft om fotos te copieren naar de doelmap.

Er was op een of andere manier een form in een form gekomen en bij het form waar het om ging ontbrakt het enctype. Gewoon schrijffoutje dus.

Hoi Jos,

Ik zou al ik jou was niet alleen op extensie checken maar ook of het wel een afbeelding is.
Een afbeelding zorgt ervoor dat mime altijd gevuld is bij de functie getimagesize.
Er zijn vast ook nog andere checks beschikbaar, maar een beetje extra veiligheid kan geen kwaad.

$fileInfo = getimagesize($tempFile);

if($fileInfo["mime"] == "")
{
echo 'Invalid mime type.';
}
else
{
//verder met uploaden
}

;) en misschien is het dan ook handig ervoor te zorgen dat de plaatjes geen uitvoerrechten hebben. Anders kun je alsnog php code in de comments van een plaatje zetten waarmee die prima voorbij de mime check komt.

Om de veiligheids wan voorstelling met checken op minetype uit de wereld te helpen.
Checken op mine type is NIET veilig dit wordt op de server gezet door middel van een client side http header welke te wijzingen valt.

Wanneer je veilig wilt controleren of een afbeelding ook echt een afbeelding is zou je moeten gaan werken met de gd library en als nog de afbeeldingen opslaan in een map waar de php parser is uitgeschakeld


Zag net dat iemand me voor was had dit bericht laat na het typen verzonden