Mercurial Mercurial > jdk-sandbox / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
8180575: Null pointer dereference in LoadNode::Identity()
authorthartmann
Mon, 22 May 2017 09:23:59 +0200
changeset 45244 24df46d012d5
parent 45243 a66cea8a8f83
child 45245 48fed68b35c1
8180575: Null pointer dereference in LoadNode::Identity() Summary: Fixed a missing null check on the return value of AddPNode::Ideal_base_and_offset() found by Parfait. Reviewed-by: kvn
hotspot/src/share/vm/opto/memnode.cpp file | annotate | diff | comparison | revisions 
--- a/hotspot/src/share/vm/opto/memnode.cpp	Mon May 22 09:18:12 2017 +0200
+++ b/hotspot/src/share/vm/opto/memnode.cpp	Mon May 22 09:23:59 2017 +0200
@@ -1123,6 +1123,9 @@
       // Use _idx of address base (could be Phi node) for boxed values.
       intptr_t   ignore = 0;
       Node*      base = AddPNode::Ideal_base_and_offset(in(Address), phase, ignore);
+      if (base == NULL) {
+        return this;
+      }
       this_iid = base->_idx;
     }
     const Type* this_type = bottom_type();
jdk-sandbox