[TYPO3-core] RFC #16656 : bug : ImageMagick does not work with quotes in exec() path on Windows

[Jigal van Hemert]

Hi,

On 17-12-2010 18:18, Jeff Segars wrote:
> I guess technically this bug could occur anywhere that exec() is used
> but its just the case that we only see it in image processing right now?

It would happen with any exec(), system(), etc. call if the three 
conditions are met:
- OS is Windows
- more than two double quotes
- PHP < 5.3

> Adding a wrapper for exec() would fix it everywhere (in the theoretical
> cases we haven't seen), but would expand the scope of the patch so I'm
> not sure if its worth it or not. What do you think?

In a lot of cases exec() is used for 'cp', 'mv', etc. commands. What 
worries me more is that it happens only with PHP < 5.3. Last weeks 
release of 5.2 was the last in the 5.2 range, so if 4.6 will have a 
minimum of PHP 5.3 we should remove the wrapper again.

Will you give your vote for a global wrapper? :-)

-- 
Kind regards / met vriendelijke groet,

Jigal van Hemert
skype:jigal.van.hemert
msn: jigal at xs4all.nl
http://twitter.com/jigalvh