RFR: 8344011: Remove usage of security manager from Class and reflective APIs
Roger Riggs
rriggs at openjdk.org
Wed Nov 13 17:39:59 UTC 2024
On Wed, 13 Nov 2024 10:32:34 GMT, Alan Bateman <alanb at openjdk.org> wrote:
> 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 temporarily excluded as the loading of a dynamically generated class at archive time. I'm checking with Calvin and Ioi on whether this test should be updated or replaced.
>
> Testing: tier1-5
lgtm
-------------
Marked as reviewed by rriggs (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/22063#pullrequestreview-2433993768
More information about the core-libs-dev
mailing list