Legacy ImageMagick Discussions Archive

Hello,

I just download the latest image magick package and I found an annoying bug in constitute.c / ReadImage
The function incorrectly set the image units as shown above.

The concerned lines are:

value=GetImageProperty(next,"TIFF:ResolutionUnit");
if (value == (char *) NULL)
value=GetImageProperty(next,"EXIF:YResolution"); // Aïe Aïe Aïe! Error this should be EXIF:ResolutionUnit!
if (value != (char *) NULL)
{
next->units=(ResolutionType) (atoi(value)-1);
(void) DeleteImageProperty(next,"EXIF:ResolutionUnit");
(void) DeleteImageProperty(next,"TIFF:ResolutionUnit");
}


This bad bug results in the lost of units for JPEG image for example.

I hope it will be corrected in a futur release.

Manuel

Thanks for the copy/paste problem your reported. We have a patch against ImageMagick 6.3.6-4 Beta that fixes the problem.