Merge pull request #35 from ruby/fix-define-finalizer-ary

peterzhu2118 · web-flow · commit 65d5d2e43460 · 2025-05-21T10:35:31.000-04:00
Fix object ID in rb_gc_impl_define_finalizer
diff --git a/gc/mmtk/mmtk.c b/gc/mmtk/mmtk.c
@@ -946,7 +946,7 @@ rb_gc_impl_define_finalizer(void *objspace_ptr, VALUE obj, VALUE block)
         rb_ary_push(table, block);
     }
     else {
-        table = rb_ary_new3(2, block);
+        table = rb_ary_new3(2, rb_obj_id(obj), block);
         rb_obj_hide(table);
         st_add_direct(objspace->finalizer_table, obj, table);
     }

Original file line number	Diff line number	Diff line change
`@@ -946,7 +946,7 @@ rb_gc_impl_define_finalizer(void *objspace_ptr, VALUE obj, VALUE block)`
`946`	`946`	`rb_ary_push(table, block);`
`947`	`947`	`}`
`948`	`948`	`else {`
`949`		`- table = rb_ary_new3(2, block);`
	`949`	`+ table = rb_ary_new3(2, rb_obj_id(obj), block);`
`950`	`950`	`rb_obj_hide(table);`
`951`	`951`	`st_add_direct(objspace->finalizer_table, obj, table);`
`952`	`952`	`}`