RFR: 8357425: (fs) SecureDirectoryStream setPermissions should use fchmodat [v3]

Brian Burkhalter bpb at openjdk.org
Mon Jun 2 16:16:07 UTC 2025


On Sat, 31 May 2025 06:27:39 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8357425: Change view used to set directory permissions
>
> test/jdk/java/nio/file/DirectoryStream/SecureDS.java line 187:
> 
>> 185:         Set<PosixFilePermission> permsDir = getPosixFilePermissions(aDir);
>> 186: 
>> 187:         SecureDirectoryStream<Path> stream =
> 
> If you are doing any more edits then you can change this to use try-with-resources.

So changed in https://github.com/openjdk/jdk/pull/25534/commits/bf2a80f06d2dbe95d03fd79bbeae65cf4318beea.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25534#discussion_r2121610856


More information about the nio-dev mailing list