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