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

Coleen Phillmore coleen.phillimore at oracle.com
Wed Dec 4 06:06:51 PST 2013


Ok.

On 12/4/2013 7:57 AM, harold seigel wrote:
> 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
>>
>



More information about the hotspot-runtime-dev mailing list