RFR JDK-8066709 Make some JDK system properties read only

Roger Riggs Roger.Riggs at Oracle.com
Wed Jun 27 13:20:43 UTC 2018


I rechecked, almost all were within doPriv and the others have new 
explicit SM checks.

Thanks, Roger

On 6/27/2018 1:54 AM, mandy chung wrote:
> Looks good to me.  The part that I'm unsure is whether you caught all 
> the callers that are outside doPrivileged to StaticProperty.* will 
> need to do its property permission check.  This requires API 
> inspection and testing that I assume you covered that.
>
> Mandy
>
> On 6/26/18 7:10 PM, Roger Riggs wrote:
>> Hi,
>>
>> Updated webrev:
>>
>> http://cr.openjdk.java.net/~rriggs/webrev-static-property-8066709/index.html 
>>
>>
>> Applied changes from prior comments and droped a change no longer 
>> needed due
>> to the TLS 1.3 removal of ClientKeyExchangeService.java.
>>
>> The CSR has been approved without possibly confusing @implNote in 
>> System.getProperties
>> about caching of specific properties, including java.home, etc.
>>
>> Thanks for any additional comments.
>>
>> Roger
>>
>>
>>
>> On 6/19/18 11:52 AM, Brent Christian wrote:
>>> On 6/19/18 8:08 AM, Roger Riggs wrote:
>>>>>
>>>>> * src/java.base/share/classes/java/lang/System.java :
>>>>>
>>>>> Should the @implNote with the list of cached properties be added 
>>>>> everywhere the @apiNote is being added ?  Right now the @implNote 
>>>>> is only added to getProperties().
>>>>>
>>>> The repetition was getting tiresome and the base of all the 
>>>> xxxProperties methods is getProperties.
>>>>   Joe suggested having one copy of the full information  and 
>>>> referring to that from the individual @apiNotes.
>>>
>>> Fair enough.
>>>
>>>>> * src/java.base/share/classes/jdk/internal/util/StaticProperty.java :
>>>>>
>>>>>   45     private StaticProperty() {
>>>>>   46
>>>>>   47     }
>>>>>
>>>>> Maybe put this all on one line?
>>>>>
>>>> Will do
>>>
>>> Thanks,
>>> -Brent
>>>
>>



More information about the core-libs-dev mailing list