RFR: 8299635: Hotspot update for deprecated sprintf in Xcode 14 [v5]

[David Holmes]

On Sat, 14 Jan 2023 19:01:52 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:

>> The sprintf is deprecated in Xcode 14 because of security concerns. The issue was addressed in [JDK-8296812](https://bugs.openjdk.org/browse/JDK-8296812) for hotspot impl, and [JDK-8299378](https://bugs.openjdk.org/browse/JDK-8299378) for building, but the test case was not covered. The failure was reported in [PR 11793](https://github.com/openjdk/jdk/pull/11793#issuecomment-1371151565), while running tier1 testing.
>> 
>> This patch is trying to find the use of sprintf in hotspot iml and test cases.
>
> Xue-Lei Andrew Fan has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains six additional commits since the last revision:
> 
>  - Merge
>  - correction for ppc
>  - missed update for debug mode
>  - more in src/hotspot
>  - typo correction
>  - 8299635: More test issues for deprecated sprintf in Xcode 14

src/hotspot/os/windows/os_windows.cpp line 387:

> 385:     os::snprintf_checked(buf, sizeof(buf), "%s%s;%s%s%s",
> 386:             Arguments::get_java_home(), EXT_DIR,
> 387:             path, PACKAGE_DIR, EXT_DIR);

When the call is split across multiple lines like this, the subsequent lines should align with the first parameter i.e. align under `buf`.

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

PR: https://git.openjdk.org/jdk/pull/11935