On Thu, 25 May 2023 12:17:27 GMT, Alan Bateman <[email protected]> wrote:
>> Aleksey Shipilev has updated the pull request incrementally with two
>> additional commits since the last revision:
>>
>> - Handle privileged properties
>> - Use ByteArray to convert. Do version/variant preparations straight on
>> locals. Move init out of optimistic lock section.
>
> src/java.base/share/classes/java/util/UUID.java line 127:
>
>> 125: static {
>> 126: try {
>> 127: PRNG_NAME = System.getProperty(PROP_NAME_PRNG_NAME,
>> null);
>
> You'll have to use GetPropertyAction.privilegedGetProperty as this will
> otherwise fail if someone runs with a SM.
Good spot. Should be fixed now.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/14135#discussion_r1206512318
