On Fri, 15 Nov 2024 16:15:02 GMT, Chen Liang <li...@openjdk.org> wrote:
>> src/java.base/share/classes/jdk/internal/util/ClassFileDumper.java line 82:
>>
>>> 80:
>>> 81: private ClassFileDumper(String key, String path) {
>>> 82: /*
>>
>> The comment might still be relevant if changed to System.getProperty.
>
> It's fine now: previously user defined security manager was effectively
> loading user code that uses java.lang.invoke, now users cannot specify
> security managers, so this problem does not exist any more.
In that case, the code should be calling System.getProperty and not the backup
VM saved props.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/22141#discussion_r1844128182
