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