JDK-8334164 : The fix for JDK-8322811 should use _filename.is_set() rather than strcmp()
  • Type: Bug
  • Component: hotspot
  • Sub-Component: runtime
  • Affected Version: 24
  • Priority: P4
  • Status: Open
  • Resolution: Unresolved
  • Submitted: 2024-06-12
  • Updated: 2024-06-18
The Version table provides details related to the release that this issue/RFE will be addressed.

Unresolved : Release in which this issue/RFE will be addressed.
Resolved: Release in which this issue/RFE has been resolved.
Fixed : Release in which this issue/RFE has been fixed. The release containing this fix may be available for download as an Early Access Release or a General Availability Release.

To download the current JDK release, click here.
JDK 24
24Unresolved
Related Reports
Relates :  
JDK-8322811 - jcmd System.dump_map help info has conflicting statements
Description
JDK-8322811 fixes help output for the System.dump_map -F argument so instead of

 -F : [optional] file path (defaults: "vm_memory_map_<pid>.txt") (STRING, no default value) 
 
It instead reads:

 -F : [optional] file path (STRING, "vm_memory_map_<pid>.txt") 

Although the implementation produces the correct output, there is a bug in it. If you specify the literal "vm_memory_map_<pid>.txt" as the map file name, <pid> is still replaced with the actual process ID, but it shouldn't be.

The correct way to check if a default was provided is to use _filename.is_set() rather than strcmp to "vm_memory_map_<pid>.txt". See the fix for JDK-8323546 as an example.
Comments
A pull request was submitted for review. URL: https://git.openjdk.org/jdk/pull/19706 Date: 2024-06-13 16:55:15 +0000
14-06-2024