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

Brian Burkhalter bpb at openjdk.org
Fri May 30 18:10:09 UTC 2025


On Fri, 30 May 2025 05:35:01 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
>
> src/java.base/unix/classes/sun/nio/fs/UnixSecureDirectoryStream.java line 434:
> 
>> 432:                 }
>> 433:             } catch (UnixException x) {
>> 434:                 x.rethrowAsIOException(file);
> 
> 'file' will be null when changing the directory's permission, can you double check that case?

Verified.

Also updated the test in [bf2316e](https://github.com/openjdk/jdk/pull/25534/commits/bf2316ed8dbaa5215d564609417fca4b4f3363c8).

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

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


More information about the nio-dev mailing list