diff --git a/ChangeLog b/ChangeLog index a6bd57c80f..88336249f0 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,10 @@ +Sat Nov 17 00:50:23 2012 CHIKANAGA Tomoyuki + + * marshal.c (w_object): add flonum to arg->data to keep reference index + consistency. [ruby-core:49323] [Bug #7348] + + * test/ruby/test_marshal.rb: add a test for above. + Sat Nov 17 00:40:25 2012 Nobuyoshi Nakada * common.mk (incs): dist files need probes.dmyh. diff --git a/marshal.c b/marshal.c index ae383829d2..9e81d3cf8c 100644 --- a/marshal.c +++ b/marshal.c @@ -637,6 +637,7 @@ w_object(VALUE obj, struct dump_arg *arg, int limit) w_symbol(SYM2ID(obj), arg); } else if (FLONUM_P(obj)) { + st_add_direct(arg->data, obj, arg->data->num_entries); w_byte(TYPE_FLOAT, arg); w_float(RFLOAT_VALUE(obj), arg); } diff --git a/test/ruby/test_marshal.rb b/test/ruby/test_marshal.rb index 85cec0adca..e68839472d 100644 --- a/test/ruby/test_marshal.rb +++ b/test/ruby/test_marshal.rb @@ -492,4 +492,11 @@ class TestMarshal < Test::Unit::TestCase assert_equal(Rational(1, 2), Marshal.load("\x04\bU:\rRational[\ai\x06i\a")) assert_raise(ArgumentError){Marshal.load("\x04\bU:\rRational[\bi\x00i\x00i\x00")} end + + def test_marshal_flonum_reference + bug7348 = '[ruby-core:49323]' + e = [] + ary = [ [2.0, e], [e] ] + assert_equal(ary, Marshal.load(Marshal.dump(ary)), bug7348) + end end