RFR (XS) 8219583: Windows build failure after JDK-8214777 (Avoid some GCC 8.X strncpy() errors in HotSpot)

Kim Barrett kim.barrett at oracle.com
Fri Feb 22 21:01:53 UTC 2019


> On Feb 22, 2019, at 3:42 PM, Aleksey Shipilev <shade at redhat.com> wrote:
> 
> On 2/22/19 8:56 PM, Kim Barrett wrote:
>>> New code introduced in JDK-8214777 calls to "naked" snprintf, which fails to compile in my Windows
>>> configs. It seems to be the omen to replace these with the usual jio_snprintf calls. There is also
>>> os::snprintf we can use instead, but shared runtime code really likes jio_snprintf, and it will
>>> delegate as needed.
>> 
>> Please use os::snprintf: https://bugs.openjdk.java.net/browse/JDK-8198918
> 
> Even if the same file (and other files in those components) use jio_snprintf, right? I did the first
> patch with os::snprintf, but reverted it on stylistic grounds. No validation with -Wformat sounds
> ominous, though.
> 
> -Aleksey

Right.  I think we should be replacing the jio_snprintf calls (but there are a lot of them, 307 today).
(And jio_vsnprintf too (for consistency) though the checking there is much weaker.)



More information about the hotspot-runtime-dev mailing list