oss-sec mailing list archives

Re: ImageMagick Is On Fire -- CVE-2016-3714


From: Tim <tim-security () sentinelchicken org>
Date: Tue, 3 May 2016 16:51:10 -0700


Or, replace the strings with arrays and use execve() instead of system().
 
^^^

That.

system() should be taken out into the street and shot.  There's just
no good reason for a respectable programmer to use it.

Not saying that's the *only* thing they would need to do, but we need
to encourage development platforms, in general, to stop offering up
awful interfaces like this.  Heck, Node.js offers a child_process.exec()
call that isn't exec at all.  It is (approximately) system().  Surely
that won't lead to any problems...

tim


Current thread: