8269064: Dropped messages of AsyncLogWriter cause memleak

Xin Liu · simonis · commit 1323be54d268 · 2021-06-23T08:45:49.000Z
Reviewed-by: dholmes, simonis
diff --git a/src/hotspot/share/logging/logAsyncWriter.cpp b/src/hotspot/share/logging/logAsyncWriter.cpp
@@ -46,6 +46,7 @@ void AsyncLogWriter::enqueue_locked(const AsyncLogMessage& msg) {
     uint32_t* counter = _stats.add_if_absent(msg.output(), 0, &p_created);
     *counter = *counter + 1;
     // drop the enqueueing message.
+    os::free(msg.message());
     return;
   }
 

Original file line number	Diff line number	Diff line change
`@@ -46,6 +46,7 @@ void AsyncLogWriter::enqueue_locked(const AsyncLogMessage& msg) {`
`46`	`46`	`uint32_t* counter = _stats.add_if_absent(msg.output(), 0, &p_created);`
`47`	`47`	`counter = counter + 1;`
`48`	`48`	`// drop the enqueueing message.`
	`49`	`+ os::free(msg.message());`
`49`	`50`	`return;`
`50`	`51`	`}`
`51`	`52`