[#58149] [ruby-trunk - Feature #9076][Open] New one-argument block syntax: &. — "asterite (Ary Borenszweig)" <ary@...>

23 messages 2013/11/04

[#58176] [ruby-trunk - Bug #9082][Open] popen3 hangs when stderr gets lots of output — "rosenfeld (Rodrigo Rosenfeld Rosas)" <rr.rosas@...>

15 messages 2013/11/05

[#58207] [ruby-trunk - Bug #9089][Open] rb_fix2uint no longer raises a RangeError when given negative values — "NoKarma (Arthur Schreiber)" <schreiber.arthur@...>

9 messages 2013/11/06

[#58243] [ruby-trunk - Feature #9098][Open] Indent heredoc against the left margin by default when "indented closing identifier" is turned on. — "sikachu (Prem Sichanugrist)" <s@...>

24 messages 2013/11/09

[#58306] [ruby-trunk - Bug #9106][Open] 'gem install' doesn't copy .so files of ext libs — "tagomoris (Satoshi TAGOMORI)" <tagomoris@...>

15 messages 2013/11/13

[#58324] [ruby-trunk - Feature #9108][Open] Hash sub-selections — "wardrop (Tom Wardrop)" <tom@...>

28 messages 2013/11/14

[#58342] [ruby-trunk - Feature #9112][Open] Make module lookup more dynamic (Including modules into a module after it has already been included) — "PragTob (Tobias Pfeiffer)" <pragtob@...>

16 messages 2013/11/14

[#58350] [ruby-trunk - Feature #9113][Open] Ship Ruby for Linux with jemalloc out-of-the-box — "sam.saffron (Sam Saffron)" <sam.saffron@...>

59 messages 2013/11/15

[#58374] [ruby-trunk - Bug #9115][Open] Logger traps all exceptions; breaks Timeout — "cphoenix (Chris Phoenix)" <cphoenix@...>

10 messages 2013/11/16

[#58375] [ruby-trunk - Feature #9116][Open] String#rsplit missing — "artagnon (Ramkumar Ramachandra)" <artagnon@...>

12 messages 2013/11/16

[#58396] [ruby-trunk - Bug #9121][Open] [PATCH] Remove rbtree implementation of SortedSet due to performance regression — "xshay (Xavier Shay)" <contact@...>

15 messages 2013/11/18

[#58404] [ruby-trunk - Feature #9123][Open] Make Numeric#nonzero? behavior consistent with Numeric#zero? — "sferik (Erik Michaels-Ober)" <sferik@...>

40 messages 2013/11/18

[#58411] [ruby-trunk - Bug #9124][Open] TestSocket errors in test-all on Arch 64-bit — "jonforums (Jon Forums)" <redmine@...>

14 messages 2013/11/18

[#58438] [ruby-trunk - Bug #9129][Open] Regression in support for IPv6 literals in URIs with Net::HTTP — "kallistec (Daniel DeLeo)" <dan@...>

11 messages 2013/11/19

[#58545] [ruby-trunk - Feature #9145][Open] Queue#pop(true) return nil if empty instead of raising ThreadError — "jsc (Justin Collins)" <redmine@...>

9 messages 2013/11/24

[#58653] [ruby-trunk - Bug #9170][Open] Math.sqrt returns different types when mathn is included; breaks various gems - this bug can be reproduced in Ruby 1.8 as well — "kranzky (Jason Hutchens)" <JasonHutchens@...>

7 messages 2013/11/28

[ruby-core:58250] [ruby-trunk - Bug #9009] Wrong binding when tracing c-calls

From: deivid (David Rodríguez) <deivid.rodriguez@...>
Date: 2013-11-10 15:47:39 UTC
List: ruby-core #58250
Issue #9009 has been updated by deivid (David Rodr鱈guez).


=begin
Hi, I've translate your example to use the TracePoint API

This is the example and result:

    class IO
      def some_method
      end
    end

    puts "true receiver is #{$stdout}\n\n"

    TracePoint.trace(:c_call, :call) do |tp|
      puts "#{tp.event} #{tp.defined_class} #{tp.method_id}"
      puts "  apparent receiver = #{tp.self} vs #{tp.binding.eval('self')}"
      Thread.current.backtrace_locations(2).each { |loc| puts "    #{loc}" }
    end

    $stdout.write ""
    $stdout.some_method


results in

    true receiver is #<IO:0x8bf4e58>

    c_call IO write
      apparent receiver = #<IO:0x8bf4e58> vs main
        test.rb:15:in `<main>'
    call IO some_method
      apparent receiver = #<IO:0x8bf4e58> vs #<IO:0x8bf4e58>
        test.rb:2:in `some_method'
        test.rb:16:in `<main>'

My comments:

 * Seems like the new way to get the receiver is the (({#self})) instance method of the (({TracePoint})) class. However, the documentation states "(({Same as #binding: trace.binding.eval('self')}))", so that should probably be corrected or further explained.

* Regarding c-frames not being pushed onto the frame stack, they actually are, but after the Tracepoint/set_trace_func event. I made the same mistake here: #8538. Have a look at source:vm_insnhelper.c#L1514 and observe how (({vm_push_frame})) is called after (({EXEC_EVENT_HOOK})).

 * I still think the behaviour should be consistent and when inside a TracePoint event the current frame should either be already in the backtrace or not, but not behave differently for c methods and ruby methods.

=end
----------------------------------------
Bug #9009: Wrong binding when tracing c-calls
https://bugs.ruby-lang.org/issues/9009#change-42841

Author: yekka (Nikolay Bozhenov)
Status: Open
Priority: Normal
Assignee: 
Category: 
Target version: 
ruby -v: ruby 2.1.0dev (2013-10-04 trunk 43141) [x86_64-linux]
Backport: 1.9.3: UNKNOWN, 2.0.0: UNKNOWN


When I use set_trace_func to trace ruby code, I get a wrong binding in
case of c-calls. In this case binding.eval("self") is not the receiver
of the call. Whereas in case of ruby-calls binding.eval("self") yields
the receiver of the call.
The underlying problem is that c-calls aren't pushed onto the frame stack.
It seems that currently there is no way to find out the receiver of c-call
inside tracing function.

Example of code:

$ cat test.rb
class IO
    def some_method
    end
end

puts "true receiver is #{$stdout}\n\n"

set_trace_func proc { |event, file, line, id, binding, classname|
    if event == "call" or event == "c-call"
        puts "#{event} #{id}:"
        puts "\tapparent receiver = #{binding.eval("self")}"
        puts "\tbacktrace:"
        caller.each { |l| puts "\t\t#{l}" }
        puts
    end
}

$stdout.write ""      # c-call
$stdout.some_method   # ruby-call


Execution:
  
$ ruby test.rb 
true receiver is #<IO:0x00000000bed2a0>

c-call write:
        apparent receiver = main
        backtrace:
                test.rb:18:in `<main>'

call some_method:
        apparent receiver = #<IO:0x00000000bed2a0>
        backtrace:
                test.rb:2:in `some_method'
                test.rb:19:in `<main>'


Expected result:                

true receiver is #<IO:0x00000000bed2a0>

c-call write:
        apparent receiver = #<IO:0x00000000bed2a0>
        backtrace:
                somewhere:in `write'
                test.rb:18:in `<main>'

call some_method:
        apparent receiver = #<IO:0x00000000bed2a0>
        backtrace:
                test.rb:2:in `some_method'
                test.rb:19:in `<main>'



-- 
http://bugs.ruby-lang.org/

In This Thread

Prev Next