RFR: 8289984: Files:isDirectory and isRegularFile methods not throwing SecurityException

Iris Clark iris at openjdk.org
Fri Jul 8 18:44:47 UTC 2022


On Fri, 8 Jul 2022 18:13:56 GMT, Lance Andersen <lancea at openjdk.org> wrote:

> Hi all,
> 
> Please review this change to address the fact that we were missing a permissions check when the SecurityManager is enabled
> 
> The failing jck tests are passing locally and just waiting on another platform to complete on Mach5  and will be good to go
> 
> Best
> Lance

Marked as reviewed by iris (Reviewer).

test/jdk/java/nio/file/Files/CheckPermissions.java line 219:

> 217: 
> 218:             prepare();
> 219:             exists(file);

Sorry for the poor comment placement.  Do you need to add the bugid back at line 25?

-------------

PR: https://git.openjdk.org/jdk/pull/9433


More information about the nio-dev mailing list