commit 97809afd1c1f02999a6509c92dae149ab8d02168
author Man Cao <manc@openjdk.org> Wed Mar 15 07:17:01 2023 +0000
committer Christoph Langer <clanger@openjdk.org> Wed Mar 15 07:17:01 2023 +0000
tree f16b235052fb80a7041a360851694faffe3b8286
parent c6f00859fd555968e2c93d0633c82854a5fb5b8e

8303937: Corrupted heap dumps due to missing retries for os::write()

Reviewed-by: clanger
Backport-of: bf16b5b9880eb89b283006db090dce4346aa877b

src/hotspot/share/services/heapDumperCompression.cpp

diff --git a/src/hotspot/share/services/heapDumperCompression.cpp b/src/hotspot/share/services/heapDumperCompression.cpp
index 96cbf7e..be9a5da 100644
--- a/src/hotspot/share/services/heapDumperCompression.cpp
+++ b/src/hotspot/share/services/heapDumperCompression.cpp
@@ -53,10 +53,14 @@
   assert(_fd >= 0, "Must be open");
   assert(size > 0, "Must write at least one byte");
 
-  ssize_t n = (ssize_t) os::write(_fd, buf, (uint) size);
+  while (size > 0) {
+    ssize_t n = os::write(_fd, buf, (uint) size);
+    if (n <= 0) {
+      return os::strerror(errno);
+    }
 
-  if (n <= 0) {
-    return os::strerror(errno);
+    buf += n;
+    size -= n;
   }
 
   return NULL;