ruby-dev

Issue #5152 has been updated by Tomoyuki Chikanaga.


すみません、ソースコードを良く読むと RB_GC_GUARD() を挿入しないといけないのはさっきのところじゃなくて関数の最後のほうに挿入する必要がありそうでした。

また parse_ddd_cb() にも GC 保護が必要な変数がありそうでした。

@@ -1323,6 +1324,7 @@ parse_ddd_cb(VALUE m, VALUE hash)
        }
        break;
     }
+    RB_GC_GUARD(s2);
     if (!NIL_P(s3)) {
        cs3 = RSTRING_PTR(s3);
        l3 = RSTRING_LEN(s3);
@@ -1354,6 +1356,7 @@ parse_ddd_cb(VALUE m, VALUE hash)
            }
        }
     }
+    RB_GC_GUARD(s3);
     if (!NIL_P(s4)) {
        l4 = RSTRING_LEN(s4);

@@ -1392,6 +1395,7 @@ parse_ddd_cb(VALUE m, VALUE hash)
            set_hash("offset", date_zone_to_diff(rb_str_new2(s1)));
        }
     }
+    RB_GC_GUARD(s5);

     return 1;
 }

----------------------------------------
Bug #5152: TestDateNew#test_civil が GC.stress = true 下で Failure
http://redmine.ruby-lang.org/issues/5152

Author: Tomoyuki Chikanaga
Status: Open
Priority: Normal
Assignee: tadayoshi funaba
Category: ext
Target version: 1.9.3
ruby -v: ruby 1.9.4dev (2011-08-03 trunk 32823) [x86_64-linux]


make test-all TESTS="-vq --gc-stress date/test_date_new.rb -n test_civil"
のように GC.stress = true 下で DateTime のテストが Failure します。

  1) Failure:
test_civil(TestDateNew) [/home/chikanaga/opt/ruby-trunk/src/ruby/test/date/test_date_new.rb:100]:
<[-4712, 1, 1, 0, 0, 0, (3/8)]> expected but was
<[-4712, 1, 1, 0, 0, 0, (0/1)]>.

以下のようなスクリプトでも再現します。また ruby_1_9_3 でも再現しました。

$ cat datetime.rb
require "date"
GC.stress = true
dt = DateTime.civil(-4712,1,1, 0,0,0, '+0900')
p dt.offset   # => (3/8) になるはずが (0/1) になる

date_parse.c の date_zone_to_diff での str の GC 保護されていないためだと思います。
以下のような変更で期待した offset を得られるようになりました。

diff --git a/ext/date/date_parse.c b/ext/date/date_parse.c
index 3605ff7..e41606b 100644
--- a/ext/date/date_parse.c
+++ b/ext/date/date_parse.c
@@ -374,6 +374,7 @@ date_zone_to_diff(VALUE str)
            sp = 0;
        }
     }
+    RB_GC_GUARD(str);
     if (d > dest) {
        if (*(d - 1) == ' ')
            --d;



-- 
http://redmine.ruby-lang.org

Thread

Prev Next

In This Thread

Prev Next

[#44287] [Ruby 1.9 - Bug #5127] ruby 1.9.3 prev1 で Array#[]=( Array.new ) が1.9.2より2倍遅い — Motohiro KOSAKI <kosaki.motohiro@...>

[#44288] [Ruby 1.9 - Bug #5127] ruby 1.9.3 prev1 で Array#[]=( Array.new ) が1.9.2より2倍遅い — Tomoyuki Chikanaga <nagachika00@...>

[#44289] [Ruby 1.9 - Feature #5128][Open] 日本語ドキュメントをUTF-8に — Shyouhei Urabe <shyouhei@...>

[#44303] [Ruby 1.9 - Bug #5132][Open] test_parallel.rb fail when it run before 'make install' — Ayumu AIZAWA <ayumu.aizawa@...>

[#44304] [Ruby 1.9 - Feature #5133][Open] Array#unzip as an alias of Array#transpose — Kenta Murata <muraken@...>

[#44314] [Ruby 1.9 - Bug #5136][Open] :"!".inspect の結果が1.9.3devと1.9.2とでは異なる — Ayumu AIZAWA <ayumu.aizawa@...>

[#44315] [Ruby 1.9 - Bug #5139][Open] sigsegv のスタックオーバフロー — Tomoyuki Chikanaga <nagachika00@...>

[#44318] [Ruby 1.9 - Feature #5142][Open] Remove ruby-mode.el from ruby's repo — Yui NARUSE <naruse@...>

[#44327] [Ruby 1.9 - Bug #5150][Open] Ruby-trunk Crashed in running test-rubyspec — Ayumu AIZAWA <ayumu.aizawa@...>

[#44329] [Ruby 1.9 - Bug #5151][Open] test/socket/test_socket.rb fail when udp connection failed — Ayumu AIZAWA <ayumu.aizawa@...>

[#44337] [Ruby 1.9 - Bug #5152][Open] TestDateNew#test_civil が GC.stress = true 下で Failure — Tomoyuki Chikanaga <nagachika00@...>

[#44342] [Ruby 1.9 - Bug #5155][Open] [BUG] pthread_cond_signal: Resource temporarily unavailable (EAGAIN) — Kenta Murata <muraken@...>

[#44355] [Ruby 1.9 - Bug #5159][Open] r32777のためSolaris上のgcc-4.4でCFLAGS="-std=gnu99"のときコンパイルエラー — Naohisa Goto <ngotogenome@...>

[#44356] [Ruby 1.9 - Bug #5160][Open] Float::INFINITY and Float::NAN incorrect in big endian architecture — Naohisa Goto <ngotogenome@...>

[#44361] [Ruby 1.9 - Bug #5127] ruby 1.9.3 prev1 で Array#[]=( Array.new ) が1.9.2より2倍遅い — Narihiro Nakamura <authorNari@...>

[#44362] [Ruby 1.9 - Bug #5175][Open] Ruby1.9.2p290 MySQLアクセスエラー — eiichi maekawa <eiichi_maekawa@...>

[#44363] [Ruby 1.9 - Bug #5176][Open] UTF-8以外でpsychを使用し、YAML.dumpを実行するとエラーが発生する — Masafumi Kiribayashi <masafumi.kiribayashi@...>

[#44368] [Ruby 1.9 - Feature #5180][Open] net/http の接続時に用いる IP アドレスの指定 — Yui NARUSE <naruse@...>

[#44400] [Ruby 1.9 - Bug #5195][Open] Queue#popでsleepしているthreadをwakeupさせるとQueueの@waitingにそのthreadがpushされてしまう — Masaki Matsushita <glass.saga@...>

[#44404] [Ruby 1.9 - Bug #5208][Open] \b and \B don't match properly — Ken Takata <kentkt@...>

[#44409] [Ruby 1.9 - Bug #5213][Open] GC.stress=true で TestSH#test_commercial が Failure になる — Tomoyuki Chikanaga <nagachika00@...>

[#44412] Releasing r33028 as Ruby 1.9.3 RC1 — Yugui <yugui@...>

[#44413] [Ruby 1.9 - Bug #5217][Open] lineno is broken when source code has about 7000 lines — Yusuke Endoh <mame@...>

[#44420] [Ruby 1.9 - Bug #5222][Open] test_script_from_stdin のエラー — Naohisa Goto <ngotogenome@...>

[#44424] [Ruby 1.9 - Backport #5223][Open] please backport r33030 — Yusuke Endoh <mame@...>

[#44435] [BUG: ruby 1.9.3dev (2011-08-02 revision 32810) [i686-linux]] C の Proc の比較のバグ — Satoshi Shiba <shiba@...>

[#44436] [BUG: ruby 1.9.3dev (2011-08-27 revision 33101) [i686-linux]] class オブジェクトの開放時に segv — Satoshi Shiba <shiba@...>

[#44438] [Ruby 1.9 - Bug #5249] C の Proc の比較のバグ — Yui NARUSE <naruse@...>

[ruby-dev:44340] [Ruby 1.9 - Bug #5152] TestDateNew#test_civil が GC.stress = true 下で Failure

Thread

In This Thread

[#44362] [Ruby 1.9 - Bug #5175][Open] Ruby1.9.2p290　MySQLアクセスエラー — eiichi maekawa <eiichi_maekawa@...>