{"id":2072,"date":"2017-09-06T15:42:04","date_gmt":"2017-09-06T13:42:04","guid":{"rendered":"http:\/\/blogs.gentoo.org\/ago\/?p=2072"},"modified":"2017-09-06T20:52:12","modified_gmt":"2017-09-06T18:52:12","slug":"graphicsmagick-memory-allocation-failure-in-magickmalloc-memory-c-2","status":"publish","type":"post","link":"https:\/\/blogs.gentoo.org\/ago\/2017\/09\/06\/graphicsmagick-memory-allocation-failure-in-magickmalloc-memory-c-2\/","title":{"rendered":"graphicsmagick: memory allocation failure in MagickMalloc (memory.c)"},"content":{"rendered":"<p><strong>Description<\/strong>:<br \/>\n<a href=\"http:\/\/www.graphicsmagick.org\/\">graphicsmagick<\/a> is a collection of tools and libraries for many image formats.<\/p>\n<p>The relevant ASan output of the issue:<\/p>\n<pre><font size=\"2\"># gm convert -negate -clip $file out\r\n==25373==End of process memory map.\r\n==25373==AddressSanitizer CHECK failed: \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/sanitizer_common\/sanitizer_common.cc:120 \"((0 &amp;&amp; \"unable to mmap\")) != (0)\" (0x0, 0x0)\r\n    #0 0x4d966f in AsanCheckFailed \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/asan\/asan_rtl.cc:69\r\n    #1 0x4f43d5 in __sanitizer::CheckFailed(char const*, int, char const*, unsigned long long, unsigned long long) \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/sanitizer_common\/sanitizer_termination.cc:79\r\n    #2 0x4e3a02 in __sanitizer::ReportMmapFailureAndDie(unsigned long, char const*, char const*, int, bool) \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/sanitizer_common\/sanitizer_common.cc:120\r\n    #3 0x4ed305 in __sanitizer::MmapOrDie(unsigned long, char const*, bool) \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/sanitizer_common\/sanitizer_posix.cc:132\r\n    #4 0x420a02 in __sanitizer::LargeMmapAllocator::Allocate(__sanitizer::AllocatorStats*, unsigned long, unsigned long) \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/asan\/..\/sanitizer_common\/sanitizer_allocator_secondary.h:41\r\n    #5 0x420a02 in __sanitizer::CombinedAllocator&lt;__sanitizer::SizeClassAllocator64, __sanitizer::SizeClassAllocatorLocalCache&lt;__sanitizer::SizeClassAllocator64 &gt;, __sanitizer::LargeMmapAllocator &gt;::Allocate(__sanitizer::SizeClassAllocatorLocalCache&lt;__sanitizer::SizeClassAllocator64 &gt;*, unsigned long, unsigned long, bool, bool) \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/asan\/..\/sanitizer_common\/sanitizer_allocator_combined.h:70\r\n    #6 0x420a02 in __asan::Allocator::Allocate(unsigned long, unsigned long, __sanitizer::BufferedStackTrace*, __asan::AllocType, bool) \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/asan\/asan_allocator.cc:407\r\n    #7 0x420a02 in __asan::asan_malloc(unsigned long, __sanitizer::BufferedStackTrace*) \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/asan\/asan_allocator.cc:782\r\n    #8 0x4cf664 in malloc \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/asan\/asan_malloc_linux.cc:67\r\n    #9 0x7fc323f784d6 in MagickMalloc \/var\/tmp\/portage\/media-gfx\/graphicsmagick-1.3.26\/work\/GraphicsMagick-1.3.26\/magick\/memory.c:156:10\r\n    #10 0x7fc31e2b41a3 in ReadSUNImage \/var\/tmp\/portage\/media-gfx\/graphicsmagick-1.3.26\/work\/GraphicsMagick-1.3.26\/coders\/sun.c:549:20\r\n    #11 0x7fc323b96e88 in ReadImage \/var\/tmp\/portage\/media-gfx\/graphicsmagick-1.3.26\/work\/GraphicsMagick-1.3.26\/magick\/constitute.c:1607:13\r\n    #12 0x7fc323a29f18 in ConvertImageCommand \/var\/tmp\/portage\/media-gfx\/graphicsmagick-1.3.26\/work\/GraphicsMagick-1.3.26\/magick\/command.c:4348:22\r\n    #13 0x7fc323a660c5 in MagickCommand \/var\/tmp\/portage\/media-gfx\/graphicsmagick-1.3.26\/work\/GraphicsMagick-1.3.26\/magick\/command.c:8869:17\r\n    #14 0x7fc323b1185b in GMCommandSingle \/var\/tmp\/portage\/media-gfx\/graphicsmagick-1.3.26\/work\/GraphicsMagick-1.3.26\/magick\/command.c:17396:10\r\n    #15 0x7fc323b0e991 in GMCommand \/var\/tmp\/portage\/media-gfx\/graphicsmagick-1.3.26\/work\/GraphicsMagick-1.3.26\/magick\/command.c:17449:16\r\n    #16 0x7fc322379680 in __libc_start_main \/var\/tmp\/portage\/sys-libs\/glibc-2.23-r4\/work\/glibc-2.23\/csu\/..\/csu\/libc-start.c:289\r\n    #17 0x419cd8 in _init (\/usr\/bin\/gm+0x419cd8)\r\n\r\n\/usr\/bin\/gm convert: abort due to signal 6 (SIGABRT) \"Abort\"...\r\n<\/font><\/pre>\n<p><strong>Affected version:<\/strong><br \/>\n1.3.26<\/p>\n<p><strong>Fixed version:<\/strong><br \/>\nN\/A<\/p>\n<p><strong>Commit fix:<\/strong><br \/>\n<a href=\"http:\/\/hg.code.sf.net\/p\/graphicsmagick\/code\/rev\/493da54370aa\">http:\/\/hg.code.sf.net\/p\/graphicsmagick\/code\/rev\/493da54370aa<\/a><\/p>\n<p><strong>Credit:<\/strong><br \/>\nThis bug was discovered by Agostino Sarubbo of Gentoo.<\/p>\n<p><strong>CVE:<\/strong><br \/>\nCVE-2017-14165<\/p>\n<p><strong>Reproducer:<\/strong><br \/>\n<a href=\"https:\/\/github.com\/asarubbo\/poc\/blob\/master\/00334-graphicsmagick-memallocfailure-MagickMalloc\">https:\/\/github.com\/asarubbo\/poc\/blob\/master\/00334-graphicsmagick-memallocfailure-MagickMalloc<\/a><\/p>\n<p><strong>Timeline:<\/strong><br \/>\n2017-07-14: bug discovered and reported to upstream privately<br \/>\n2017-08-16: bug reported to the public upstream bugtracker<br \/>\n2017-08-20: upstream released a fix<br \/>\n2017-09-06: blog post about the issue<br \/>\n2017-09-06: CVE assigned<\/p>\n<p><strong>Note:<\/strong><br \/>\nThis bug was found with <a href=\"http:\/\/lcamtuf.coredump.cx\/afl\">American Fuzzy Lop<\/a>.<br \/>\nThis bug was identified with bare metal servers donated by <a href=\"https:\/\/www.packet.net\/\">Packet<\/a>. This work is also supported by the <a href=\"https:\/\/www.coreinfrastructure.org\">Core Infrastructure Initiative<\/a>.<\/p>\n<p><strong>Permalink:<\/strong><\/p>\n<blockquote data-secret=\"3UVQS2Jt4c\" class=\"wp-embedded-content\"><p><a href=\"http:\/\/blogs.gentoo.org\/ago\/2017\/09\/06\/graphicsmagick-memory-allocation-failure-in-magickmalloc-memory-c-2\/\">graphicsmagick: memory allocation failure in MagickMalloc (memory.c)<\/a><\/p><\/blockquote>\n<p><iframe class=\"wp-embedded-content\" sandbox=\"allow-scripts\" security=\"restricted\" style=\"position: absolute; clip: rect(1px, 1px, 1px, 1px);\" src=\"http:\/\/blogs.gentoo.org\/ago\/2017\/09\/06\/graphicsmagick-memory-allocation-failure-in-magickmalloc-memory-c-2\/embed\/#?secret=3UVQS2Jt4c\" data-secret=\"3UVQS2Jt4c\" width=\"600\" height=\"338\" title=\"&#8220;graphicsmagick: memory allocation failure in MagickMalloc (memory.c)&#8221; &#8212; agostino&#039;s blog\" frameborder=\"0\" marginwidth=\"0\" marginheight=\"0\" scrolling=\"no\"><\/iframe><\/p>\n","protected":false},"excerpt":{"rendered":"<p>Description: graphicsmagick is a collection of tools and libraries for many image formats. The relevant ASan output of the issue: # gm convert -negate -clip $file out ==25373==End of process memory map. ==25373==AddressSanitizer CHECK failed: \/var\/tmp\/portage\/sys-libs\/compiler-rt-sanitizers-4.0.1\/work\/compiler-rt-4.0.1.src\/lib\/sanitizer_common\/sanitizer_common.cc:120 &#8220;((0 &amp;&amp; &#8220;unable to &hellip; <a href=\"https:\/\/blogs.gentoo.org\/ago\/2017\/09\/06\/graphicsmagick-memory-allocation-failure-in-magickmalloc-memory-c-2\/\">Continue reading <span class=\"meta-nav\">&rarr;<\/span><\/a><\/p>\n","protected":false},"author":140,"featured_media":0,"comment_status":"open","ping_status":"open","sticky":false,"template":"","format":"standard","meta":{"spay_email":"","jetpack_publicize_message":"","jetpack_is_tweetstorm":false,"jetpack_publicize_feature_enabled":true},"categories":[12,10],"tags":[],"jetpack_publicize_connections":[],"jetpack_featured_media_url":"","jetpack_sharing_enabled":true,"jetpack_shortlink":"https:\/\/wp.me\/p2EaBc-xq","_links":{"self":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts\/2072"}],"collection":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts"}],"about":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/types\/post"}],"author":[{"embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/users\/140"}],"replies":[{"embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/comments?post=2072"}],"version-history":[{"count":5,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts\/2072\/revisions"}],"predecessor-version":[{"id":2136,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/posts\/2072\/revisions\/2136"}],"wp:attachment":[{"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/media?parent=2072"}],"wp:term":[{"taxonomy":"category","embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/categories?post=2072"},{"taxonomy":"post_tag","embeddable":true,"href":"https:\/\/blogs.gentoo.org\/ago\/wp-json\/wp\/v2\/tags?post=2072"}],"curies":[{"name":"wp","href":"https:\/\/api.w.org\/{rel}","templated":true}]}}