Request for reviews (S): 7077439: Possible reference through NULL in loopPredicate.cpp:726

[Vladimir Kozlov]

Thank you, Tom

Vladimir

Tom Rodriguez wrote:
> Looks good.
> 
> tom
> 
> On Aug 10, 2011, at 12:47 PM, Vladimir Kozlov wrote:
> 
>> http://cr.openjdk.java.net/~kvn/7077439/webrev
>>
>> Fixed 7077439: Possible reference through NULL in loopPredicate.cpp:726
>>
>> VM crashed at the next line because cl->loopexit() == NULL when I tried to port 7070134 into previous Hotspot sources:
>>
>>    BoolTest::mask bt = cl->loopexit()->test_trip();
>>
>> I did not see such crush with latest HS22 sources but it does not mean it can't happen. The check cl->is_valid_counted_loop() should be used in the code to avoid such crush. Note, this check is superset of cl->stride_is_con() so the later could be replaced.
>>
>