Make WordPress Core

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#8149 closed defect (bug) (fixed)

Automatic upgrade runs out of memory

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


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 7 years ago.

Download all attachments as: .zip

Change History (6)

7 years ago

#1 @ryan
7 years ago

Sounds good.

#2 @westi
7 years ago

  • Keywords dev-reviewed added


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
7 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
7 years ago

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

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

#5 @ryan
7 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.