jmx-dev RFR: 8344394: Remove SecurityManager and related calls from java.management.rmi

Daniel Fuchs dfuchs at openjdk.org
Fri Nov 22 16:22:17 UTC 2024


On Wed, 20 Nov 2024 10:14:40 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

> Remove redundant SecurityManager, AccessController references
> (following on from JDK-8338411: Implement JEP 486: Permanently Disable the Security Manager).

src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java line 32:

> 30: import java.rmi.UnmarshalException;
> 31: import java.rmi.server.Unreferenced;
> 32: import java.security.PrivilegedActionException;

should this be removed too?

src/java.management.rmi/share/classes/javax/management/remote/rmi/RMIConnectionImpl.java line 1286:

> 1284: 
> 1285:     /** @throws UnsupportedOperationException {@inheritDoc} */
> 1286:     private Object doPrivilegedOperation(final int operation,

Maybe rename "doPrivilegedOperation" into an overload of "doOperation"

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22270#discussion_r1854188609
PR Review Comment: https://git.openjdk.org/jdk/pull/22270#discussion_r1854196303


More information about the jmx-dev mailing list