RFR: 8307855: update for deprecated sprintf for src/utils
Tobias Hartmann
thartmann at openjdk.org
Fri May 12 14:18:45 UTC 2023
On Wed, 10 May 2023 18:26:40 GMT, Xue-Lei Andrew Fan <xuelei at openjdk.org> wrote:
> Hi,
>
> May I have this update reviewed?
>
> The sprintf is deprecated in Xcode 14, and Microsoft Virtual Studio, because of security concerns. The issue was addressed in [JDK-8296812](https://bugs.openjdk.org/browse/JDK-8296812) for building failure, and [JDK-8299378](https://bugs.openjdk.org/browse/JDK-8299378)/[JDK-8299635](https://bugs.openjdk.org/browse/JDK-8299635)/[JDK-8301132](https://bugs.openjdk.org/browse/JDK-8301132) for testing issues . This is a break-down update for sprintf uses in the src/utils directory.
>
> Thanks,
> Xuelei
Looks good to me otherwise.
src/utils/hsdis/binutils/hsdis-binutils.c line 2:
> 1: /*
> 2: 3* Copyright (c) 2008, 2023, Oracle and/or its affiliates. All rights reserved.
Suggestion:
* Copyright (c) 2008, 2023, Oracle and/or its affiliates. All rights reserved.
-------------
Marked as reviewed by thartmann (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/13915#pullrequestreview-1424602363
PR Review Comment: https://git.openjdk.org/jdk/pull/13915#discussion_r1192434582
More information about the hotspot-compiler-dev
mailing list