RFR: 8350869: os::stat doesn't follow symlinks on Windows [v2]
Anton Artemov
duke at openjdk.org
Thu May 15 13:33:03 UTC 2025
On Thu, 15 May 2025 07:50:46 GMT, David Holmes <dholmes at openjdk.org> wrote:
>> Anton Artemov has updated the pull request incrementally with one additional commit since the last revision:
>>
>> 8350869: Addressed reviewer's comments.
>
> src/hotspot/os/windows/os_windows.cpp line 4628:
>
>> 4626: // This method dereferences a symbolic link
>> 4627: static WCHAR* get_path_to_target(const wchar_t* wide_path)
>> 4628: {
>
> Suggestion:
>
> static WCHAR* get_path_to_target(const wchar_t* wide_path) {
Addressed in the latest commit.
> src/hotspot/os/windows/os_windows.cpp line 4638:
>
>> 4636:
>> 4637: if (target_path_size == 0)
>> 4638: {
>
> Suggestion:
>
> if (target_path_size == 0) {
Addressed in the latest commmit.
> test/hotspot/jtreg/runtime/LoadClass/TestSymlinkLoad.java line 89:
>
>> 87: public static void createLinkInSeparateFolder( final String pathToFolderForSymlink, final Path target, final String className) throws IOException {
>> 88: File theDir = new File(pathToFolderForSymlink);
>> 89: if (!theDir.exists()){
>
> Suggestion:
>
> if (!theDir.exists()) {
Addressed in the latest commit.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/25233#discussion_r2091181371
PR Review Comment: https://git.openjdk.org/jdk/pull/25233#discussion_r2091180762
PR Review Comment: https://git.openjdk.org/jdk/pull/25233#discussion_r2091181892
More information about the hotspot-runtime-dev
mailing list