RFR (XS) 8224033: os::snprintf should be used in virtualizationSupport.cpp

Aleksey Shipilev shade at redhat.com
Thu May 16 11:01:28 UTC 2019


Bug:
  https://bugs.openjdk.java.net/browse/JDK-8224033

Fix:

diff -r 382101e97784 src/hotspot/share/utilities/virtualizationSupport.cpp
--- a/src/hotspot/share/utilities/virtualizationSupport.cpp     Thu May 16 10:52:36 2019 +0200
+++ b/src/hotspot/share/utilities/virtualizationSupport.cpp     Thu May 16 13:00:35 2019 +0200
@@ -66,5 +66,5 @@
     if (sg_error == VMGUESTLIB_ERROR_SUCCESS) {
       has_resource_information = true;
-      snprintf(extended_resource_info_at_startup, sizeof(extended_resource_info_at_startup), "%s",
result_info);
+      os::snprintf(extended_resource_info_at_startup, sizeof(extended_resource_info_at_startup),
"%s", result_info);
       GuestLib_StatFree(result_info, result_size);
     }
@@ -72,5 +72,5 @@
     if (sg_error == VMGUESTLIB_ERROR_SUCCESS) {
       has_host_information = true;
-      snprintf(host_information, sizeof(host_information), "%s", result_info);
+      os::snprintf(host_information, sizeof(host_information), "%s", result_info);
       GuestLib_StatFree(result_info, result_size);
     }


os::snprintf should be used in new code because -Wformat does not check raw snprintf (see
JDK-8198918). It also fixes Windows build failure without PCH. Can run more tests on-demand, but I
think it is trivial, so I only made the builds.

Testing: {Linux, Windows} x86_64 fastdebug builds;
-- 
Thanks,
-Aleksey



More information about the hotspot-dev mailing list