Quantcast

[9] RFR(XS): 8180617: Null pointer dereference in InitializeNode::complete_stores

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[9] RFR(XS): 8180617: Null pointer dereference in InitializeNode::complete_stores

Tobias Hartmann-2
Hi,

please review the following patch:
https://bugs.openjdk.java.net/browse/JDK-8180617
http://cr.openjdk.java.net/~thartmann/8180617/webrev.00/

Fixed a missing null check on the return value of InitializeNode::allocation() found by Parfait.

Thanks,
Tobias
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [9] RFR(XS): 8180617: Null pointer dereference in InitializeNode::complete_stores

Zoltán Majó
Hi Tobias,

the fix looks good to me. Thank you!

Best regards,

Zoltan

On 05/19/2017 09:27 AM, Tobias Hartmann wrote:

> Hi,
>
> please review the following patch:
> https://bugs.openjdk.java.net/browse/JDK-8180617
> http://cr.openjdk.java.net/~thartmann/8180617/webrev.00/
>
> Fixed a missing null check on the return value of InitializeNode::allocation() found by Parfait.
>
> Thanks,
> Tobias

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: [9] RFR(XS): 8180617: Null pointer dereference in InitializeNode::complete_stores

Tobias Hartmann-2
Hi Zoltan,

thanks for the review!

Best regards,
Tobias

On 19.05.2017 10:30, Zoltán Majó wrote:

> Hi Tobias,
>
> the fix looks good to me. Thank you!
>
> Best regards,
>
> Zoltan
>
> On 05/19/2017 09:27 AM, Tobias Hartmann wrote:
>> Hi,
>>
>> please review the following patch:
>> https://bugs.openjdk.java.net/browse/JDK-8180617
>> http://cr.openjdk.java.net/~thartmann/8180617/webrev.00/
>>
>> Fixed a missing null check on the return value of InitializeNode::allocation() found by Parfait.
>>
>> Thanks,
>> Tobias
>
Loading...