diff options
-rw-r--r-- | ChangeLog | 9 | ||||
-rw-r--r-- | test/ruby/test_exception.rb | 6 | ||||
-rw-r--r-- | vm.c | 9 |
3 files changed, 18 insertions, 6 deletions
@@ -1,3 +1,12 @@ +Wed Dec 7 18:55:56 2011 Nobuyoshi Nakada <nobu@ruby-lang.org> + + * 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 <drbrain@segment7.net> * 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 @@ -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 * |