8338248: PartialArrayStateAllocator::Impl leaks Arena array

zhengyu123 · zhengyu123 · commit e70c9bccaae3 · 2024-08-12T23:00:04.000Z
Reviewed-by: kbarrett, shade
diff --git a/src/hotspot/share/gc/shared/partialArrayState.cpp b/src/hotspot/share/gc/shared/partialArrayState.cpp
@@ -100,6 +100,7 @@ PartialArrayStateAllocator::Impl::~Impl() {
   for (uint i = 0; i < _num_workers; ++i) {
     _arenas[i].~Arena();
   }
+  FREE_C_HEAP_ARRAY(Arena*, _arenas);
 }
 
 PartialArrayState* PartialArrayStateAllocator::Impl::allocate(uint worker_id,

Original file line number	Diff line number	Diff line change
`@@ -100,6 +100,7 @@ PartialArrayStateAllocator::Impl::~Impl() {`
`100`	`100`	`for (uint i = 0; i < _num_workers; ++i) {`
`101`	`101`	`_arenas[i].~Arena();`
`102`	`102`	`}`
	`103`	`+ FREE_C_HEAP_ARRAY(Arena*, _arenas);`
`103`	`104`	`}`
`104`	`105`
`105`	`106`	`PartialArrayState* PartialArrayStateAllocator::Impl::allocate(uint worker_id,`