8184009: Missing null pointer check in InterpreterRuntime::update_mdp_for_ret()

Wed, 19 Jul 2017 09:00:13 +0200

author
thartmann
date
Wed, 19 Jul 2017 09:00:13 +0200
changeset 8885
1832e44a9889
parent 8884
35fe0be5277b
child 8886
fbb8f75498f4

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);

mercurial