[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.GSO.2.20.1605032020420.23612@freddy.simplesystems.org>
Date: Tue, 3 May 2016 20:42:30 -0500 (CDT)
From: Bob Friesenhahn <bfriesen@...ple.dallas.tx.us>
To: oss-security@...ts.openwall.com
Subject: Re: ImageMagick Is On Fire -- CVE-2016-3714
On Tue, 3 May 2016, Seth Arnold wrote:
> On Wed, May 04, 2016 at 12:05:16AM +0000, Brandon Dees wrote:
>> is it appropriate to ask if the same issues are present in GraphicsMagick
>> as well?
>
> I haven't investigated deeply but it seems very plausible to me:
> Here's the delegates.xml work-alike:
> https://sourceforge.net/p/graphicsmagick/code/ci/default/tree/config/delegates.mgk.in
>
> This appears to be executed via:
> https://sourceforge.net/p/graphicsmagick/code/ci/default/tree/magick/delegate.c
> which tries to escape arguments using UnixShellTextEscape(). This function
> appears to replace \`"$ chars with backslash-escaped versions. I'm not
> sure this is a safe mechanism either.
Please provide me with a working exploit.
Be aware that this quoting method is only used for the few
delegates.mgk rules which require shell-like syntax to work.
Otherwise the external program is run using execvp() without a shell.
I am aware that the handling for Microsoft Windows is not quite secure
and in fact Windows concatentates all the spawnvp() vector arguments
into one long string and each program parses command line arguments
using its own algorithm without a secure quoting mechanism so
command-line programs can never possibly be secured.
In order to achieve the best security with GraphicsMagick (with some
possible loss of function due to missing file formats), please define
this environment variable:
MAGICK_CODER_STABILITY=PRIMARY
Use 'gm convert -list formats' and check the second column of output
to see what formats are classified as Primary, Stable, and Unstable.
Primary formats are considered common and trustworthy.
There is also a way that C/C++ programs using the libraries can bless
the files which will be accessed before the access occurs (not yet
controlled by a configuration file).
Thanks,
Bob
--
Bob Friesenhahn
bfriesen@...ple.dallas.tx.us, http://www.simplesystems.org/users/bfriesen/
GraphicsMagick Maintainer, http://www.GraphicsMagick.org/
Powered by blists - more mailing lists
Please check out the
Open Source Software Security Wiki, which is counterpart to this
mailing list.
Confused about mailing lists and their use?
Read about mailing lists on Wikipedia
and check out these
guidelines on proper formatting of your messages.
