RFR 8174092: Remove array-related access checks from Reflection::verify_class_access()

[harold seigel]

Thanks George!

Harold


On 2/9/2017 10:00 AM, George Triantafillou wrote:
> Hi Harold,
>
> Your changes look good.
>
> -George
>
> On 2/9/2017 9:49 AM, harold seigel wrote:
>> Hi,
>>
>> Please review this JDK-10 fix for bug JDK-8174092.  The new_class 
>> parameters passed to Reflection::verify_class_access() and 
>> verify_class_access_msg() are always going to be of type 
>> InstanceKlass*.  So, this fix changes its type to InstanceKlass* and 
>> removes code that checks for other Klass* types.
>>
>> Note that the InstanceKlass::cast(Klass*) function used by this fix 
>> will assert if the Klass* is not an InstanceKlass*.  So, no 
>> additional asserts that a Klass* is an InstanceKlass* are needed.
>>
>> Open webrev: 
>> http://cr.openjdk.java.net/~hseigel/bug_8174092/webrev/index.html
>>
>> JBS Bug: https://bugs.openjdk.java.net/browse/JDK-8174092
>>
>> The fix was tested with the JCK Lang and vm tests, the JTreg hotspot, 
>> java/io, java/lang, and java/util tests, the nsk colocated and the 
>> non-colocated tests, and with JPRT.
>>
>> Thanks, Harold
>>
>