From dabb4ab518228fa87ee805f0af8cbceaf2bb455e Mon Sep 17 00:00:00 2001 From: Tobias Hartmann Date: Wed, 19 Jul 2017 09:00:13 +0200 Subject: [PATCH] 8184009: Missing null pointer check in InterpreterRuntime::update_mdp_for_ret() Adds a missing null check (guarantee) found by Parfait. Reviewed-by: coleenp, shade --- hotspot/src/share/vm/interpreter/interpreterRuntime.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/hotspot/src/share/vm/interpreter/interpreterRuntime.cpp b/hotspot/src/share/vm/interpreter/interpreterRuntime.cpp index aaad8042cc9..6c1f627ac09 100644 --- a/hotspot/src/share/vm/interpreter/interpreterRuntime.cpp +++ b/hotspot/src/share/vm/interpreter/interpreterRuntime.cpp @@ -1033,6 +1033,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);