# HG changeset patch # User thartmann # Date 1500447613 -7200 # Node ID 1832e44a9889240e06c4b1a84b0eb5b6f97f81f2 # Parent 35fe0be5277b749b66725bfbdbdbcf4cb7bf4915 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 diff -r 35fe0be5277b -r 1832e44a9889 src/share/vm/interpreter/interpreterRuntime.cpp --- a/src/share/vm/interpreter/interpreterRuntime.cpp Thu Aug 10 12:08:50 2017 +0530 +++ b/src/share/vm/interpreter/interpreterRuntime.cpp Wed Jul 19 09:00:13 2017 +0200 @@ -979,6 +979,7 @@ // 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);