On Sat, 14 Jan 2023 19:01:52 GMT, Xue-Lei Andrew Fan <xue...@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
