RFR(S): JDK-8214444: Wrong strncat limits in dfa.cpp

Dmitry Chuyko dmitry.chuyko at bell-sw.com
Mon Dec 3 08:54:34 UTC 2018


Vladimir, thanks for the review.

I've got submit repo results prior to RFR:

[Mach5] mach5-one-dchuyko-JDK-8214444-20181130-1002-12450: PASSED

-Dmitry

On 11/30/18 10:28 PM, Vladimir Kozlov wrote:
> Looks good.
>
> Please, test it through 'submit' repo.
>
> Thanks,
> Vladimir
>
> On 11/30/18 4:30 AM, Dmitry Chuyko wrote:
>> Hello,
>>
>> Please review a small fix for another error found by modern GCC. 
>> strncat()-s with wrong limits are replaced by snprintf()-s. For 
>> VS2013 compatibility there is additional define and force null 
>> termination.
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8214444
>> webrev: http://cr.openjdk.java.net/~dchuyko/8214444/webrev.00/
>> testing: submit repo
>>
>> -Dmitry
>>


More information about the hotspot-compiler-dev mailing list