PHP file upload unexpected error - file too large?

[Giulio Neri]

Hi,
I have a short basic script to upload files. It works fine with small files,
but with longer files it gets stuck.

Here's the fragment, the input file is loaded in "$file" from an HTML form
as usual.

....
$img_str = fread(fopen($file, "r"), filesize($file));
$data = addslashes($img_str);
// --------------------- up to here the execution is correct and fast (<1
second)

$sql="INSERT INTO $table (".
" file_name,".
" file_type,".
" file_size,".
" bin_data)".
" VALUES (".
" '$file_name',".
" '$file_type',".
" '$file_size',".
" '$data')";
// --------------------- it gets stuck
here!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
mysql_query($sql) or die ("SQL error ..$sql");

I have already set the php.ini variables as follows:
memory_limit = 16M
upload_max_filesize = 6M

I also checked the execution time to max_execution_time = 300 (even though I
don't believe it should be necessary).
The server is a P4 1.8 GHz, 256MB RAM

Any suggestion? Am I missing out something?

Thanks in advance,
G.

[Erwin Moller]

Giulio Neri wrote:

Hi,
I have a short basic script to upload files. It works fine with small
files, but with longer files it gets stuck.

Here's the fragment, the input file is loaded in "$file" from an HTML form
as usual.

...
$img_str = fread(fopen($file, "r"), filesize($file));
$data = addslashes($img_str);
// --------------------- up to here the execution is correct and fast (<1
second)

$sql="INSERT INTO $table (".
" file_name,".
" file_type,".
" file_size,".
" bin_data)".
" VALUES (".
" '$file_name',".
" '$file_type',".
" '$file_size',".
" '$data')";
Are you aware that the query is very strange?
Since you put ' instead of " around stringliterals, the SQL you produce
might differ somewhat from what you expect.

Always try:
echo $sql;
before posting to a newsgroup. :P

You will have something like this:

INSERT INTO sometable (file_name,file_type,file_size,bin_data)
VALUES
('$file_name', '$file_type' etc. etc)

instead of the expected VALUES for the vars you use.

Check your ' and "

Regards,
Erwin Moller
// --------------------- it gets stuck
here!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
mysql_query($sql) or die ("SQL error ..$sql");

I have already set the php.ini variables as follows:
memory_limit = 16M
upload_max_filesize = 6M

I also checked the execution time to max_execution_time = 300 (even though
I don't believe it should be necessary).
The server is a P4 1.8 GHz, 256MB RAM

Any suggestion? Am I missing out something?

Thanks in advance,
G.

[Giulio Neri]

Erwin,
the single quotation mark is required in the sql syntax to insert text /
non-numeric values (mysql gives me an error otherwise).

The real problem here is that the string concatenation fails and I cannot
make it work...

I also tried with something like:

$sql="hello: $data";

or

$sql="hello: " . $data;

and in both cases it fails (I believe it runs out of memory...).

The file size is 4MB.
Thanks for any additional advise.
G.
"Erwin Moller"
<si******************************************@spam yourself.com> wrote in
message news:42***********************@news.xs4all.nl...

Giulio Neri wrote:

Hi,
I have a short basic script to upload files. It works fine with small
files, but with longer files it gets stuck.

Here's the fragment, the input file is loaded in "$file" from an HTML form as usual.

...
$img_str = fread(fopen($file, "r"), filesize($file));
$data = addslashes($img_str);
// --------------------- up to here the execution is correct and fast (<1 second)

$sql="INSERT INTO $table (".
" file_name,".
" file_type,".
" file_size,".
" bin_data)".
" VALUES (".
" '$file_name',".
" '$file_type',".
" '$file_size',".
" '$data')";

Are you aware that the query is very strange?
Since you put ' instead of " around stringliterals, the SQL you produce
might differ somewhat from what you expect.

Always try:
echo $sql;
before posting to a newsgroup. :P

You will have something like this:

INSERT INTO sometable (file_name,file_type,file_size,bin_data)
VALUES
('$file_name', '$file_type' etc. etc)

instead of the expected VALUES for the vars you use.

Check your ' and "

Regards,
Erwin Moller

// --------------------- it gets stuck
here!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
mysql_query($sql) or die ("SQL error ..$sql");

I have already set the php.ini variables as follows:
memory_limit = 16M
upload_max_filesize = 6M

I also checked the execution time to max_execution_time = 300 (even though I don't believe it should be necessary).
The server is a P4 1.8 GHz, 256MB RAM

Any suggestion? Am I missing out something?

Thanks in advance,
G.

[Erwin Moller]

Giulio Neri wrote:

Erwin,
the single quotation mark is required in the sql syntax to insert text /
non-numeric values (mysql gives me an error otherwise).

Right,

But did you echo the SQL-string before executing?
What came out?

Regards,
Erwin Moller
The real problem here is that the string concatenation fails and I cannot
make it work...
that is why I advised you to check the ' and " in the first place.

Try something like this.

$sql="INSERT INTO $table (file_name,file_type,file_size,bin_data)";
$sql.= "VALUES ('".$file_name"', '".$file_type."' etc. etc

Just use . to concatenate.

Good luck.
Let me/us know if you succeed.

Regards,
Erwin Moller


I also tried with something like:

$sql="hello: $data";

or

$sql="hello: " . $data;

and in both cases it fails (I believe it runs out of memory...).

The file size is 4MB.
Thanks for any additional advise.
G.
"Erwin Moller"
<si******************************************@spam yourself.com> wrote in
message news:42***********************@news.xs4all.nl...

Giulio Neri wrote:

> Hi,
> I have a short basic script to upload files. It works fine with small
> files, but with longer files it gets stuck.
>
> Here's the fragment, the input file is loaded in "$file" from an HTML form > as usual.
>
> ...
> $img_str = fread(fopen($file, "r"), filesize($file));
> $data = addslashes($img_str);
> // --------------------- up to here the execution is correct and fast (<1 > second)
>
> $sql="INSERT INTO $table (".
> " file_name,".
> " file_type,".
> " file_size,".
> " bin_data)".
> " VALUES (".
> " '$file_name',".
> " '$file_type',".
> " '$file_size',".
> " '$data')";

Are you aware that the query is very strange?
Since you put ' instead of " around stringliterals, the SQL you produce
might differ somewhat from what you expect.

Always try:
echo $sql;
before posting to a newsgroup. :P

You will have something like this:

INSERT INTO sometable (file_name,file_type,file_size,bin_data)
VALUES
('$file_name', '$file_type' etc. etc)

instead of the expected VALUES for the vars you use.

Check your ' and "

Regards,
Erwin Moller

> // --------------------- it gets stuck
> here!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
> mysql_query($sql) or die ("SQL error ..$sql");
>
> I have already set the php.ini variables as follows:
> memory_limit = 16M
> upload_max_filesize = 6M
>
> I also checked the execution time to max_execution_time = 300 (even though > I don't believe it should be necessary).
> The server is a P4 1.8 GHz, 256MB RAM
>
> Any suggestion? Am I missing out something?
>
> Thanks in advance,
> G.

[Giulio Neri]

Hi Erwin,
I did insert a exit($sql); before executing as well as a exit("test");

in both cases the program did not reach the exit command line

It is almost definitely something to do with the string concatenation, as
many other tests I have done show and I don;t know how to get around that!

Thanks for helping me.

"Erwin Moller"
<si******************************************@spam yourself.com> wrote in
message news:42***********************@news.xs4all.nl...

Giulio Neri wrote:

Erwin,
the single quotation mark is required in the sql syntax to insert text /
non-numeric values (mysql gives me an error otherwise).

Right,

But did you echo the SQL-string before executing?
What came out?

Regards,
Erwin Moller

The real problem here is that the string concatenation fails and I cannot make it work...

that is why I advised you to check the ' and " in the first place.

Try something like this.

$sql="INSERT INTO $table (file_name,file_type,file_size,bin_data)";
$sql.= "VALUES ('".$file_name"', '".$file_type."' etc. etc

Just use . to concatenate.

Good luck.
Let me/us know if you succeed.

Regards,
Erwin Moller

I also tried with something like:

$sql="hello: $data";

or

$sql="hello: " . $data;

and in both cases it fails (I believe it runs out of memory...).

The file size is 4MB.
Thanks for any additional advise.
G.
"Erwin Moller"
<si******************************************@spam yourself.com> wrote in
message news:42***********************@news.xs4all.nl...

Giulio Neri wrote:

> Hi,
> I have a short basic script to upload files. It works fine with small
> files, but with longer files it gets stuck.
>
> Here's the fragment, the input file is loaded in "$file" from an HTML

form

> as usual.
>
> ...
> $img_str = fread(fopen($file, "r"), filesize($file));
> $data = addslashes($img_str);
> // --------------------- up to here the execution is correct and

fast (<1

> second)
>
> $sql="INSERT INTO $table (".
> " file_name,".
> " file_type,".
> " file_size,".
> " bin_data)".
> " VALUES (".
> " '$file_name',".
> " '$file_type',".
> " '$file_size',".
> " '$data')";

Are you aware that the query is very strange?
Since you put ' instead of " around stringliterals, the SQL you produce
might differ somewhat from what you expect.

Always try:
echo $sql;
before posting to a newsgroup. :P

You will have something like this:

INSERT INTO sometable (file_name,file_type,file_size,bin_data)
VALUES
('$file_name', '$file_type' etc. etc)

instead of the expected VALUES for the vars you use.

Check your ' and "

Regards,
Erwin Moller

> // --------------------- it gets stuck
> here!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!!
> mysql_query($sql) or die ("SQL error ..$sql");
>
> I have already set the php.ini variables as follows:
> memory_limit = 16M
> upload_max_filesize = 6M
>
> I also checked the execution time to max_execution_time = 300 (even

though

> I don't believe it should be necessary).
> The server is a P4 1.8 GHz, 256MB RAM
>
> Any suggestion? Am I missing out something?
>
> Thanks in advance,
> G.

[Erwin Moller]

Giulio Neri wrote:

Hi Erwin,
I did insert a exit($sql); before executing as well as a exit("test");

in both cases the program did not reach the exit command line

It is almost definitely something to do with the string concatenation, as
many other tests I have done show and I don;t know how to get around that!

Thanks for helping me.

Did you read the rest of my posting?
It contained an improved $sql statement.
Try to use it in your situation.

(I said 2 times, 'Regards, Erwin Moller')
So that probably made you stop reading after the first. :P
Sorry.

Regards,
Erwin Moller

[Giulio Neri]

The syntax I used is the same reported in many SQL, MySQL and PHP manuals.

Anyway I solved the problem:

1- increased memory available to script
memory_limit = 16M
upload_max_filesize = 6M

2- use of unset($temp_variables) in order to release memory

3- set mysql configuration file option:
max_allowed_packet=16M

It all works now...

Thanks anyway,
G.
"Erwin Moller"
<si******************************************@spam yourself.com> wrote in
message news:42***********************@news.xs4all.nl...

Giulio Neri wrote:

Hi Erwin,
I did insert a exit($sql); before executing as well as a exit("test");

in both cases the program did not reach the exit command line

It is almost definitely something to do with the string concatenation, as many other tests I have done show and I don;t know how to get around that!
Thanks for helping me.

Did you read the rest of my posting?
It contained an improved $sql statement.
Try to use it in your situation.

(I said 2 times, 'Regards, Erwin Moller')
So that probably made you stop reading after the first. :P
Sorry.

Regards,
Erwin Moller