Make WordPress Core

Opened 16 years ago

Closed 16 years ago

Last modified 16 years ago

#8149 closed defect (bug) (fixed)

Automatic upgrade runs out of memory

Reported by: dd32's profile DD32 Owned by:
Milestone: 2.7 Priority: normal
Severity: normal Version: 2.7
Component: Upgrade/Install Keywords: dev-reviewed
Focuses: Cc:

Description

It appears that for some people, 32MB of PHP memory_limit just isnt enough for the PclZip class to do its work

It might be an idea to set the memory limit in the unzip function to something higher (Like happens in the Thumbnail creation -- Since GD can use obscene ammounts of memory sometimes, it attempts to set it to 256MB memory_limit) to prevent these errors occuring half way through the upgrade decompression

Attachments (1)

8149.diff (512 bytes) - added by ryan 16 years ago.

Download all attachments as: .zip

Change History (6)

@ryan
16 years ago

#1 @ryan
16 years ago

Sounds good.

#2 @westi
16 years ago

  • Keywords dev-reviewed added

+1

i doubt it's actually the unzipping that has swallowed all 32M but other stuff has just got us too close to the limit.

#3 @DD32
16 years ago

Yeah, most of the 32M is used up by the rest of WP, But loading 7MB worth of strings in PHP definately takes its toll, along with the temporary variables & all that which is used internally by the class pushes it over the edge.

ryan: that patch is basically what i was thinking of myself, Wasnt sure 256 was needed, but setting it too high couldnt cause issues could it?

#4 @ryan
16 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [9612]) Increase memory limit when unzipping. fixes #8149

#5 @ryan
16 years ago

I chose 256 since that is what we're using in image.php and it seems to be working okay.

Note: See TracTickets for help on using tickets.