upload-script, billeder på maksimalt 100Kb

jeg vil tro at if($size[0] > 450) { - skal ændres til if($size[0] > 100) {

der regnes i bytes, så: 100*1024

getimagesize() finder ikke billedets størrelse i bytes :)

Sorry, getimagesize() bruges til at finde billedets dimentioner.
if ( $_FILES['upfile']['size'][0])> 100*1024 ) {
// fejl
}

Den finder billedets størrelse (px) i bredden, højden, samt billedets type.

Du skal bruge $_FILES['upfile']['size'], som indeholder billedets størrelse i bytes.

sone+taskmgr>> der var jeg vist lidt for hurtig ;o)

avminarm: Det ka' jo ske for selv de bedste ;)

Scriptet ja, tager højde for dimensionen samt mime-typerne (filendelsen), dette er helt fint. Ønsker blot at få tilføjet et check på Kb-størrelsen. Hvis muligt så post hele scriptet inkl. tilføjelsen.

Prøv dette:

<?

$pass = sprintf( "%04d", rand( 0, 999999 ) );

if (isset($_POST['Submit'])) {
//stien, hvortil filerne skal placeres
$dist = "./temp/";
//Filer som kan uploades
$filetyper = array("image/gif", "image/jpeg", "image/jpeg", "image/png");
$size = getimagesize($_FILES['upfile']['tmp_name'][0]);
$bytesize = $_FILES['upfile']['size'][0];

if($size[0] > 450) {
echo"Billedet er for stort, prøv igen...<br>";
$ok = "nej";
}elseif($bytesize > 100*1024){
echo "Billedet fylder mere end 100 kb, det går ikke.<br>";
$ok = "nej";
}
if($ok != "nej") {
for($i = 0; $i <= 2; $i++) {
if (isset($_FILES['upfile']['name'][$i]) && $_FILES['upfile']['name'][$i] !="") {
//se http://dk.php.net/manual/en/function.is-uploaded-file.php for "is_uploaded_file"
if(is_uploaded_file($_FILES['upfile']['tmp_name'][$i])) {
//Flyt filer fra upload-tmp til et dir. Husk at ændre "$dist" ovenover så det passer med din server :O)
$navn = $pass + 1;
move_uploaded_file($_FILES['upfile']['tmp_name'][$i], $dist . $navn . ".gif");
echo"Din billed ";

echo $pass;

echo" er nu uploadet...<br>";


echo"<img src='$dist$navn.gif' border=0 width=110 height=82></a><br>";

}
}
}
}
}
?>

<style type="text/css">
<!--
.style1 {font-family: Verdana, Arial, Helvetica, sans-serif}
-->
</style>

<span class="style1">Upload billeder <br>
</span>
<form action="<? echo $_SERVER['PHP_SELF']; ?>" method="post" name="form1" class="style1" enctype='multipart/form-data'>
<br>
Vælg billedfil:
<input type="file" name="upfile[]">
<br><br>
<input type="submit" name="Submit" value="Indl&aelig;s">
</form>

En uheldig detalje er dog at scriptet tillader upload af fx php-filer, dog udelukkende omdøbt til gif. De 100 point er dine taskmgr og på vej.

[sone lidt fornærmet]: Min løsning kl. 28/07-2004 14:59:47 dur ikke, eller...?

Så kan du bare udvide dit tjek lidt (er ikke testet):

if($size[0] > 450) {
echo"Billedet er for stort, prøv igen...<br>";
$ok = "nej";
}elseif($bytesize > 100*1024){
echo "Billedet fylder mere end 100 kb, det går ikke.<br>";
$ok = "nej";
}elseif(!in_array($_FILES['upfile']['type'][0],$filetyper)){
echo "Filtypen er ikke tilladt.<br>";
$ok = "nej";
}

Blev afledt af dit ”Sorry”, sone - hvad gør vi nu taskmgr?

Du kan jo fordele dem som du syn's :) sone fortjener naturligvis også point og jeg tager dem alligevel ikke så højttydeligt.

sone 50/50?

Checket spærrer fint for uønskede filtyper, lukker gif igennem, men desværre ikke jpg. Afventer lige et svar fra sone, før jeg smider pointene.

Prøv at ændre til:
$filetyper = array("image/gif", "image/jpg", "image/jpeg", "image/png");

Du er nødt til at checke på filens extension, det er ikke nok med image/* - du kan smide et phpscript i røven af en gif, og det vil blive kørt hvis filen hedder billede.php (som har en image/gif mime-type).

Eller selv sætte en extension alt efter hvad mime-typen er.

Har ændret, men får stadig: ”Filtypen er ikke tilladt”

Hermed 50 point til både sone og taskmgr, takker for indsatsen.