mirror of
https://github.com/ruby/ruby.git
synced 2022-11-09 12:17:21 -05:00
vm.c: fix `cfp consistency error' which occurs
when raising exception in bmethod call event * vm.c (invoke_bmethod): set FINISH flag just before calling vm_exec. [ruby-dev:50162] [Bug #13705] git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@59956 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
parent
13f5dcb9f2
commit
698f2db224
2 changed files with 23 additions and 1 deletions
|
@ -1741,4 +1741,25 @@ class TestSetTraceFunc < Test::Unit::TestCase
|
|||
tp_return_value(:f_break_in_rescue),
|
||||
'[Bug #13369]'
|
||||
end
|
||||
|
||||
def test_trace_point_raising_exception_in_bmethod_call
|
||||
bug13705 = '[ruby-dev:50162]'
|
||||
assert_normal_exit %q{
|
||||
define_method(:m) {}
|
||||
|
||||
tp = TracePoint.new(:call) do
|
||||
raise ''
|
||||
end
|
||||
|
||||
tap do
|
||||
tap do
|
||||
begin
|
||||
tp.enable
|
||||
m
|
||||
rescue
|
||||
end
|
||||
end
|
||||
end
|
||||
}, bug13705
|
||||
end
|
||||
end
|
||||
|
|
3
vm.c
3
vm.c
|
@ -990,7 +990,7 @@ invoke_bmethod(rb_thread_t *th, const rb_iseq_t *iseq, VALUE self, const struct
|
|||
int arg_size = iseq->body->param.size;
|
||||
VALUE ret;
|
||||
|
||||
vm_push_frame(th, iseq, type | VM_FRAME_FLAG_FINISH | VM_FRAME_FLAG_BMETHOD, self,
|
||||
vm_push_frame(th, iseq, type | VM_FRAME_FLAG_BMETHOD, self,
|
||||
VM_GUARDED_PREV_EP(captured->ep),
|
||||
(VALUE)me,
|
||||
iseq->body->iseq_encoded + opt_pc,
|
||||
|
@ -1000,6 +1000,7 @@ invoke_bmethod(rb_thread_t *th, const rb_iseq_t *iseq, VALUE self, const struct
|
|||
|
||||
RUBY_DTRACE_METHOD_ENTRY_HOOK(th, me->owner, me->def->original_id);
|
||||
EXEC_EVENT_HOOK(th, RUBY_EVENT_CALL, self, me->def->original_id, me->called_id, me->owner, Qnil);
|
||||
VM_ENV_FLAGS_SET(th->ec.cfp->ep, VM_FRAME_FLAG_FINISH);
|
||||
ret = vm_exec(th);
|
||||
EXEC_EVENT_HOOK(th, RUBY_EVENT_RETURN, self, me->def->original_id, me->called_id, me->owner, ret);
|
||||
RUBY_DTRACE_METHOD_RETURN_HOOK(th, me->owner, me->def->original_id);
|
||||
|
|
Loading…
Reference in a new issue