Invalid memory access with XML profiles

"Alexander E. Patrakov" <[email protected]> Sat, 3 Jan 2009 23:30:39 +0500
Newsgroups gmane.comp.video.image-magick.devel
Message-ID <[email protected]>
Hello.

In GetXMPProperty(), we have:

  profile=GetImageProfile(image,"xmp");
  // snip irrelevant code
  p=(const char *) GetStringInfoDatum(profile);
  // snip code that sometimes advances p somewhat, but doesn't modify
anything else
  xmp=NewXMLTree((char *) p,exception);

i.e., here p may be not 0-terminated. However, NewXmlTree() calls
strlen() on its argument. This is a bug.

No patch so far, because I should not make patches at midnight.

-- 
Alexander E. Patrakov