RFR: 8338248: PartialArrayStateAllocator::Impl leaks Arena array

Kim Barrett kbarrett at openjdk.org
Mon Aug 12 18:24:30 UTC 2024


On Mon, 12 Aug 2024 17:52:55 GMT, Zhengyu Gu <zgu at openjdk.org> wrote:

> Simple fix for a memory leak

Marked as reviewed by kbarrett (Reviewer).

src/hotspot/share/gc/shared/partialArrayState.cpp line 104:

> 102:   }
> 103: 
> 104:   FREE_C_HEAP_ARRAY(Arena*, _arenas);

Bleh.  Thanks for catching this.  I don't see a need for the extra blank line, but otherwise this is good.

-------------

PR Review: https://git.openjdk.org/jdk/pull/20557#pullrequestreview-2233741819
PR Review Comment: https://git.openjdk.org/jdk/pull/20557#discussion_r1714200858


More information about the hotspot-gc-dev mailing list