diff options
-rw-r--r-- | ChangeLog | 6 | ||||
-rw-r--r-- | marshal.c | 3 | ||||
-rw-r--r-- | test/ruby/test_marshal.rb | 9 |
3 files changed, 16 insertions, 2 deletions
@@ -1,3 +1,9 @@ +Wed Mar 28 15:06:18 2012 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * marshal.c (r_symreal): default to ASCII-8BIT for non-ascii symbols, + otherwise it should be converted to US-ASCII in rb_intern_str() if + possible. [ruby-core:43762][Bug #6209] + Wed Mar 28 08:44:24 2012 Aaron Patterson <aaron@tenderlovemaking.com> * ext/psych/lib/psych.rb: updating version to match gem @@ -1161,8 +1161,7 @@ r_symreal(struct load_arg *arg, int ivar) idx = id2encidx(id, r_object(arg)); } } - if (idx < 0) idx = rb_usascii_encindex(); - rb_enc_associate_index(s, idx); + if (idx > 0) rb_enc_associate_index(s, idx); id = rb_intern_str(s); st_insert(arg->symbols, (st_data_t)n, (st_data_t)id); diff --git a/test/ruby/test_marshal.rb b/test/ruby/test_marshal.rb index 15951bff36..1dea7e14dd 100644 --- a/test/ruby/test_marshal.rb +++ b/test/ruby/test_marshal.rb @@ -455,6 +455,15 @@ class TestMarshal < Test::Unit::TestCase assert_equal(o1, o2) end + def test_marshal_symbol_ascii8bit + bug6209 = '[ruby-core:43762]' + o1 = "\xff".force_encoding("ASCII-8BIT").intern + m = Marshal.dump(o1) + o2 = nil + assert_nothing_raised(EncodingError, bug6209) {o2 = Marshal.load(m)} + assert_equal(o1, o2, bug6209) + end + class PrivateClass def initialize(foo) @foo = foo |