Can I please get a review of this change which moves the `jdk.naming.rmi` 
module to be loaded by the platform loader, instead of the boot loader?

This module doesn't require any extra priveleges and as such isn't needed to be 
part of the boot loader. This module provides the 
`javax.naming.spi.InitialContextFactory` service. Before this change, the 
service would be located and loaded using the boot classloader and now will 
instead be loaded using the platform classloader. I believe this change to the 
service loading is an observable change and would thus require a CSR. I'll 
create one shortly.

A new jtreg test has been introduced to verify that the `InitialContextFactory` 
service provided by the `jdk.naming.rmi` module doesn't get loaded by the boot 
loader. The test passes with this change and fails (as expected) without the 
source change.

---------
- [x] I confirm that I make this contribution in accordance with the [OpenJDK 
Interim AI Policy](https://openjdk.org/legal/ai).

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

Commit messages:
 - 8154193: Deprivilege jdk.naming.rmi module

Changes: https://git.openjdk.org/jdk/pull/31480/files
  Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=31480&range=00
  Issue: https://bugs.openjdk.org/browse/JDK-8154193
  Stats: 80 lines in 2 files changed: 78 ins; 1 del; 1 mod
  Patch: https://git.openjdk.org/jdk/pull/31480.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/31480/head:pull/31480

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

Reply via email to