diff options
author | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-05-29 15:10:01 +0000 |
---|---|---|
committer | mame <mame@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2010-05-29 15:10:01 +0000 |
commit | 264f4900123d5024141bd8d2e2210da9abc187e1 (patch) | |
tree | 4491a0ff78d4aa76025268d38a5681f24c51d2c5 | |
parent | 701c70e06815d17bec02fec91327b0d12b4520b6 (diff) |
* gc.c (force_chain_object, rb_objspace_call_finalizer): delete
finalizer entry after corresponding finalizer is executed.
This caused SEGV when test/cgi is executed with GC.stress = true.
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@28073 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | gc.c | 3 |
2 files changed, 8 insertions, 1 deletions
@@ -1,3 +1,9 @@ +Sun May 30 00:02:39 2010 Yusuke Endoh <mame@tsg.ne.jp> + + * gc.c (force_chain_object, rb_objspace_call_finalizer): delete + finalizer entry after corresponding finalizer is executed. + This caused SEGV when test/cgi is executed with GC.stress = true. + Sat May 29 23:30:33 2010 Yusuke Endoh <mame@tsg.ne.jp> * string.c (str_replace_shared): change embedded state atomically. @@ -2840,7 +2840,7 @@ force_chain_object(st_data_t key, st_data_t val, st_data_t arg) curr->table = val; curr->next = *prev; *prev = curr; - return ST_DELETE; + return ST_CONTINUE; } void @@ -2873,6 +2873,7 @@ rb_objspace_call_finalizer(rb_objspace_t *objspace) while (list) { struct force_finalize_list *curr = list; run_finalizer(objspace, curr->obj, rb_obj_id(curr->obj), curr->table); + st_delete(finalizer_table, (st_data_t*)&curr->obj, 0); list = curr->next; xfree(curr); } |