[wp-trac] [WordPress Trac] #14889: Memory Comparison Broken / WordPress Memory Limit

[WordPress Trac]

#14889: Memory Comparison Broken / WordPress Memory Limit
-----------------------------------------------+---------------------------
 Reporter:  hakre                              |       Owner:
     Type:  defect (bug)                       |      Status:  new
 Priority:  normal                             |   Milestone:  Future
Component:  General                            |  Release
 Severity:  normal                             |     Version:  3.0
 Keywords:  has-patch 3.2-early needs-testing  |  Resolution:
-----------------------------------------------+---------------------------

Comment (by hakre):

 The suggested patch does not fix the issue with comparing the values. You
 can not compare integer values because of the shorthand notation. Just
 noting not that this ticket gets closed w/o fixing the issue.

 Additionally setting WP_MEMORY_LIMIT to -1 (unlimited) will not work with
 [attachment:14889.diff].

 However regarding the 32bit integer overflow I've found out that it
 actually looks not to be a problem as even the core ini values break with
 that limit. So it's actually somewhat okay to cast to int.
 [http://www.php.net/manual/en/ini.core.php#ini.post-max-size See note
 here]. But I have not tested for that so it remains a bit ambiguous.

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