Page 1 of 1

Unit bug in constitute

Posted: 2007-10-26T01:09:50-07:00
by mootools

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:

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.


Re: Unit bug in constitute / ReadImage

Posted: 2007-10-26T05:29:52-07:00
by magick
Thanks for the copy/paste problem your reported. We have a patch against ImageMagick 6.3.6-4 Beta that fixes the problem.