From 92ff891877f228894c8fd4c77170a1cd793b9cd6 Mon Sep 17 00:00:00 2001 From: Matthias Baesken Date: Fri, 12 Feb 2021 07:50:28 +0000 Subject: [PATCH] 8261593: Do not use NULL pointer as write buffer parameter in jfrEmergencyDump.cpp write_repository_files Reviewed-by: stuefe, mgronlun --- .../share/jfr/recorder/repository/jfrEmergencyDump.cpp | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/src/hotspot/share/jfr/recorder/repository/jfrEmergencyDump.cpp b/src/hotspot/share/jfr/recorder/repository/jfrEmergencyDump.cpp index 45a3db36338..55c1b7ece4c 100644 --- a/src/hotspot/share/jfr/recorder/repository/jfrEmergencyDump.cpp +++ b/src/hotspot/share/jfr/recorder/repository/jfrEmergencyDump.cpp @@ -369,9 +369,10 @@ static void write_emergency_dump_file(const RepositoryIterator& iterator) { if (copy_block == NULL) { log_error(jfr, system)("Unable to malloc memory during jfr emergency dump"); log_error(jfr, system)("Unable to write jfr emergency dump file"); + } else { + write_repository_files(iterator, copy_block, block_size); + os::free(copy_block); } - write_repository_files(iterator, copy_block, block_size); - os::free(copy_block); } void JfrEmergencyDump::on_vm_error(const char* repository_path) {