RFR: 8292177: InitialSecurityProperty JFR event [v3]
Sean Mullan
mullan at openjdk.org
Mon Oct 10 14:22:48 UTC 2022
On Mon, 10 Oct 2022 07:31:29 GMT, Alan Bateman <alanb at openjdk.org> wrote:
>> Sean Coffey has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Check for 0 security events
>
> src/java.base/share/classes/java/security/ProtectionDomain.java line 76:
>
>> 74: static class JavaSecurityAccessImpl implements JavaSecurityAccess {
>> 75: /* cache a copy for recording purposes */
>> 76: static Properties initialSecurityProperties;
>
> This doesn't look very clean. Could the Security class hold the initial security properties and provide an accessor method that JavaSecurityAccess:getIinitialProperties could use?
Agree, and alternatively, it seems cleaner to add a new SharedSecrets class for `java.security.Security` and remove the dependency on PD.
-------------
PR: https://git.openjdk.org/jdk/pull/10394
More information about the core-libs-dev
mailing list