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

[harold seigel]

Hi Coleen,

Thanks for the review.  I did not find a test for the situation you suggest.

Harold

On 12/3/2013 3:57 PM, Coleen Phillimore wrote:
>
> 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
>