On Mon, 18 Nov 2024 17:39:44 GMT, Aleksei Efimov <aefi...@openjdk.org> wrote:
>> First cut at removal of Security Manager stuff from RMI. >> >> This covers just about every SM-related case in RMI, except for a bit of >> package checking in MarshalInputStream. This will be handled separately. >> It's covered by [JDK-8344329](https://bugs.openjdk.org/browse/JDK-8344329). >> >> Further simplifications could be done in RuntimeUtil and NewThreadAction. >> However, those changes started to become somewhat more intrusive than I'd >> like for this PR, which is focused on removing security-related stuff. > > src/java.rmi/share/classes/sun/rmi/runtime/NewThreadAction.java line 65: > >> 63: /** >> 64: * special child of the system thread group for running tasks that >> 65: * may execute user code, so that the security policy for threads in > > Do we still want to mention `security policy` here? Yeah, I should probably edit that. I don't want to change the thread group behavior, so the justification for the thread group might mention the (historical) effect on security policy. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22129#discussion_r1847310716
