Wed, 19 Jul 2017 09:00:13 +0200
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
src/share/vm/interpreter/interpreterRuntime.cpp | file | annotate | diff | comparison | revisions |
1.1 --- a/src/share/vm/interpreter/interpreterRuntime.cpp Thu Aug 10 12:08:50 2017 +0530 1.2 +++ b/src/share/vm/interpreter/interpreterRuntime.cpp Wed Jul 19 09:00:13 2017 +0200 1.3 @@ -979,6 +979,7 @@ 1.4 // ProfileData is essentially a wrapper around a derived oop, so we 1.5 // need to take the lock before making any ProfileData structures. 1.6 ProfileData* data = h_mdo->data_at(h_mdo->dp_to_di(fr.interpreter_frame_mdp())); 1.7 + guarantee(data != NULL, "profile data must be valid"); 1.8 RetData* rdata = data->as_RetData(); 1.9 address new_mdp = rdata->fixup_ret(return_bci, h_mdo); 1.10 fr.interpreter_frame_set_mdp(new_mdp);