RFR: 8344179: SecurityManager cleanup in the ZIP and JAR areas
Eirik Bjørsnøs
eirbjo at openjdk.org
Thu Nov 14 11:29:06 UTC 2024
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
More information about the core-libs-dev
mailing list