RFR: 8282577: ICC_Profile.setData(int, byte[]) invalidates the profile [v3]

[Sergey Bylokhov]

On Thu, 3 Mar 2022 21:27:36 GMT, Phil Race <prr at openjdk.org> wrote:

>> As per the bug eval, we are returning a pointer to a native raw LittleCMS profile and it should be a cooked one.
>> The simple test fails before the fix with
>> ~/jdk17/Contents/Home/bin/java SetTagDataValidation                               
>> Exception in thread "main" java.awt.color.CMMException: LCMS error 13: Couldn't link the profiles
>> 	at java.desktop/sun.java2d.cmm.lcms.LCMS.createNativeTransform(Native Method)
>> 	at java.desktop/sun.java2d.cmm.lcms.LCMS.createTransform(LCMS.java:104)
>> 	at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.doTransform(LCMSTransform.java:143)
>> 	at java.desktop/sun.java2d.cmm.lcms.LCMSTransform.colorConvert(LCMSTransform.java:633)
>> 	at java.desktop/java.awt.color.ICC_ColorSpace.toRGB(ICC_ColorSpace.java:224)
>> 	at SetTagDataValidation.main(SetTagDataValidation.java:61)
>> 
>> and with the fix  just prints 
>> java SetTagDataValidation
>> 0.4
>> 0.5000076
>> 0.6
>
> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8282577: ICC_Profile.setData(int, byte[]) invalidates the profile

The current bug looks similar to this one https://bugs.openjdk.java.net/browse/JDK-8272865 I tried to fix it the same way but I was not sure if it is safe to ignore the ReadWriteProfileTest.java or not, since it has five related bugids.

test/jdk/java/awt/color/ICC_Profile/MTGetData.java line 106:

> 104:                     icc.setData(tag, data2);
> 105:                 }
> 106:             } catch (CMMException cmmEx) {

Don't we need to throw the IllegalArgumentException instead of CMMException?

-------------

PR: https://git.openjdk.java.net/jdk/pull/7668