diff options
author | Jeremy Evans <code@jeremyevans.net> | 2022-02-17 10:24:01 -0800 |
---|---|---|
committer | Jeremy Evans <code@jeremyevans.net> | 2022-03-24 12:31:07 -0700 |
commit | 343ea9967e4a6b279eed6bd8e81ad0bdc747f254 (patch) | |
tree | 96c0a77db135ec1fdd374144e0e00238e2dfc1c3 /spec/ruby/optional | |
parent | 33b13bd9f19ac806c34d428af49a71c1aa286f7e (diff) | |
download | ruby-343ea9967e4a6b279eed6bd8e81ad0bdc747f254.tar.gz |
Raise RuntimeError if Kernel#binding is called from a non-Ruby frame
Check whether the current or previous frame is a Ruby frame in
call_trace_func before attempting to create a binding for the frame.
Fixes [Bug #18487]
Co-authored-by: Alan Wu <XrXr@users.noreply.github.com>
Diffstat (limited to 'spec/ruby/optional')
-rw-r--r-- | spec/ruby/optional/capi/binding_spec.rb | 21 |
1 files changed, 15 insertions, 6 deletions
diff --git a/spec/ruby/optional/capi/binding_spec.rb b/spec/ruby/optional/capi/binding_spec.rb index 966d650c46..2165705457 100644 --- a/spec/ruby/optional/capi/binding_spec.rb +++ b/spec/ruby/optional/capi/binding_spec.rb @@ -8,12 +8,21 @@ describe "CApiBindingSpecs" do end describe "Kernel#binding" do - it "gives the top-most Ruby binding when called from C" do - foo = 14 - b = @b.get_binding - b.local_variable_get(:foo).should == 14 - b.local_variable_set :foo, 12 - foo.should == 12 + ruby_version_is '3.2' do + it "raises when called from C" do + foo = 14 + -> { @b.get_binding }.should raise_error(RuntimeError) + end + end + + ruby_version_is ''...'3.2' do + it "gives the top-most Ruby binding when called from C" do + foo = 14 + b = @b.get_binding + b.local_variable_get(:foo).should == 14 + b.local_variable_set :foo, 12 + foo.should == 12 + end end end end |