Integrated: 8344183: (zipfs) SecurityManager cleanup in the ZipFS area

Eirik Bjørsnøs eirbjo at openjdk.org
Fri Nov 15 07:51:52 UTC 2024


On Thu, 14 Nov 2024 11:21:54 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: bfee766f
Author:    Eirik Bjørsnøs <eirbjo at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/bfee766f035fb1b122cd3f3703b9e2a2d85abfe6
Stats:     85 lines in 3 files changed: 3 ins; 57 del; 25 mod

8344183: (zipfs) SecurityManager cleanup in the ZipFS area

Reviewed-by: mullan, lancea

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

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


More information about the core-libs-dev mailing list