(11) RFR (S) JDK-8196601: IllegalAccessError: cannot access class jdk.jfr.internal.handlers.EventHandler

[Lois Foltan]

Thank you Karen!
Lois

On 2/5/2018 3:43 PM, Karen Kinnear wrote:
> Lois,
>
> Looks good. Thank you for doing this. With reflection and internal signature handling, the backward compatibility
> model is that we do not perform the access check when returning a reflection field or reflection method or performing
> internal signature processing. The access checking is done when actually performing the access of
> the reflected field or method etc. So I believe this should match the backward compatibility requirements.
>
> thank you so much,
> Karen
>
>> On Feb 5, 2018, at 3:04 PM, Paul Sandoz <paul.sandoz at oracle.com> wrote:
>>
>> Hi Lois,
>>
>> Looks good. I eyeballed your changes to ensure they reverted those recent changes.
>>
>> Paul.
>>
>>> On Feb 5, 2018, at 11:51 AM, Lois Foltan <lois.foltan at oracle.com> wrote:
>>>
>>> Please review this partial back out of a recent change that caused incorrect class accessibility checks to occur in the case of calls to java/lang/Class::getDeclaredFields as well as within JVM signature stream processing.  Testing exposed the issue where a call to getDeclaredFields resulted incorrectly in an IllegalAccessError.
>>>
>>> open webrev at http://cr.openjdk.java.net/~lfoltan/bug_jdk8196601/webrev/
>>> bug link https://bugs.openjdk.java.net/browse/JDK-8196601
>>>
>>> Testing in progress (hs-tier1-5,jdk-tier1,2,3)
>>>
>>> Thanks,
>>> Lois