RFR: 8367384: The ICC_Profile class may throw exceptions during serialization [v3]

Sergey Bylokhov serb at openjdk.org
Sat Sep 27 21:17:15 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

> As @prrace mentioned here [#27326 (comment)](https://github.com/openjdk/jdk/pull/27326#discussion_r2356533149), the closed tests need to be updated. Probably requires coordinated integration of open + closed PRs.

You can integrate this PR:
>Integration of this pull request has been delegated and may be completed by any project committer using the [/integrate](https://wiki.openjdk.org/display/SKARA/Pull+Request+Commands#PullRequestCommands-/integrate) pull request command.

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

PR Comment: https://git.openjdk.org/jdk/pull/27326#issuecomment-3342052891


More information about the client-libs-dev mailing list