RFR: 8344011: Remove usage of security manager from Class and reflective APIs [v2]

Alan Bateman alanb at openjdk.org
Wed Nov 13 19:05:21 UTC 2024


> Remove code required for the now defunct SecurityManager execution mode from java.lang.Class, friends, and reflection APIs. Careful review is required so I've set Reviewer to 2. I've tried to keep the changes as easy to review as possible and not go over board with cleanup.
> 
> sun.reflect.misc.ReflectUtil are been hollowed out. A future pass will remove empty methods and qualified exports once the changes in "far away" code and modules is done.
> 
> In Lookup's class description, the removal of the sentence "avoid package access checks for classes accessible to the lookup class"  and the link to the removed "Security manager interactions" section is in discussion/non-normative text, just missed in the JEP 486 update that remove the linked section.
> 
> runtime/cds/appcds/StaticArchiveWithLambda.java is updated as creating the archive no longer skips a generated class.
> 
> Testing: tier1-5

Alan Bateman has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:

 - StaticArchiveWithLambda no longer skips dynamically generated class when creating archive
 - Merge branch 'master' into JDK-8344011
 - Initial commit

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/22063/files
  - new: https://git.openjdk.org/jdk/pull/22063/files/b46c4fea..1687ad54

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=22063&range=01
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=22063&range=00-01

  Stats: 7342 lines in 85 files changed: 5659 ins; 433 del; 1250 mod
  Patch: https://git.openjdk.org/jdk/pull/22063.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/22063/head:pull/22063

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


More information about the core-libs-dev mailing list