RFR: 8303830: update for deprecated sprintf for jdk.accessibility
Xue-Lei Andrew Fan
xuelei at openjdk.org
Thu May 4 21:25:17 UTC 2023
On Sat, 29 Apr 2023 08:29:39 GMT, Alexander Zuev <kizune 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 jdk.accessibility module.
>>
>> Thanks,
>> Xuelei
>
> The problem with testing of this code is that a lot of it (if not all) is inside the debug and logging part, which means that not only library should be loaded (doable) and the debug is on (also doable) but some of that code is just an error report and in order to cause system to misbehave enough to cause such error is not trivial. Looking into it though.
@azuev-java Thank you very much for helping the testing. Are two reviewers required for client lib PR review?
-------------
PR Comment: https://git.openjdk.org/jdk/pull/12929#issuecomment-1535431609
More information about the client-libs-dev
mailing list