From 0f90630983db3257c544c698761a00456af81d85 Mon Sep 17 00:00:00 2001 From: Aaron Patterson Date: Wed, 11 Dec 2019 10:12:14 -0800 Subject: Update method tables only if there is a class ext pointer This makes reference updating look similar to marking, and may avoid dereferencing a wrong pointer. --- gc.c | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'gc.c') diff --git a/gc.c b/gc.c index 3a62f57e36..493a156a37 100644 --- a/gc.c +++ b/gc.c @@ -8179,14 +8179,12 @@ gc_update_object_references(rb_objspace_t *objspace, VALUE obj) switch (BUILTIN_TYPE(obj)) { case T_CLASS: case T_MODULE: - update_m_tbl(objspace, RCLASS_M_TBL(obj)); if (RCLASS_SUPER((VALUE)obj)) { UPDATE_IF_MOVED(objspace, RCLASS(obj)->super); } if (!RCLASS_EXT(obj)) break; - if (RCLASS_IV_TBL(obj)) { - gc_update_tbl_refs(objspace, RCLASS_IV_TBL(obj)); - } + update_m_tbl(objspace, RCLASS_M_TBL(obj)); + gc_update_tbl_refs(objspace, RCLASS_IV_TBL(obj)); update_class_ext(objspace, RCLASS_EXT(obj)); update_const_tbl(objspace, RCLASS_CONST_TBL(obj)); break; -- cgit v1.2.3