From 054dbe2a43c6c2183bd018a15724a59bb8fd48d3 Mon Sep 17 00:00:00 2001 From: nobu Date: Wed, 7 Dec 2011 09:56:01 +0000 Subject: * vm.c (vm_set_top_stack, vm_set_eval_stack): check for stack overflow with stack_max before push new frame. [ruby-core:41520] [Bug #5720] * vm.c (vm_set_main_stack): no stack overflow chances after vm_set_eval_stack(). git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@33970 b2dd03c8-39d4-4d8f-98ff-823fe69b080e --- ChangeLog | 9 +++++++++ test/ruby/test_exception.rb | 6 ++++++ vm.c | 9 +++------ 3 files changed, 18 insertions(+), 6 deletions(-) diff --git a/ChangeLog b/ChangeLog index 6e548b85e6..0da0d06837 100644 --- a/ChangeLog +++ b/ChangeLog @@ -1,3 +1,12 @@ +Wed Dec 7 18:55:56 2011 Nobuyoshi Nakada + + * vm.c (vm_set_top_stack, vm_set_eval_stack): check for stack + overflow with stack_max before push new frame. [ruby-core:41520] + [Bug #5720] + + * vm.c (vm_set_main_stack): no stack overflow chances after + vm_set_eval_stack(). + Wed Dec 7 09:58:15 2011 Eric Hodel * ext/bigdecimal/bigdecimal.c: Document +@, -@, hash, INFINITY, Nan. diff --git a/test/ruby/test_exception.rb b/test/ruby/test_exception.rb index 52dd4665a1..1d16da695c 100644 --- a/test/ruby/test_exception.rb +++ b/test/ruby/test_exception.rb @@ -328,4 +328,10 @@ end.join def test_errno assert_equal(Encoding.find("locale"), Errno::EINVAL.new.message.encoding) end + + def test_too_many_args_in_eval + bug5720 = '[ruby-core:41520]' + arg_string = (0...140000).to_a.join(", ") + assert_raise(SystemStackError, bug5720) {eval "raise(#{arg_string})"} + end end diff --git a/vm.c b/vm.c index bf4d3c7756..e04a44b696 100644 --- a/vm.c +++ b/vm.c @@ -106,11 +106,10 @@ vm_set_top_stack(rb_thread_t * th, VALUE iseqval) /* for return */ rb_vm_set_finish_env(th); + CHECK_STACK_OVERFLOW(th->cfp, iseq->local_size + iseq->stack_max); vm_push_frame(th, iseq, VM_FRAME_MAGIC_TOP, th->top_self, 0, iseq->iseq_encoded, th->cfp->sp, 0, iseq->local_size); - - CHECK_STACK_OVERFLOW(th->cfp, iseq->stack_max); } static void @@ -122,6 +121,8 @@ vm_set_eval_stack(rb_thread_t * th, VALUE iseqval, const NODE *cref) /* for return */ rb_vm_set_finish_env(th); + + CHECK_STACK_OVERFLOW(th->cfp, iseq->local_size + iseq->stack_max); vm_push_frame(th, iseq, VM_FRAME_MAGIC_EVAL, block->self, GC_GUARDED_PTR(block->dfp), iseq->iseq_encoded, th->cfp->sp, block->lfp, iseq->local_size); @@ -129,8 +130,6 @@ vm_set_eval_stack(rb_thread_t * th, VALUE iseqval, const NODE *cref) if (cref) { th->cfp->dfp[-1] = (VALUE)cref; } - - CHECK_STACK_OVERFLOW(th->cfp, iseq->stack_max); } static void @@ -152,8 +151,6 @@ vm_set_main_stack(rb_thread_t *th, VALUE iseqval) if (bind && iseq->local_size > 0) { bind->env = rb_vm_make_env_object(th, th->cfp); } - - CHECK_STACK_OVERFLOW(th->cfp, iseq->stack_max); } rb_control_frame_t * -- cgit v1.2.3