RFR: 8344183: (zipfs) SecurityManager cleanup in the ZipFS area [v2]
Eirik Bjørsnøs
eirbjo at openjdk.org
Thu Nov 14 14:43:50 UTC 2024
> Please review this PR which cleans up SecurityManager-related code following JEP-486 integraion.
>
> * `ZipFileSystem` is updated to not perform `AccessController::doPrivileged`
> * `ZipFileSystemProvider` is updated to not perform `AccessController::doPrivileged`
> * The test `TestPosix` is updated to perform `AccessController.doPrivileged`
>
> This change should be relatively straight-forward to review. Reviewers may want to look extra close at the exception-unwrapping code in `ZipFileSystem::initOwner` and `ZipFileSystem::initGroup`.
>
> Testing: Tests in `test/jdk/jdk/nio/zipfs` run green locally.
Eirik Bjørsnøs has updated the pull request incrementally with one additional commit since the last revision:
Update debug logging to not reference privileged operations
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/22101/files
- new: https://git.openjdk.org/jdk/pull/22101/files/d8936a5c..37104a91
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=22101&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=22101&range=00-01
Stats: 2 lines in 1 file changed: 0 ins; 0 del; 2 mod
Patch: https://git.openjdk.org/jdk/pull/22101.diff
Fetch: git fetch https://git.openjdk.org/jdk.git pull/22101/head:pull/22101
PR: https://git.openjdk.org/jdk/pull/22101
More information about the core-libs-dev
mailing list