aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorthartmann <none@none>2017-07-19 09:00:13 +0200
committerAlexey Ushakov <Alexey.Ushakov@jetbrains.com>2017-10-11 12:35:20 +0300
commit83aefef81561883c18d03feb31c4ebbf3ea9fc96 (patch)
treeefdd9577eb66edd1a424cceafdae798ae2ca1cf4
parent59994167d3667b7452ceec1f9504578102cd3fdb (diff)
downloadjdk8u_hotspot-83aefef81561883c18d03feb31c4ebbf3ea9fc96.tar.gz
8184009: Missing null pointer check in InterpreterRuntime::update_mdp_for_ret()
Summary: Adds a missing null check (guarantee) found by Parfait. Reviewed-by: coleenp, shade
-rw-r--r--src/share/vm/interpreter/interpreterRuntime.cpp1
1 files changed, 1 insertions, 0 deletions
diff --git a/src/share/vm/interpreter/interpreterRuntime.cpp b/src/share/vm/interpreter/interpreterRuntime.cpp
index 25654646c..de928e949 100644
--- a/src/share/vm/interpreter/interpreterRuntime.cpp
+++ b/src/share/vm/interpreter/interpreterRuntime.cpp
@@ -979,6 +979,7 @@ IRT_ENTRY(void, InterpreterRuntime::update_mdp_for_ret(JavaThread* thread, int r
// ProfileData is essentially a wrapper around a derived oop, so we
// need to take the lock before making any ProfileData structures.
ProfileData* data = h_mdo->data_at(h_mdo->dp_to_di(fr.interpreter_frame_mdp()));
+ guarantee(data != NULL, "profile data must be valid");
RetData* rdata = data->as_RetData();
address new_mdp = rdata->fixup_ret(return_bci, h_mdo);
fr.interpreter_frame_set_mdp(new_mdp);