[8u-dev] Request for Approval: Bulk backport of 8180565, 8180617, 8180511, 8180576, 8180575 and 8180813

[David Buck]

Hi Tobias!

 > > The review is in progress:
 > > http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-> 
May/026371.html
 >
 > Update: The new webrev was reviewed.

approved for push to 8u-dev

In the future, please be sure to include links to the bug reports when 
you request approval, as required by the push request template [0].

Here are the links for this request:

https://bugs.openjdk.java.net/browse/JDK-8180565

https://bugs.openjdk.java.net/browse/JDK-8180617

https://bugs.openjdk.java.net/browse/JDK-8180511

https://bugs.openjdk.java.net/browse/JDK-8180576

https://bugs.openjdk.java.net/browse/JDK-8180575

https://bugs.openjdk.java.net/browse/JDK-8180813

Cheers,
-Buck

[0] http://openjdk.java.net/projects/jdk8u/approval-template.html

On 2017/05/30 0:59, Tobias Hartmann wrote:
> Hi,
> 
> please approve the backports of the following fixes to JDK 8u:
> 
> 8180565: Null pointer dereferences of ConstMethod::method()
> http://cr.openjdk.java.net/~thartmann/8180565/webrev.00/
> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/8f941bab493f
> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-May/026261.html
> 
> 8180617: Null pointer dereference in InitializeNode::complete_stores
> http://cr.openjdk.java.net/~thartmann/8180617/webrev.00/
> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/0b218e675429
> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-May/026275.html
> 
> 8180511: Null pointer dereference in Matcher::ReduceInst()
> http://cr.openjdk.java.net/~thartmann/8180511/webrev.00/
> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/1f917785fbe7
> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-May/026258.html
> 
> 8180576: Null pointer dereference in Matcher::xform()
> http://cr.openjdk.java.net/~thartmann/8180576/webrev.00/
> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/286c8e80795b
> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-May/026259.html
> 
> 8180575: Null pointer dereference in LoadNode::Identity()
> http://cr.openjdk.java.net/~thartmann/8180575/webrev.00/
> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/31c842513336
> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-May/026260.html
> 
> 8180813: Null pointer dereference of CodeCache::find_blob() result
> http://cr.openjdk.java.net/~thartmann/8180813/webrev.00/
> http://hg.openjdk.java.net/jdk9/dev/hotspot/rev/63ac6d565c21
> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-May/026307.html
> 
> The backport of 8180813 does not apply cleanly because "is_compiled" is called "is_nmethod" and "as_compiled_method_or_null" is called "as_nmethod_or_null" in JDK 8u.
> 
> Here's the new webrev for 8180813:
> http://cr.openjdk.java.net/~thartmann/8180813_8u/webrev.00/
> The review is in progress:
> http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2017-May/026371.html
> 
> All other backports apply cleanly. Tested with JPRT.
> 
> Thanks,
> Tobias
>