Project

General

Profile

Actions

Bug #15720

closed

SystemStackError when referencing a refinement in a module that isn't used

Added by chocolateboy (Chocolate Boy) about 5 years ago. Updated over 4 years ago.

Status:
Closed
Target version:
-
ruby -v:
ruby 2.7.0dev (2019-03-21 trunk 67332) [x86_64-linux]
[ruby-core:91916]

Description

module M1
  refine Kernel do
    def foo
      'foo called!'
    end
  end
end

module M2
  refine Kernel do
    def bar
      'bar called!'
    end
  end
end

using M1

puts foo
puts bar

Expected:

foo called!
Traceback (most recent call last):
bug.rb:20:in `<main>': undefined local variable or method `bar' for main:Object (NameError)

Actual:

foo called!
Traceback (most recent call last):
bug.rb:20:in `<main>': stack level too deep (SystemStackError)

Platform: Linux (Arch)

ruby -v:

  • ruby 2.7.0dev (2019-03-21 trunk 67332) [x86_64-linux]
  • ruby 2.6.2p47 (2019-03-13 revision 67232) [x86_64-linux]
  • ruby 2.5.5p157 (2019-03-15 revision 67260) [x86_64-linux]

Files

bug.rb (188 Bytes) bug.rb chocolateboy (Chocolate Boy), 03/21/2019 08:07 PM
sse-unused-refinement.patch (1.85 KB) sse-unused-refinement.patch jeremyevans0 (Jeremy Evans), 06/10/2019 10:56 PM
Actions #1

Updated by chocolateboy (Chocolate Boy) about 5 years ago

  • Description updated (diff)
Actions #2

Updated by chocolateboy (Chocolate Boy) about 5 years ago

  • Description updated (diff)
Actions #3

Updated by chocolateboy (Chocolate Boy) about 5 years ago

  • Description updated (diff)

Updated by jeremyevans0 (Jeremy Evans) almost 5 years ago

I can confirm this issue. I believe it stems from the fact that in vm_call_method_each_type, cc->me is getting overwritten by ref_me even if they already have the same def but a different defined_class, which results in infinite recursion. The attached patch should fix your example, but my understanding of this code is limited. ko1 or shugo should probably review this patch and determine if it is the proper way to fix this issue. This could possibly be related to #14068.

Updated by nobu (Nobuyoshi Nakada) almost 5 years ago

Seems fine.
Just rename test_refining_module_repeatedly which redefines an existing method.

Actions #6

Updated by jeremyevans (Jeremy Evans) almost 5 years ago

  • Status changed from Assigned to Closed

Applied in changeset git|5e018214e7435030727a97ac49db038d96438e74.


Fix SystemStackError when calling a method in an unused refinement

Fixes [Bug #15720]

Actions #7

Updated by nagachika (Tomoyuki Chikanaga) almost 5 years ago

  • Backport changed from 2.4: UNKNOWN, 2.5: UNKNOWN, 2.6: UNKNOWN to 2.4: UNKNOWN, 2.5: REQUIRED, 2.6: REQUIRED

Updated by nagachika (Tomoyuki Chikanaga) over 4 years ago

  • Backport changed from 2.4: UNKNOWN, 2.5: REQUIRED, 2.6: REQUIRED to 2.4: UNKNOWN, 2.5: REQUIRED, 2.6: DONE

ruby_2_6 r67729 merged revision(s) 5e018214e7435030727a97ac49db038d96438e74.

Updated by usa (Usaku NAKAMURA) over 4 years ago

  • Backport changed from 2.4: UNKNOWN, 2.5: REQUIRED, 2.6: DONE to 2.4: UNKNOWN, 2.5: DONE, 2.6: DONE

ruby_2_5 r67765 merged revision(s) 5e018214e7435030727a97ac49db038d96438e74.

Actions

Also available in: Atom PDF

Like0
Like0Like0Like0Like0Like0Like0Like0Like0Like0