On Thu, 25 Sep 2025 03:12:17 GMT, Sergey Bylokhov <[email protected]> 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
