diff --git a/src/coreclr/src/debug/createdump/createdumpunix.cpp b/src/coreclr/src/debug/createdump/createdumpunix.cpp index 6ddc21a75c2a05..da74ddaee5c994 100644 --- a/src/coreclr/src/debug/createdump/createdumpunix.cpp +++ b/src/coreclr/src/debug/createdump/createdumpunix.cpp @@ -45,6 +45,7 @@ CreateDump(const char* dumpPathTemplate, int pid, const char* dumpType, MINIDUMP } if (!dumpWriter.WriteDump()) { + fprintf(stderr, "Writing dump FAILED\n"); goto exit; } result = true; diff --git a/src/coreclr/src/debug/createdump/dumpwriter.cpp b/src/coreclr/src/debug/createdump/dumpwriter.cpp index 677a8b07118a29..3c61aeb9e56cd0 100644 --- a/src/coreclr/src/debug/createdump/dumpwriter.cpp +++ b/src/coreclr/src/debug/createdump/dumpwriter.cpp @@ -203,24 +203,24 @@ DumpWriter::WriteDump() // Only write the regions that are backed by memory if (memoryRegion.IsBackedByMemory()) { - uint32_t size = memoryRegion.Size(); uint64_t address = memoryRegion.StartAddress(); + size_t size = memoryRegion.Size(); total += size; while (size > 0) { - uint32_t bytesToRead = std::min(size, (uint32_t)sizeof(m_tempBuffer)); + size_t bytesToRead = std::min(size, sizeof(m_tempBuffer)); size_t read = 0; if (!m_crashInfo.ReadProcessMemory((void*)address, m_tempBuffer, bytesToRead, &read)) { - fprintf(stderr, "ReadProcessMemory(%" PRIA PRIx64 ", %08x) FAILED\n", address, bytesToRead); + fprintf(stderr, "ReadProcessMemory(%" PRIA PRIx64 ", %08zx) FAILED\n", address, bytesToRead); return false; } // This can happen if the target process dies before createdump is finished if (read == 0) { - TRACE("ReadProcessMemory(%" PRIA PRIx64 ", %08x) return 0 bytes read\n", address, bytesToRead); - break; + fprintf(stderr, "ReadProcessMemory(%" PRIA PRIx64 ", %08zx) returned 0 bytes read\n", address, bytesToRead); + return false; } if (!WriteData(m_tempBuffer, read)) {