[wp-trac] [WordPress Trac] #36534: Media Upload Issue Since 4.5 Upgrade

[WordPress Trac]

#36534: Media Upload Issue Since 4.5 Upgrade
---------------------------------------+-----------------------------
 Reporter:  unicornbacon               |       Owner:  joemcgill
     Type:  defect (bug)               |      Status:  reopened
 Priority:  normal                     |   Milestone:  Future Release
Component:  Media                      |     Version:
 Severity:  critical                   |  Resolution:
 Keywords:  needs-testing needs-patch  |     Focuses:
---------------------------------------+-----------------------------

Comment (by mrdanack):

 Hello. I'm the maintainer of Imagick. I got pointed to this bug by:
 https://bugs.php.net/bug.php?id=77683

 Setting the parameter via env setings is inherently unsafe in all multi-
 threaded environments.

 It should be possible to set this via code instead by doing:

 Imagick::setResourceLimit(\Imagick::RESOURCETYPE_THREAD, 1);

 Obviously that would need to be wrapped with the appropriate check to see
 if the extension exists, and if the function exists. Or possibly calling
 Imagick::setResourceLimit(6, 1); in case the version of Imagick doesn't
 have that resource set.


 For the record - OpenMP is meant to be used for CLI programs only. As far
 as I'm aware it's not designed at all to be running inside another multi-
 threaded environment like Apache.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/36534#comment:129>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform