RFR: 8303617: update for deprecated sprintf for jdk.jdwp.agent
Chris Plummer
cjplummer at openjdk.org
Mon Mar 6 18:59:12 UTC 2023
On Sat, 4 Mar 2023 06:29:20 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 jdk.jdwp.agent module.
>
> Thanks,
> Xuelei
Copyright needs updating.
src/jdk.jdwp.agent/share/native/libdt_socket/socketTransport.c line 224:
> 222: b[received] = '\0';
> 223: /*
> 224: * We should really use snprintf here but it's not available on Windows.
This Windows comment should be removed. You can probably get rid of the jio_snprintf comment below as well since it adds little value, and doesn't make sense to just mention it here and not elsewhere.
-------------
Changes requested by cjplummer (Reviewer).
PR: https://git.openjdk.org/jdk/pull/12870
More information about the serviceability-dev
mailing list