Issue #13776 has been updated by nagachika (Tomoyuki Chikanaga).

Backport changed from 2.2: REQUIRED, 2.3: REQUIRED, 2.4: REQUIRED to 2.2: REQUIRED, 2.3: REQUIRED, 2.4: DONE

ruby_2_4 r61436 merged revision(s) 59444,59445.

----------------------------------------
Bug #13776: Refined methods ignore visibility of inherited methods
https://bugs.ruby-lang.org/issues/13776#change-68621

* Author: Mon_Ouie (Mon oue)
* Status: Closed
* Priority: Normal
* Assignee: 
* Target version: 
* ruby -v: ruby 2.5.0dev (2017-07-28 trunk 59438) [x86_64-linux]
* Backport: 2.2: REQUIRED, 2.3: REQUIRED, 2.4: DONE
----------------------------------------
This bug was pointed out by al2o3-cr on the #ruby IRC channel on freenode. The following code sleeps for 3 seconds, even though `String#sleep` is a private method.

```ruby
module Test
  refine String do
    def sleep; end
  end
end

puts "start"
"".sleep 3
puts "end"
```

This happens because `vm_call_method_each_type`, when running a method of type `VM_METHOD_TYPE_REFINED`, uses `vm_call_zsuper` to call inherited methods, which does not check the visibility of the method and directly executes it. I attached a patch which changes this code path and uses `vm_call_method` to trigger the code that honors method visibilities. I have not played extensively with this patch, but it at least does not seem to break any existing unit tests.

---Files--------------------------------
refinement.patch (1.74 KB)


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

Unsubscribe: <mailto:ruby-core-request / ruby-lang.org?subject=unsubscribe>
<http://lists.ruby-lang.org/cgi-bin/mailman/options/ruby-core>