lprof/src/lprofqt lprofmain.cpp,1.50,1.51

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" <joepizzi1scom.net>
To: "'Hal Engel'" <hvengelusers.sourceforge.net>, 
lprof-devellists.sourceforge.net

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-userlists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/lcms-user