RFR: 8282577: ICC_Profile.setData(int, byte[]) invalidates the profile [v4]
Phil Race
prr at openjdk.java.net
Thu Mar 10 04:15:14 UTC 2022
> 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 two additional commits since the last revision:
- 8282577: ICC_Profile.setData(int, byte[]) invalidates the profile
- 8282577: ICC_Profile.setData(int, byte[]) invalidates the profile
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/7668/files
- new: https://git.openjdk.java.net/jdk/pull/7668/files/5bf31b11..29c836ac
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=7668&range=03
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=7668&range=02-03
Stats: 25 lines in 3 files changed: 11 ins; 9 del; 5 mod
Patch: https://git.openjdk.java.net/jdk/pull/7668.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/7668/head:pull/7668
PR: https://git.openjdk.java.net/jdk/pull/7668
More information about the client-libs-dev
mailing list