diff options
-rw-r--r-- | ChangeLog | 5 | ||||
-rw-r--r-- | error.c | 2 | ||||
-rw-r--r-- | test/ruby/test_float.rb | 12 | ||||
-rw-r--r-- | version.h | 2 |
4 files changed, 19 insertions, 2 deletions
@@ -1,3 +1,8 @@ +Mon Jan 24 21:04:45 2011 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * error.c (rb_invalid_str): prevent intermediate variable from GC. + [ruby-core:34820] + Sat Jan 22 11:21:40 2011 Aaron Patterson <aaron@tenderlovemaking.com> * ext/psych/parser.c (parse): fixing off-by-one error on line numbers @@ -927,7 +927,7 @@ nometh_err_args(VALUE self) void rb_invalid_str(const char *str, const char *type) { - VALUE s = rb_str_inspect(rb_str_new2(str)); + volatile VALUE s = rb_str_inspect(rb_str_new2(str)); rb_raise(rb_eArgError, "invalid value for %s: %s", type, RSTRING_PTR(s)); } diff --git a/test/ruby/test_float.rb b/test/ruby/test_float.rb index fffa96e313..b465814140 100644 --- a/test/ruby/test_float.rb +++ b/test/ruby/test_float.rb @@ -442,6 +442,18 @@ class TestFloat < Test::Unit::TestCase assert(Float(o).nan?) end + def test_invalid_str + bug4310 = '[ruby-core:34820]' + assert_raise(ArgumentError, bug4310) { + stress, GC.stress = GC.stress, true + begin + Float('a'*10000) + ensure + GC.stress = stress + end + } + end + def test_num2dbl assert_raise(TypeError) do 1.0.step(2.0, "0.5") {} @@ -1,5 +1,5 @@ #define RUBY_VERSION "1.9.2" -#define RUBY_PATCHLEVEL 169 +#define RUBY_PATCHLEVEL 170 #define RUBY_VERSION_MAJOR 1 #define RUBY_VERSION_MINOR 9 #define RUBY_VERSION_TEENY 1 |