8334164: The fix for JDK-8322811 should use _filename.is_set() rather…

… than strcmp() Reviewed-by: dholmes, cjplummer
openjdk · Jun 20, 2024 · cabd104 · cabd104 · openjdk-notifier · Jun 20, 2024
1 parent d7dad50
commit cabd104
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/hotspot/share/services/diagnosticCommand.cpp b/src/hotspot/share/services/diagnosticCommand.cpp
@@ -1203,11 +1203,11 @@ SystemDumpMapDCmd::SystemDumpMapDCmd(outputStream* output, bool heap) :
 void SystemDumpMapDCmd::execute(DCmdSource source, TRAPS) {
   stringStream defaultname;
   const char* name = nullptr;
-  if (::strcmp(default_filename, _filename.value()) == 0) {
+  if (_filename.is_set()) {
+    name = _filename.value();
+  } else {
     defaultname.print("vm_memory_map_%d.txt", os::current_process_id());
     name = defaultname.base();
-  } else {
-    name = _filename.value();
   }
   fileStream fs(name);
   if (fs.is_open()) {