diff --git a/ChangeLog b/ChangeLog index 1de0a2acb3..32e9289275 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,9 @@ +Wed Mar 21 07:21:24 2007 Akinori MUSHA + + * ext/thread/thread.c (remove_one): Preserve List invariants; + submitted by: MenTaLguY + in [ruby-core:10598] and [ruby-bugs:PR#9388]. + Tue Mar 20 22:54:50 2007 Yukihiro Matsumoto * marshal.c (w_extended): erroneous check condition when dump diff --git a/ext/thread/thread.c b/ext/thread/thread.c index d617185a26..b0badf11b8 100644 --- a/ext/thread/thread.c +++ b/ext/thread/thread.c @@ -163,13 +163,18 @@ static void remove_one(List *list, VALUE value) { Entry **ref; + Entry *prev; Entry *entry; - for (ref = &list->entries, entry = list->entries; + for (ref = &list->entries, prev = NULL, entry = list->entries; entry != NULL; - ref = &entry->next, entry = entry->next) { + ref = &entry->next, prev = entry, entry = entry->next) { if (entry->value == value) { *ref = entry->next; + list->size--; + if (!entry->next) { + list->last_entry = prev; + } recycle_entries(list, entry, entry); break; }