RFR: 8338851: Hoist os::Posix::realpath() to os::realpath() and implement on Windows [v20]
David Holmes
dholmes at openjdk.org
Thu Oct 17 00:51:26 UTC 2024
On Wed, 16 Oct 2024 12:49:34 GMT, Simon Tooke <stooke at openjdk.org> wrote:
>> test/hotspot/gtest/runtime/test_os.cpp line 425:
>>
>>> 423: EXPECT_TRUE(returnedBuffer == nullptr);
>>> 424: #if defined(_WINDOWS)
>>> 425: EXPECT_TRUE(errno == ENAMETOOLONG);
>>
>> Why is this the case? Our implementation does not set it and `_fullpath` makes no mention of it.
>
> This is Windows behaviour - Windows 10 with VS 2022 at least. The existence of a file is not checked, but that non-existent filename better fit into the given buffer!
Okay. This behaviour is not specified but the test passed on our CI on Windows Server 2016 and 2019.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/20683#discussion_r1803965341
More information about the serviceability-dev
mailing list