RFR JDK-8066709 Make some JDK system properties read only

mandy chung mandy.chung at oracle.com
Wed Jun 27 05:54:41 UTC 2018


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