[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