[8u-dev] Request for approval: 8163969: Cyclic interface initialization causes JVM crash

[George Triantafillou]

Hi Coleen,

Small typo in src/share/vm/oops/instanceKlass.cpp:

889   // Next, if C is a class rather than an interface, initialize it's 
super class and super

change to

889   // Next, if C is a class rather than an interface, initialize its 
super class and super

Otherwise, looks good.

-George

On 10/12/2016 5:10 PM, Coleen Phillimore wrote:
> Summary: Backport change to correct interface initialization.
>
> There were too many changes to instanceKlass.cpp for a clean 
> backport.  Also in JDK8, this corrects interface initialization to not 
> initialize the whole interface hierarchy if an interface, not class, 
> initializes initialization.  This is to correctly follow JLS 12.4.2 
> step 7.   I filed a compatibility request (in review) to document the 
> difference in behavior, which I believe will not be noticed.
>
> Tested with JPRT, including runtime jtreg lambda-features tests, and 
> JCK tests.
>
> open webrev at http://cr.openjdk.java.net/~coleenp/8163969.8.01/webrev
> bug link https://bugs.openjdk.java.net/browse/JDK-8163969.8
>
> Thanks,
> Coleen