[8] RFR 8028377: test/sun/security/provider/KeyStore/DKSTest.sh attempts to write to ${test.src}

Vincent Ryan vincent.x.ryan at oracle.com
Tue Nov 19 14:34:52 UTC 2013


Thanks Alan. I've changed to using the two-arg constructor for File
and I've updated the webrev in place.

On 19/11/2013 14:23, Alan Bateman wrote:
> On 19/11/2013 14:16, Vincent Ryan wrote:
>> Please review the following test fix:
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8028377
>> Webrev: http://cr.openjdk.java.net/~vinnie/8028377/webrev.00/
>>
>> It now writes temporary keystores in the test scratch area
>> (not the test source area).
> This looks okay to me. One suggestion is to use the two-arg File
> constructor rather than String concatenation.
>
> -Alan.



More information about the security-dev mailing list