Although the original implementation of `JLI_Open()` borrowed from the code in src/hotspot/os/windows/os_windows.cpp, the improvements to os_windows.cpp don't seem to have been applied to java_md.c, causing some tests to fail when the path to JAR files is longer than `MAXPATH` (i.e. 260) characters on Windows (see associated JBS issue 8385024 for details). Since `JLI_Open()` is not just invoked inside tests, this is not a test-specific issue, so fixing the test is not the right solution.
This patch applies the recent changes from os_windows.cpp to java_md.c so that `JLI_Open()` can correctly handle longer than `MAXPATH` paths. The new code is almost the same as that in `wide_abs_unc_path()` in os_windows.cpp, except that the code in java_md.c uses `JLI_MemAlloc()` and `JLI_MemFree()` for memory allocation and deallocation. Although it would be ideal to have just one implementation between HotSpot and the launcher, the dependencies of the two components prevents us from having a single implementation. --------- - [x] I confirm that I make this contribution in accordance with the [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). ------------- Commit messages: - Reapply changes from os_windows.cpp to `JLI_Open()` in java_md.c Changes: https://git.openjdk.org/jdk/pull/31209/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=31209&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8385024 Stats: 107 lines in 1 file changed: 77 ins; 5 del; 25 mod Patch: https://git.openjdk.org/jdk/pull/31209.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/31209/head:pull/31209 PR: https://git.openjdk.org/jdk/pull/31209
