fix(liveness): release jweak ref when liveness table overflows or realloc fails (#556)

jbachorik · web-flow · commit b3f7d38c0a11 · 2026-05-29T08:19:36.000+02:00
diff --git a/ddprof-lib/src/main/cpp/livenessTracker.cpp b/ddprof-lib/src/main/cpp/livenessTracker.cpp
@@ -370,7 +370,10 @@ void LivenessTracker::track(JNIEnv *env, AllocEvent &event, jint tid,
       } else {
         Log::debug("Cannot add sampled object to Liveness tracking table, it's "
                    "overflowing");
+        env->DeleteWeakGlobalRef(ref);
       }
+    } else {
+      env->DeleteWeakGlobalRef(ref);
     }
   }
   skipped = 0; // reset the subsampling skipped bytes

Original file line number	Diff line number	Diff line change
`@@ -370,7 +370,10 @@ void LivenessTracker::track(JNIEnv *env, AllocEvent &event, jint tid,`
`370`	`370`	`} else {`
`371`	`371`	`Log::debug("Cannot add sampled object to Liveness tracking table, it's "`
`372`	`372`	`"overflowing");`
	`373`	`+ env->DeleteWeakGlobalRef(ref);`
`373`	`374`	`}`
	`375`	`+ } else {`
	`376`	`+ env->DeleteWeakGlobalRef(ref);`
`374`	`377`	`}`
`375`	`378`	`}`
`376`	`379`	`skipped = 0; // reset the subsampling skipped bytes`