RFR: 8299635: Hotspot update for deprecated sprintf in Xcode 14 [v5]
David Holmes
dholmes at openjdk.org
Tue Jan 17 01:39:14 UTC 2023
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
I would suggest constraining this PR to src/hotspot and test/hotspot and deal with the JDK serviceability files in a different PR (and there may be other JDK files impacted too).
src/hotspot/share/utilities/globalDefinitions.hpp line 191:
> 189: FORBID_C_FUNCTION(char* strerror(int), "use os::strerror");
> 190: FORBID_C_FUNCTION(char* strtok(char*, const char*), "use strtok_r");
> 191: FORBID_C_FUNCTION(int sprintf(char*, const char*, ...), "use os::snprintf");
I have to wonder whether this actually works too. Perhaps @kbarrett can comment?
-------------
PR: https://git.openjdk.org/jdk/pull/11935
More information about the serviceability-dev
mailing list