Skip to content

Commit

Permalink
Add shouldPostEvent check for JVMTI FramePop event
Browse files Browse the repository at this point in the history
The shouldPostEvent check was missed for the JVMTI FramePop event
in eclipse-openj9#16654.

Signed-off-by: Gengchen Tuo <gengchen.tuo@ibm.com>
  • Loading branch information
thallium committed Jun 16, 2023
1 parent 83a7f81 commit 33051de
Showing 1 changed file with 6 additions and 7 deletions.
13 changes: 6 additions & 7 deletions runtime/jvmti/jvmtiHook.c
Original file line number Diff line number Diff line change
Expand Up @@ -2046,25 +2046,24 @@ jvmtiHookMonitorWaited(J9HookInterface** hook, UDATA eventNum, void* eventData,
static void
jvmtiHookFramePop(J9HookInterface** hook, UDATA eventNum, void* eventData, void* userData)
{
J9JVMTIEnv * j9env = userData;
J9VMFramePopEvent * data = eventData;
J9JVMTIEnv *j9env = userData;
J9VMFramePopEvent *data = eventData;
jvmtiEventFramePop callback = j9env->callbacks.FramePop;
J9VMThread *currentThread = data->currentThread;
J9Method *method = data->method;

Trc_JVMTI_jvmtiHookFramePop_Entry();

ENSURE_EVENT_PHASE_LIVE(jvmtiHookFramePop, j9env);

/* Call the event callback */

if (callback != NULL) {
J9VMThread * currentThread= data->currentThread;
if ((NULL != callback) && shouldPostEvent(currentThread, method)) {
jthread threadRef;
UDATA hadVMAccess;
UDATA javaOffloadOldState = 0;

if (prepareForEvent(j9env, currentThread, currentThread, JVMTI_EVENT_FRAME_POP, &threadRef, &hadVMAccess, TRUE, 0, &javaOffloadOldState)) {
J9JavaVM * vm = currentThread->javaVM;
J9Method * method= data->method;
J9JavaVM *vm = currentThread->javaVM;
jmethodID methodID;

methodID = getCurrentMethodID(currentThread, method);
Expand Down

0 comments on commit 33051de

Please sign in to comment.