RFR(XS) 8008811: [parfait] Null pointer deference in hotspot/src/share/vm/opto/loopopts.cpp

Vladimir Kozlov vladimir.kozlov at oracle.com
Tue Mar 5 15:07:44 PST 2013

In loopnode.hpp reverse check to != NULL so that real code will be first.

In loopnode.cpp check dp != NULL because dp->as_Proj() is reference 
through dp. as_Proj() is cast.

bolphi check should be before replace_node() call.

I don't understand why you need next check:
if (reg == NULL) return NULL;

insert_region_before_proj() returns "reg = new (C)RegionNode(2)" so it 
can't be NULL.


On 3/5/13 2:22 PM, Morris Meyer wrote:
> Folks,
> Could I get a review for this parfait issue? This has been through JPRT.
> Thanks much,
>          --mm
> WEBREV - http://cr.openjdk.java.net/~morris/8008811.01
> BUG - https://jbs.oracle.com/bugs/browse/JDK-8008811

More information about the hotspot-compiler-dev mailing list