RFR: JDK-8047769 SecureRandom should be more frugal with file descriptors
Peter Levart
peter.levart at gmail.com
Thu Jan 1 19:21:48 UTC 2015
On 12/29/2014 04:51 PM, Alan Bateman wrote:
> On 29/12/2014 09:45, Peter Levart wrote:
>>
>> Thanks for looking at this, Alan.
>>
>> You're right about File.getCanonicalFile(). It already checks read
>> permission for a file. The additional explicit check is superfluous.
>> I have removed it.
>>
>> With explicit check I wanted the API to behave uniformly regardless
>> of whether the returned stream is opened by getInputStream() call or
>> an already opened stream is just returned. getCannonicalFile()
>> already takes care of it. Here's the updated webrev:
>>
>> http://cr.openjdk.java.net/~plevart/jdk9-dev/FileInputStreamPool.8047769/webrev.03/
>>
> Updated patch looks good to me.
>
> -Alan.
Thanks, Alan.
Peter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://mail.openjdk.org/pipermail/security-dev/attachments/20150101/f24dd8e6/attachment.htm>
More information about the security-dev
mailing list