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