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