8349783: g1RemSetSummary.cpp:344:68: runtime error: member call on null pointer of type 'struct G1HeapRegion'

Ivan Walulya · Ivan Walulya · commit 24b7f815ae4c · 2025-02-13T09:47:02.000Z
Reviewed-by: tschatzl, ayang
diff --git a/src/hotspot/share/gc/g1/g1RemSetSummary.cpp b/src/hotspot/share/gc/g1/g1RemSetSummary.cpp
@@ -341,7 +341,6 @@ class HRRSStatsIter: public G1HeapRegionClosure {
 
     if (max_cardset_mem_sz_group() != nullptr) {
       G1CSetCandidateGroup* cset_group = max_cardset_mem_sz_group();
-      G1HeapRegionRemSet* rem_set = max_rs_mem_sz_region()->rem_set();
       out->print_cr("    Collectionset Candidate Group with largest cardset = %u:(%u regions), "
                     "size = %zu occupied = %zu",
                     cset_group->group_id(), cset_group->length(),
@@ -389,7 +388,6 @@ void G1RemSetSummary::print_on(outputStream* out, bool show_thread_times) {
     }
     out->cr();
   }
-
   HRRSStatsIter blk;
   G1CollectedHeap::heap()->heap_region_iterate(&blk);
   blk.do_cset_groups();

Original file line number	Diff line number	Diff line change
`@@ -341,7 +341,6 @@ class HRRSStatsIter: public G1HeapRegionClosure {`
`341`	`341`
`342`	`342`	`if (max_cardset_mem_sz_group() != nullptr) {`
`343`	`343`	`G1CSetCandidateGroup* cset_group = max_cardset_mem_sz_group();`
`344`		`- G1HeapRegionRemSet* rem_set = max_rs_mem_sz_region()->rem_set();`
`345`	`344`	`out->print_cr(" Collectionset Candidate Group with largest cardset = %u:(%u regions), "`
`346`	`345`	`"size = %zu occupied = %zu",`
`347`	`346`	`cset_group->group_id(), cset_group->length(),`
`@@ -389,7 +388,6 @@ void G1RemSetSummary::print_on(outputStream* out, bool show_thread_times) {`
`389`	`388`	`}`
`390`	`389`	`out->cr();`
`391`	`390`	`}`
`392`		`-`
`393`	`391`	`HRRSStatsIter blk;`
`394`	`392`	`G1CollectedHeap::heap()->heap_region_iterate(&blk);`
`395`	`393`	`blk.do_cset_groups();`