Last modified: 2014-05-10 16:25:11 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T32550, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 30550 - $wgMaxShellMemory too low for stock ImageMagick on RHEL / CentOS 6
$wgMaxShellMemory too low for stock ImageMagick on RHEL / CentOS 6
Status: RESOLVED WONTFIX
Product: MediaWiki
Classification: Unclassified
File management (Other open bugs)
unspecified
All Linux
: Low normal (vote)
: ---
Assigned To: Nobody - You can work on this!
:
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-08-24 17:19 UTC by Zhen Lin
Modified: 2014-05-10 16:25 UTC (History)
4 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments

Description Zhen Lin 2011-08-24 17:19:19 UTC
The stock ImageMagick shipped with RHEL / CentOS 6 is compiled with OpenMP, and this seems to vastly increase its (virtual) memory usage. The default setting for $wgMaxShellMemory, 100MB, is not even enough for convert to start up. Raising it to 1024MB only allows it to work on small images—convert segfaults on a 1,485 × 2,070 pixel, 1.79 MB JPEG. Given that this only limits *virtual* memory usage, and that this is (or, at least, will be) a common stock setup, perhaps it is worth considering changing the default value of $wgMaxShellMemory to 'unlimited', or otherwise changing what it means.
Comment 1 Max Semenik 2011-08-24 17:52:46 UTC
(In reply to comment #0)

> changing the default value of $wgMaxShellMemory
> to 'unlimited', or otherwise changing what it means.

That would create a great DoS attack vector: upload a big image, rescale it continuously to slightly different sizes, bang! By the way, RHEL and alikes are far from being recommended distros for MW due to retardedly old packages.

If ImageMagick is _that_ broken, simply don't use it. We may want to make $wgMaxShellMemory for installer limited to some sane value to detect broken IM by making it fail.
Comment 2 Mark A. Hershberger 2011-08-25 16:31:40 UTC
Ping Chad to get his opinion if this is something that Installer should do.
Comment 3 Chad H. 2011-08-25 16:35:37 UTC
No, I don't think the installer should handle this...but that's just my opinion.
Comment 4 Jesús Martínez Novo (Ciencia Al Poder) 2014-05-10 13:14:32 UTC
$wgMaxShellMemory is now 300MB. Giving that it's already a configurable setting, does this bug really need to remain open?
Comment 5 Max Semenik 2014-05-10 16:25:11 UTC
(In reply to Jesús Martínez Novo (Ciencia Al Poder) from comment #4)
Amen. Especially since packagers for these OSes can override this setting in installer.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links