Re: Patch for JDK-8142882: rebinding of a DirectMethodHandle

2015-11-12 Thread Andrew Hughes
- Original Message - > Proposed fix for: https://bugs.openjdk.java.net/browse/JDK-8142882 > FWIW, the bug isn't visible. > http://cr.openjdk.java.net/~rasbold/8142882/webrev.00/ > > Don't pre-dispatch the receiver of a DirectMethodHandle, as it may allow a > protected method to be acces

Re: Patch for JDK-8142882: rebinding of a DirectMethodHandle

2015-11-12 Thread John Rose
Yes, it is safe to remove those overrides from DMH. You can count me as a reviewer. I like the test; it should be forward-ported to 9 if we don't already have something like it. — John > On Nov 12, 2015, at 11:46 AM, Chuck Rasbold wrote: > > Proposed fix for: https://bugs.openjdk.java.net/browse

Patch for JDK-8142882: rebinding of a DirectMethodHandle

2015-11-12 Thread Chuck Rasbold
Proposed fix for: https://bugs.openjdk.java.net/browse/JDK-8142882 http://cr.openjdk.java.net/~rasbold/8142882/webrev.00/ Don't pre-dispatch the receiver of a DirectMethodHandle, as it may allow a protected method to be accessed. This fix is a subset of the change for JDK-8050166