RFR 8223771: (zipfs) FileSystemProvider.newFileSystem(Path, Map) should throw IOException when called with a file that cannot be open

Alan Bateman Alan.Bateman at oracle.com
Thu Oct 3 18:43:19 UTC 2019


On 03/10/2019 19:01, Lance Andersen wrote:
> I updated the patch: 
> http://cr.openjdk.java.net/~lancea/8223771/webrev.01/ as well as the 
> CSR to reflect the change and lack of a property.
Looks good. I tweaked a few things in the CSR and added myself as a 
Reviewer.

>
> I reverted the change  which just returned the filename when the 
> Exception is thrown.  My reasoning for just returning the filename was 
> based on 
> https://www.oracle.com/technetwork/java/seccodeguide-139067.html#2 .
>
Right, there's a discussion topic there as exception messages can leak 
interesting file paths.

-Alan


More information about the nio-dev mailing list