RFR (S) 8031045: Access checks should precede additional per-instruction checks

frederic parain frederic.parain at oracle.com
Wed Jan 22 05:20:23 PST 2014


Looks good to me.

Fred

On 22/01/2014 14:08, harold seigel wrote:
> Hi,
>
> Please review this straightforward backport of this small fix for bug
> 8031045 to JDK-8u.  The fix changes the implementation so that it does
> an access check before doing the non-static method ICCE error check.
> This ensures that if the method is both not accessible and not static,
> that an IllegalAccessError exception will get throw.  The fix is the
> same for both JDK-9 and JDK-8u.
>
> This fix was tested with the test in the bug report, the JCK lang and vm
> tests, the defmeth tests, the ute mlvm and quick tests, and the hotspot,
> java util, and java lang jtreg tests.
>
> Open webrev at: http://cr.openjdk.java.net/~hseigel/bug_8031045_8u/
> <http://cr.openjdk.java.net/%7Ehseigel/bug_8031045_8u/>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8031045
>
> Thanks! Harold

-- 
Frederic Parain - Oracle
Grenoble Engineering Center - France
Phone: +33 4 76 18 81 17
Email: Frederic.Parain at oracle.com


More information about the hotspot-runtime-dev mailing list