One of the LPROF developers ran into a number of profiles that are having a problem in cmsio1.c with the values that are being returned a call to AdjustEndianness32 in cmsReadICCText were it is getting the length of the text field from the profile header. The value before AdjustEndianness32 is called is 18 and after it is called it is 301989888 which is clearly wrong. I suspect that the profiles in question have the values stored in little endian format and I think that this is non-complant with the ICC spec. Is this correct?
Also how should this be handled to prevent fatal errors? At the very least I would like to be able to ignore profiles that are not ICC compliant. Hal ---------- Forwarded Message ---------- Subject: RE: [Lprof-devel] lprof/src/lprofqt lprofmain.cpp,1.50,1.51 Date: Monday 06 March 2006 08:06 pm From: "Joe Pizzi" <[EMAIL PROTECTED]> To: "'Hal Engel'" <[EMAIL PROTECTED]>, [EMAIL PROTECTED] I got the Windows version running again (thanks, Gerhard; your fixes worked) on MSVC. I got to debugging it. When we are in LoadConfig, in ICCProfileSelector::FillValues, we are apparently reading the ICC profiles installed on the machine. Inside of this, there is a place in ReadEmbeddedTextTag where, when the BaseType is icSigTextDescriptionType, that we read AsciiCount, then AdjustEndianess32. The value before the adjust endianness is 18, after it is 301989888. I believe that we are byte-swapping when we shouldn't be. I don't know the ICC specification, however. Is it the case that the file is defined to have BigEndian data in it, and the ICC profile that I have from the hardware vendor is not compliant, or are we adjusting endianness when we shouldn't be? Anybody know? Joe Pizzi ------------------------------------------------------- This SF.Net email is sponsored by xPML, a groundbreaking scripting language that extends applications into web and mobile media. Attend the live webcast and join the prime developer group breaking into this new coding territory! http://sel.as-us.falkag.net/sel?cmd=lnk&kid=110944&bid=241720&dat=121642 _______________________________________________ Lcms-user mailing list Lcms-user@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/lcms-user
