RFR: 8345565: Remove remaining SecurityManager motivated APIs from sun.reflect.util [v2]
Alan Bateman
alanb at openjdk.org
Thu Dec 5 15:24:43 UTC 2024
On Thu, 5 Dec 2024 15:16:46 GMT, Chen Liang <liach at openjdk.org> wrote:
>> I had hoped to keep the change focused on the removal of the left over ReflectUtil.checkPackageAccess, the else throw AssertionError was a surprise to see. So yes, I agree it can be removed but I'd prefer not touch it now, future work in this area can remove it.
>
> Indeed, we should clean this up with things like MethodUtil.
MethodUtil is for APIs that do reflective stuff on behave of callers so not tied to the SM execution mode. That is for another time.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22572#discussion_r1871580656
More information about the serviceability-dev
mailing list