RFR: 8344187: Remove SecurityManager and related calls from java.lang.instrument

[Alan Bateman]

On Thu, 14 Nov 2024 13:03:28 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

> Remove redundant SecurityManager, AccessController references
> (following on from JDK-8338411: Implement JEP 486: Permanently Disable the Security Manager).
> 
> test/jdk/java/lang/instrument/ still passing.

Marked as reviewed by alanb (Reviewer).

src/java.instrument/share/classes/sun/instrument/InstrumentationImpl.java line 661:

> 659:         static final StackWalker walker;
> 660:         static {
> 661:             walker = StackWalker.getInstance(StackWalker.Option.RETAIN_CLASS_REFERENCE);

If you want, you could remove the static block, but what you have is okay too.

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

PR Review: https://git.openjdk.org/jdk/pull/22106#pullrequestreview-2441725031
PR Review Comment: https://git.openjdk.org/jdk/pull/22106#discussion_r1846005325