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