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