Bug #7635: debug_inspector API segfaults when opened from inside an eval frame

• Status: Closed
• Assignee: ko1 (Koichi Sasada)
• Target version: 2.0.0
• ruby -v: ruby 2.0.0dev (2012-12-29 trunk 38656) [x86_64-darwin11.4.0]

=begin

(({rb_debug_inspector_open})) segfaults when it is called from inside an eval frame.

Using this C extension:

#include "ruby/ruby.h"

static VALUE
cb()
{
return Qnil;
}

static VALUE
debug_inspector()
{
return rb_debug_inspector_open(cb, NULL);
}

void
Init_debug_inspector()
{
rb_define_global_function("debug_inspector", debug_inspector, 0);
}

Calling (({debug_inspector})) from the top level and from within methods works fine, calling inside eval from the top level works fine, but calling inside eval from within a method segfaults.

Here is a test program that segfaults:

require "./debug_inspector" # the c extension above

def x
eval "debug_inspector"
end

x

=end

[ko1 (Koichi Sasada)]

• Category set to core
• Status changed from Open to Assigned
• Assignee set to ko1 (Koichi Sasada)
• Priority changed from Normal to 5
• Target version set to 2.0.0

Ah.. I missed this ticket.

[ko1 (Koichi Sasada)]

• Status changed from Assigned to Closed
• % Done changed from 0 to 100

This issue was solved with changeset r38970.
Charlie, thank you for reporting this issue.
Your contribution to Ruby is greatly appreciated.
May Ruby be with you.

---

• vm_backtrace.c: fix issue of rb_debug_inspector_open().
  The order of making binding should be stack (frame) top to bottom.
  [Bug #7635]
  And also fix issue of collecting klass. Collecting klass is same
  as TracePoint#defined_class.
  (previous version, it returns T_ICLASS (internal objects).
• test/-ext-/debug/test_debug.rb: add a test.
• ext/-test-/debug/extconf.rb, init.c, inspector.c: ditto.
• vm_backtrace.c: remove magic number and add enum CALLER_BINDING_*.
• vm_backtrace.c, include/ruby/debug.h: add new C api (experimental)
  rb_debug_inspector_frame_self_get().
• vm.c, vm_core.h, vm_trace.c: move decl. of
  rb_vm_control_frame_id_and_class() and constify first parameter.