[jdk17u-dev] RFR: 8339810: Clean up the code in sun.tools.jar.Main to properly close resources and use ZipFile during extract

Martin Balao mbalao at openjdk.org
Sat Mar 22 05:21:30 UTC 2025


Hi,

I'd like to have a review for the 17u backport of [8339810](https://bugs.openjdk.org/browse/JDK-8339810).

Changes to the 21u patch:

 * src/jdk.jartool/share/classes/sun/tools/jar/Main.java
   * Copyright date. Manually fixed.
   * 17u does not have 8258117 so the context is different and changes do not apply cleanly.

Testing:

 * No regressions found in jdk/sun/security/tools/jarsigner.

Thanks,
Martin.-

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

Commit messages:
 - Backport 8fce5275fc94ebc404a6a37f5ea0407140de63c1

Changes: https://git.openjdk.org/jdk17u-dev/pull/3399/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk17u-dev&pr=3399&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8339810
  Stats: 181 lines in 1 file changed: 32 ins; 53 del; 96 mod
  Patch: https://git.openjdk.org/jdk17u-dev/pull/3399.diff
  Fetch: git fetch https://git.openjdk.org/jdk17u-dev.git pull/3399/head:pull/3399

PR: https://git.openjdk.org/jdk17u-dev/pull/3399


More information about the jdk-updates-dev mailing list