[9] RFR(XS): 8180565: Null pointer dereferences of ConstMethod::method()

Igor Ignatyev igor.ignatyev at oracle.com
Fri May 19 21:10:08 UTC 2017


Hi Tobias,

in ConstMethod::print_on, don't we need st->cr() when 'm' is null?

Thanks,
-- Igor
> On May 18, 2017, at 11:16 PM, Tobias Hartmann <tobias.hartmann at oracle.com> wrote:
> 
> Hi Vladimir,
> 
> thanks for the review!
> 
> Best regards,
> Tobias
> 
> On 18.05.2017 19:10, Vladimir Kozlov wrote:
>> Good.
>> 
>> Vladimir
>> 
>> 
>> On 5/18/17 7:08 AM, Tobias Hartmann wrote:
>>> Hi,
>>> 
>>> please review the following patch:
>>> https://bugs.openjdk.java.net/browse/JDK-8180565
>>> http://cr.openjdk.java.net/~thartmann/8180565/webrev.00/
>>> 
>>> ConstMethod::method() returns _constants->pool_holder()->method_with_idnum(_method_idnum) which may be NULL. We need to check for NULL before dereferencing.
>>> 
>>> Thanks,
>>> Tobias
>>> 



More information about the hotspot-compiler-dev mailing list