On Mon, 18 Nov 2024 18:05:12 GMT, Aleksei Efimov <[email protected]> wrote:
>> src/java.rmi/share/classes/sun/rmi/server/MarshalInputStream.java line 69: >> >>> 67: >>> 68: /** table to hold sun classes to which access is explicitly >>> permitted */ >>> 69: protected static Map<String, Class<?>> permittedSunClasses >> >> It looks like `permittedSunClasses` map can be removed alongside to >> `checkSunClass(String, AccessControlException)` method, since it is only >> called on [line >> 198](https://github.com/openjdk/jdk/pull/22129/files#diff-491ee160d740152081c000fc513091e00181c41fd3c423c466fd30448a2a47e6R198): >> >> } catch (@SuppressWarnings("removal") AccessControlException e) { >> return checkSunClass(className, e); > > Sorry, missed your comment in a PR description - ignore the comment above Right, this will be covered by [JDK-8344329](https://bugs.openjdk.org/browse/JDK-8344329). ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/22129#discussion_r1847311299
