RFR: JDK-6725221 Standardize obtaining boolean properties with defaults
Roger Riggs
rriggs at openjdk.java.net
Mon May 23 18:37:51 UTC 2022
On Tue, 10 May 2022 19:24:24 GMT, Mark Powers <duke at openjdk.java.net> wrote:
>> src/java.base/share/classes/java/lang/reflect/AccessibleObject.java line 777:
>>
>>> 775: if (!printStackPropertiesSet && VM.initLevel() >= 1) {
>>> 776: printStackWhenAccessFails = GetBooleanAction.
>>> 777: privilegedGetProperty("sun.reflect.debugModuleAccessChecks");
>>
>> Running with `-Dsun.reflect.debugModuleAccessChecks` should set printStackWhenAccessFails to true, not false.
>
> You are right. The old way maps the null string to true, and the new way maps it to false. I did not notice that. At this point, I see no value in making the "true".equals and "false".equals changes. Too much can break. I'll reverse these changes.
This change still needs to be reversed.
-------------
PR: https://git.openjdk.java.net/jdk/pull/8559
More information about the security-dev
mailing list