This is an automated email from the ASF dual-hosted git repository.

kou pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/main by this push:
     new e21d057a22 GH-48880: [Ruby] Fix a bug that Arrow::ExecutePlan nodes 
may be GC-ed (#48919)
e21d057a22 is described below

commit e21d057a225396f21091f00c8743b90057bdb89f
Author: Sutou Kouhei <[email protected]>
AuthorDate: Mon Jan 26 11:02:42 2026 +0900

    GH-48880: [Ruby] Fix a bug that Arrow::ExecutePlan nodes may be GC-ed 
(#48919)
    
    ### Rationale for this change
    
    We must mark all nodes in `Arrow::ExecutePlan` but only the first node is 
marked.
    
    ### What changes are included in this PR?
    
    Fix typos in variable name.
    
    ### Are these changes tested?
    
    Yes.
    
    ### Are there any user-facing changes?
    
    Yes.
    * GitHub Issue: #48880
    
    Authored-by: Sutou Kouhei <[email protected]>
    Signed-off-by: Sutou Kouhei <[email protected]>
---
 ruby/red-arrow/ext/arrow/arrow.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/ruby/red-arrow/ext/arrow/arrow.cpp 
b/ruby/red-arrow/ext/arrow/arrow.cpp
index 404ec8996f..0c582d0707 100644
--- a/ruby/red-arrow/ext/arrow/arrow.cpp
+++ b/ruby/red-arrow/ext/arrow/arrow.cpp
@@ -59,7 +59,7 @@ namespace red_arrow {
   {
     auto plan = GARROW_EXECUTE_PLAN(object);
     auto nodes = garrow_execute_plan_get_nodes(plan);
-    for (auto node = nodes; nodes; nodes = g_list_next(nodes)) {
+    for (auto node = nodes; node; node = g_list_next(node)) {
       rbgobj_gc_mark_instance(node->data);
     }
   }

Reply via email to