image upload, image type not working

[Paul_Ferrie]

For some reason the upload script is only allowing certain jpeg or png files. I am unable to find a reason why 🙁
Can some one help me please
Cheers

include('connection.php');
//Build an array of allowed files types
	$types_array = array("image/gif","image/jpeg","image/pjpeg","image/x-png");
//Check file against array	
	if (!in_array($_FILES['userfile']['type'], $types_array)) {
//Return error if file typ dosnt match array	
   	header("Location: addimage.php?error=That file type is not allowed!"); 
	exit;
	}
//Set max file size
	$max_filesize = 10000;
//Check file does not exceed allowed size	
	if ($_FILES['userfile']['size'] > $max_filesize) {
  	header("Location: addimage.php?error=Image is to large!<br> Image should be no more than 1000kb");
   	exit; 
	}	
//Get image width and hieght
    $imagesize = getimagesize($_FILES['userfile']['tmp_name']);
    $imagewidth = $imagesize[0];
    $imageheight = $imagesize[1];
//Set allowed width and height	
    $maxwidth = 200;
    $maxheight = 200;
//Check image does not exeed allowed width and hieght
	if(($imagewidth > $maxwidth )or( $imageheight > $maxheight)) {
    header("Location: addimage.php?error=Image exceeds dimensions 200*200"); 
	exit;
	}
//Set the file name to all lowercase
 	$name = strtolower(eregi_replace('#| |\?|!', '',$_FILES['userfile']['name']));
//Set the destination of the file to be stored on the server
	$destination = '../img/'.$name;
//Move image to destination 	
    if(move_uploaded_file($_FILES['userfile']['tmp_name'], $destination)) {
//Set pic to hold the address of the image in the database
//Example ../img/some_image.jpg
	$pict='../img/'.$name;
//Insert all details to the database
    $query = mysql_query("UPDATE newsarchive SET pict = '$pict' WHERE newsId = '$newsId'"); 	 
//If query failed...
	if (!$query) {
//Inform of an error and quit
    fail("Couldn't set image");
     } 
header("Location: main.php?success=New picture has been has been added to row.  Please complete the rest of the fields"); 
}

[Mordecai]

Echo out the mimetype ($_FILES['userfile']['type'] and see what it's coming up as.

header("Location: main.php?success=New picture has been has been added to row.  Please complete the rest of the fields");

Also, this line doesn't mean anything. It will just confuse the browser, since you haven't specified protocol, server, or any of the other needed parts to a URI. I always have to ask myself, "why do people send Location headers without a full URI?"...
Note: this goes for all of your header()s, not just the last one.

[Paul_Ferrie]

I will try echoing the file type and see what it comes up with. Dont see how it is going to help me though.

cheers]

[Weedpacket]

Try using [man]getimagesize[/man] to determine the type of file; then you won't have to rely on the client to send exactly the right MIME string you're looking for when it uploads the file (E.g., you've missed "image/png", which is the official MIME type of a PNG image file).

Also

$max_filesize = 10000; 

limits the file size to just under 10kB, not to 1000kB.

[Paul_Ferrie]

I forgot that getimagesize has an array of information on the file being uploaded. why didnt i think of that earlier

cheers