From 82bbce826f7f170270856bec8ef852667028d143 Mon Sep 17 00:00:00 2001 From: Koichi Sasada Date: Fri, 4 Dec 2020 14:39:03 +0900 Subject: [PATCH] fix index `i` should not be used here because `i` will be `alen` when jumped here by "goto restart". --- ractor.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/ractor.c b/ractor.c index f17a75bc3f..d7a9c85523 100644 --- a/ractor.c +++ b/ractor.c @@ -802,6 +802,7 @@ ractor_select(rb_execution_context_t *ec, const VALUE *rs, int alen, VALUE yield bool interrupted = false; enum ractor_wait_status wait_status = 0; bool yield_p = (yielded_value != Qundef) ? true : false; + const int rs_len = alen; struct ractor_select_action { enum ractor_select_action_type { @@ -840,8 +841,8 @@ ractor_select(rb_execution_context_t *ec, const VALUE *rs, int alen, VALUE yield restart: if (yield_p) { - actions[i].type = ractor_select_action_yield; - actions[i].v = Qundef; + actions[rs_len].type = ractor_select_action_yield; + actions[rs_len].v = Qundef; wait_status |= wait_yielding; alen++;