[BUGFIX] Properly escape the ImageMagick frame selector 62/6662/7
authorGeorg Ringer <georg.ringer@gmail.com>
Wed, 18 Dec 2013 13:15:27 +0000 (14:15 +0100)
committerStefan Neufeind <typo3.neufeind@speedpartner.de>
Thu, 9 Jan 2014 01:49:46 +0000 (02:49 +0100)
commitc5fcc5fd798f418c03cfeb2846f4f8260b844f14
treea306d5b364e6665089e8112a6306018b49909d67
parent6c4c4d9593d47de6f31fbed541492cde8586b12a
[BUGFIX] Properly escape the ImageMagick frame selector

The frame selector looks like a glob pattern and gets interpreted as
such by the shell. Currently the generated shell argument is, for
example, 'image.png'[0] which matches a file called image.png0.
Thus, if such a file exists, the wrong argument would be passed to
ImageMagick.

Only if there is no filename that matches the pattern, the string is
correctly passed to ImageMagick (i.e. as-is). But even in that case
there is a performance penalty, because the shell has to scan the whole
directory to check if the file exists. This becomes especially bad if
you're not dealing with a local file system.

By properly escaping the frame selector we stop the shell from
interpreting the glob pattern and both problems get fixed.

Resolves: #31797
Releases: 6.2, 6.1
Change-Id: Ib6dc6556bb9f1d64a0154b0cbe9e253b185c74c2
Reviewed-on: https://review.typo3.org/6662
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php