> Please review this PR which removes code depending on SecurityManager from > the `java.lang.module` and `jdk.internal.module` packages: > > * `ModuleFinder::ofSystem` is updated to not check the > `"accessSystemModules"` RuntimePermission > * The `ModuleReferences.ExplodedModuleReader` constructor is updated to > remove the check that the caller has access to the directory. > * `Modules::addProvides` is updated to not use > `AccessController::doPrivileged` when getting the module class loader > * `SystemModuleFinder::ofSystem` is updated to not returned a wrapped result > calling `AccessController::doPrivileged` > * `SystemModuleFinders.SystemImage` is updated to remove a class level > comment referring to the SM > * `SystemModuleFinders.SystemModuleReader::checkPermissionToConnect` is > removed. The URI constructor parameter in `SystemModuleReader` becomes unused > and is removed. > > > Testing: I have run `test/jdk/java/lang/module`, > `test/jdk/java/lang/ModuleTests`, and `test/jdk/java/lang/ModuleLayer` > successfully. GHA results pending.
Eirik Bjørsnøs has updated the pull request incrementally with two additional commits since the last revision: - Remove empty line in addProvides - Add full stop after class description for SystemImage ------------- Changes: - all: https://git.openjdk.org/jdk/pull/22113/files - new: https://git.openjdk.org/jdk/pull/22113/files/2fcec8fc..7d53f122 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=22113&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22113&range=00-01 Stats: 2 lines in 2 files changed: 0 ins; 1 del; 1 mod Patch: https://git.openjdk.org/jdk/pull/22113.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/22113/head:pull/22113 PR: https://git.openjdk.org/jdk/pull/22113
