strange image upload issues

[Druid_McGrew]

Ok I have an image upload that works great and checks for an existing file and file types. I have recently added a error check that makes sure the file size is below 1.4 mb

The problem I have is its not works. So for error checking purposes I have made the script just show me the file size on a new page and do nothing else. Here it is

if (isset($_POST['Save_photo'])) {
//another photo uploader
$fileName1 = trim($_FILES['new_photo']['name']);
$tmpName = $_FILES['new_photo']['tmp_name'];
$fileSize = $_FILES['new_photo']['size'];
$fileType = $_FILES['new_photo']['type'];	

$randName = md5(rand() * time());
$fileName = $randName.$fileName1;



print $fileSize; exit();

}

Now here is my issue. If I upload a file of say about 730KB it echos

756305

If I upload a file over 1.2MB or more it echos

I dont understand why. There is no other code on the page at the moment! Im quite new to php so I dont know any other ways to sort this out. Please help.

Thank you

[jorgepinho]

Hi,

You have to check you PHP config, there are size limits for uploads.

Verify that with phpinfo() on an empty page.

[Druid_McGrew]

cool thank you that brings back loads of info. Do you know what it is im looking for?

[jorgepinho]

So, just create an empty .php file and write:

<?php
phpinfo();
?>

then look for "upload_max_filesize"

Probably you have a very low value... you have to change that on php.ini

[Druid_McGrew]

Ha you will love this. Max_file_uploads is set to 20. I'm assuming that's MB as theres no value next to it. Also its set to accept all types of imagery.

😕

[jorgepinho]

Hi..

So you max upload is 20 bytes.... is enough ?

http://www.php.net/manual/en/ini.core.php#ini.upload-max-filesize

it is still strange.... but use the M anyway to try

[Druid_McGrew]

yes Im simply trying to restrict files over 1.4MB Sounds easy enough yet it doesnt work. Obviously its not working because when I try to print the size its saying 0 when its actually like 2.2MB but if the file is below 1MB it shows the size. Im a proper newbie as well so I really dont understand whats going on 🙁

if (isset($_POST['Save_photo'])) {
//another photo uploader
$fileName1 = trim($_FILES['new_photo']['name']);
$tmpName = $_FILES['new_photo']['tmp_name'];
$fileSize = $_FILES['new_photo']['size'];
$fileType = $_FILES['new_photo']['type'];		

ini_set('max_upload_filesize', 1519131);


print $fileSize; exit();


}

[jorgepinho]

Just before anything check the upload status with:

echo $_FILES['new_photo']['error'];

and the description:

file-upload.errors

[Druid_McGrew]

jorgepinho thank you for spending so much time on this. Right I did that and I get an error code of

0 on 730KB image

and

10 on a 2730 image.

There is no code for an error 10 on the manual it would seem

[bradgrafelman]

Er... can you do a [man]var_dump/man on the $_FILES array when you upload the larger image and paste the output for us to see? Seems quite strange that you should be getting an undocumented error code.

[Druid_McGrew]

Ok after a var_dump on a image that was 2743KB here is what it produces

array(1) { ["new_photo"]=> array(5) { ["name"]=> string(12) "1004.jpg" ["type"]=> string(0) "" ["tmp_name"]=> string(0) "" ["error"]=> int(1) ["size"]=> int(0) } }

[bradgrafelman]

Ah well there you go. The error was "1" after all.

As was mentioned above, [man]file-upload.errors[/man] will explain what that means (and the fix should become obvious at that point).

[Druid_McGrew]

Yes but when I run that phpinfo() thingy my max_upload_file size is set to 20????? I just double checked it.

[bradgrafelman]

20 what? Just "20" ? If so, that's obviously your problem. From [man]ini.core#ini.upload-max-filesize[/man]:

PHP Manual wrote:

upload_max_filesize [man]integer[/man]

The maximum size of an uploaded file.

When an [man]integer[/man] is used, the value is measured in bytes. Shorthand notation, as described in this FAQ, may also be used.

[Druid_McGrew]

yes there is no number next to it its just says 20. The thing is if i try images that are under 1.4MB it works absolutely fine.

Alls im worried about is if my site gets busy and people are uploading images left right and center i dont want to be worrying about crashing my hosts server because I havent restricted the upload properly I am with HEART INTERNET. Are my worries valid or am I worring about nothing? here is my code for the upload...

if (isset($_POST['Save_photo'])) {
//another photo uploader
//echo $_FILES['new_photo']['error'];
$fileName1 = trim($_FILES['new_photo']['name']);
$tmpName = $_FILES['new_photo']['tmp_name'];
$fileSize = $_FILES['new_photo']['size'];
$fileType = $_FILES['new_photo']['type'];		
$randName = md5(rand() * time());
$fileName = $randName.$fileName1;

$limit = 1519131;

//print $fileSize; exit();


//$limit_size = 1519131;
$folder = "{$_SERVER['DOCUMENT_ROOT']}members/images/{$row_rs_Club['county']}/";



//if ($fileSize >= $limit_size){ print "<div id='clear'></div><p class='formerrors'>Your uploaded file size is more than 1.4MB so please reduce the file size and then upload. Visit the help page to know how to reduce the file size.</p><div id='clear'>";
//exit();}	
//	}	
if ($fileName1=="") {$message =  "<div id='clear'></div><p class='formerrors'>Please choose a file to upload</p><div id='clear'></div>";}
//elseif ($fileType!== 'image/jpeg' || 'image/gif' || 'image/png' || 'image/JPG') {$error = "false";}
$types = array('image/jpeg', 'image/gif', 'image/png', 'image/JPG', 'image/pjpeg', 'image/x-png');
if (in_array($fileType, $types)) {

move_uploaded_file($tmpName , $folder.$fileName); 
$url = "New-photo-details.php?image=$image";        

	header("Location: $url");
	}

[Druid_McGrew]

ops without blocking out the file size bit 😉

[Druid_McGrew]

One thing. Thank you to all that posted a reply on this forum. It is much more friendly than other forums out there!!!!!

[bradgrafelman]

There's really no point in looking at the code at this point (regardless of what appears to work) since we've already uncovered a configuration problem.

Configure PHP properly, and then we'll go over the code.

[Druid_McGrew]

cool ok well just so i understand the configuration issue is with my host provider? im sorry for not understanding this properly.

[bradgrafelman]

Druid McGrew;10983947 wrote:

the configuration issue is with my host provider?

It could be, but ultimately you should also have control over PHP's configuration (a very large portion of it, anyway). In other words, your host should provide some way for you to define your own values for most of the PHP directives.

For example, some hosts have setup PHP to look for a secondary php.ini file in their customer's home directories (e.g. if your website was located at /home/druid/public_html/, PHP might look for /home/druid/php.ini). Others, who use Apache and have PHP integrated as an ISAPI module (as opposed to a CGI binary; you can check this via [man]phpinfo/man near the top), will let you use the 'php_value' (et al.) directives in a .htaccess file placed in the root of your website.

In summary, yes it is a configuration problem but no you shouldn't have to rely on your host to change it rather than doing so yourself.