[wp-trac] [WordPress Trac] #13847: WP increasing the memory above other server limit

WordPress Trac wp-trac at lists.automattic.com
Mon Aug 23 12:54:45 UTC 2010


#13847: WP increasing the memory above other server limit
----------------------------+-----------------------------------------------
 Reporter:  priitsalumaa    |       Owner:  westi          
     Type:  defect (bug)    |      Status:  accepted       
 Priority:  normal          |   Milestone:  Awaiting Triage
Component:  Administration  |     Version:  3.0            
 Severity:  normal          |    Keywords:  has-patch      
----------------------------+-----------------------------------------------

Comment(by hakre):

 Replying to [comment:15 Denis-de-Bernardy]:
 > Isn't there a suhosin ini setting that allows to pull the max memory
 limit directly? I suspect the current patch ought to be changed in such a
 way that, when present, it gets used instead of a flat 256MB, along with a
 warning to the end user in a way or another where it's useful (i.e.
 upgrades and file uploads).

 Sure, my first patch is ''only'' adding a constant instead of using the
 magic numbers. So it address only the furthermost problem that stands in
 the way to enable a user to deal with the problem w/o modifying core-code.

 The standard PHP Memory Setting can be read by using the {{{ini_get()}}}
 function (I assume that by proxy this will be gathering the same value
 suhosin is checking against) and if that set to the maximum available
 memory would certainly solve the problem but if so, there would be no need
 at all to use {{{ini_set()}}} all over the place because it's then assumed
 that the server has been already configured.

 But wordpress is assuming that the server is not properly configured and
 therefore assumes it's the better admin by just setting this up high.

 As you can see it's not trivial to find the right solution so I started by
 removing that flaw that is solveable right away: reducing the magic
 numbers to a single constant.

 ----

 Related: #14425

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/13847#comment:16>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list