RFR: 8367384: The ICC_Profile class may throw exceptions during serialization [v3]
Harshitha Onkar
honkar at openjdk.org
Fri Oct 3 17:30:50 UTC 2025
On Thu, 25 Sep 2025 03:12:17 GMT, Sergey Bylokhov <serb at openjdk.org> wrote:
>> Additional checks were recently added to ICC_Profile (see [JDK-8347377](https://bugs.openjdk.org/browse/JDK-8347377)). As a result, objects previously stored as valid profiles may now throw an IllegalArgumentException during serialization. Discussion about serialization was started in https://github.com/openjdk/jdk/pull/23044 but it seems at the end non-serialization related check was [verified](https://github.com/openjdk/jdk/pull/23044/commits/a5201b5f353e8957a1274261372496768edbc7a2). =(
>>
>> The patch itself is simple, but I found that we do not have good test coverage in this area. So I added two tests to cover all possible variants specified by the serialization spec.
>
> Sergey Bylokhov has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains five additional commits since the last revision:
>
> - Merge branch 'openjdk:master' into JDK-8367384
> - Add naming conventions used in test file names
> - sort catch
> - Update ValidateICCHeaderData.java
> - 8367384: The ICC_Profile class may throw exceptions during serialization
CI Testing for open PR + closed test changes looks good. I'll be integrating both PR soon.
-------------
PR Review: https://git.openjdk.org/jdk/pull/27326#pullrequestreview-3300091995
More information about the client-libs-dev
mailing list