On Fri, 13 Nov 2026 16:41:38 GMT, Lance Andersen <[email protected]> wrote:
>> Please review this PR which moves the CEN Header size validation in >> ZipOutputStream::writeCEN after the extended timestamp calculation which can >> result in the extra field length and extra field data being modified. >> >> Mach5 tier1 - tier3 ran cleanly >> >> --------- >> - [X] I confirm that I make this contribution in accordance with the >> [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). > > Lance Andersen has updated the pull request incrementally with one additional > commit since the last revision: > > Add suggested null check Good, it means the timestamp extra-field size is taken into account. I only skimmed the test, seems reasonable. ------------- Marked as reviewed by alanb (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/33801#pullrequestreview-5895071638
