RFR: 8350939: Revisit Windows PDH buffer size calculation for OperatingSystemMXBean

Kevin Walls kevinw at openjdk.org
Thu Mar 6 12:30:06 UTC 2025


On Mon, 3 Mar 2025 12:22:01 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

> Following on from JDK-8350820, which backed out the _snprintf to snprintf change (JDK-8336289) in OperatingSystemImpl.c on Windows, because the counter names were being truncated (so CPU monitoring was not possible).
> 
> This change moves to snprintf again, but the counter names are not truncated.
> 
> snprintf must need the null terminator to fit inside the buffer length given.  It does not, and snprintf truncates (and always add the null terminator).

Thanks for reviews.
Reran with edited problemlist to run relevant tests, all good.
Hope we can clear problemlist in near future!...

-------------

PR Comment: https://git.openjdk.org/jdk/pull/23861#issuecomment-2703704150


More information about the serviceability-dev mailing list