RFR: 8354799: ZipInputStream doesn't verify CRC for ZIP entry with empty file data [v3]

Jaikiran Pai jpai at openjdk.org
Mon May 12 11:27:35 UTC 2025


> Can I please get a review of this change which addresses the issue noted in https://bugs.openjdk.org/browse/JDK-8354799?
> 
> `java.util.zip.ZipInputStream` when dealing with a `STORED` entry of zero size was missing a CRC check for that entry. The change in this PR addresses that and introduces a new jtreg test which reproduces the issue and verifies the fix.
> 
> tier testing is currently in progress.

Jaikiran Pai has updated the pull request incrementally with one additional commit since the last revision:

  Lance's suggestion - simplify argument creation, given just a handful number of argument values

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/25116/files
  - new: https://git.openjdk.org/jdk/pull/25116/files/0aea22fa..d1361736

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=25116&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=25116&range=01-02

  Stats: 9 lines in 1 file changed: 0 ins; 1 del; 8 mod
  Patch: https://git.openjdk.org/jdk/pull/25116.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/25116/head:pull/25116

PR: https://git.openjdk.org/jdk/pull/25116


More information about the core-libs-dev mailing list