commit 52dda598963cefc82cfc5e55342ff631a07fd6d4
author Andrew Rossignol <aarossig@google.com> Tue Aug 09 10:54:38 2016 -0700
committer Andrew Rossignol <aarossig@google.com> Tue Aug 09 11:47:09 2016 -0700
tree e97df002875209f91874112734e2a4eae844ccda
parent 4a5d23596afe03d7ae3e7501cd32afa5d7eca8a2

ContextHub: Avoid reference leaks

The ContextHub HAL was leaking references in passOnOsResponse. An
example of this is when a sensor hub reset occurs. If many of these
resets happen, the local reference pool is exhausted and a crash
in the HAL occurs.

Bug: 30706325
Change-Id: Ica5328137a7033f512d7fb1ccf6fa138eab55091

core/jni/android_hardware_location_ContextHubService.cpp

diff --git a/core/jni/android_hardware_location_ContextHubService.cpp b/core/jni/android_hardware_location_ContextHubService.cpp
index 9515a0e..7a8a574 100644
--- a/core/jni/android_hardware_location_ContextHubService.cpp
+++ b/core/jni/android_hardware_location_ContextHubService.cpp
@@ -603,6 +603,8 @@
     env->CallIntMethod(db.jniInfo.jContextHubService,
                        db.jniInfo.contextHubServiceMsgReceiptCallback,
                        jheader, jmsg);
+    env->DeleteLocalRef(jmsg);
+    env->DeleteLocalRef(jheader);
 
     delete[] msg;
 }