RFR (XS) 8027458: VM anonymous classes: wrong context for protected access checks

Coleen Phillimore coleen.phillimore at oracle.com
Tue Dec 3 12:57:56 PST 2013


Yes, this looks good.  I didn't know that anonymous classes could have 
multiple levels of host classes.  Did you find a test for that?
thanks,
Coleen

On 12/02/2013 04:42 PM, harold seigel wrote:
> Hi,
>
> Please review this small fix for bug 8027458.  It is based on John 
> Rose's suggested fix that he attached to the bug report.  For 
> protected access, the fix checks if the current class is an anonymous 
> class.  If so, it finds the current class's host class and uses the 
> host class for checking access to a protected method.
>
> webrev: http://cr.openjdk.java.net/~hseigel/bug_8027458/
> <http://cr.openjdk.java.net/%7Ehseigel/bug_8027458/>
> bug: https://bugs.openjdk.java.net/browse/JDK-8027458
>
> The fix was tested with JCK Lang and VM tests, the failing JCK tests 
> listed in the bug report, jtreg java/lang, java/util, lambda, and 
> runtime tests, ute quicktests and vm.mlvm.tests, and defmeth tests.
>
> Thanks! Harold



More information about the hotspot-runtime-dev mailing list