Please review this PR which cleans up security manager related code in 
`java.util.zip` and `java.util.jar`:

* `JarFile` and `ZipFile` are updated to use `System::getProperty` instead of 
`GetPropertyAction::privilegedGetProperty`
*  `ZipFile` is updated to not call SM::checkRead, SM::checkDelete when opening 
files
* `ZipOutputStream` is updated to use `Boolean::getBoolean` instead of 
`GetBooleanAction::privilegedGetProperty`

The field `ZipFile.startsWithLoc` is deliberately left alone, that should be 
handled separately.

Testing: This is a cleanup PR, no tests are changed or updated. ZIP and JAR 
tests run green locally. GHA results pending.

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

Commit messages:
 - Cleanup SM-dependent code in ZIP and JAR areas

Changes: https://git.openjdk.org/jdk/pull/22099/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=22099&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8344179
  Stats: 17 lines in 3 files changed: 0 ins; 12 del; 5 mod
  Patch: https://git.openjdk.org/jdk/pull/22099.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22099/head:pull/22099

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

Reply via email to