From: nobu.nakada@...
Date: 2003-12-05T17:22:02+09:00
Subject: [ruby-dev:22154] Re: core dump with ObjectSpace.each_object.

なかだです。

At Fri, 5 Dec 2003 16:52:04 +0900,
Yukihiro Matsumoto wrote:
> ||ちなみに私が考えてたのはこんなの。
> |
> |あ。私の修正よりもこっちの方が良いですね。
> 
> と思ったんですが、test/ruby/test_beginendblock.rbが失敗しま
> すね。やはりCVSのものをそのまま使わざるをえないでしょうか。

うーん、常にリストの末尾に追加するようにするのはどうかと思った
んですが、それだと順序が違っちゃいますねぇ。

とりあえずtmp_end_procsがfreeされたところを指さないようにするだ
けでしょうか。


Index: eval.c
===================================================================
RCS file: /cvs/ruby/src/ruby/eval.c,v
retrieving revision 1.600
diff -u -2 -p -r1.600 eval.c
--- eval.c	5 Dec 2003 05:54:00 -0000	1.600
+++ eval.c	5 Dec 2003 08:17:21 -0000
@@ -6627,5 +6627,5 @@ rb_exec_end_proc()
 	    }
 	    tmp = link;
-	    link = link->next;
+	    tmp_end_procs = link = link->next;
 	    free(tmp);
 	}
@@ -6645,9 +6645,8 @@ rb_exec_end_proc()
 	    }
 	    tmp = link;
-	    link = link->next;
+	    tmp_end_procs = link = link->next;
 	    free(tmp);
 	}
     }
-    tmp_end_procs = 0;
     ruby_safe_level = safe;
 }


-- 
--- 僕の前にBugはない。
--- 僕の後ろにBugはできる。
    中田 伸悦