DBTGattCharacteristic::configNotificationIndicationImpl(..): Tolerate deleted native instance @ disablev2.1.14

2 files changed, 16 insertions, 1 deletions

diff --git a/java/jni/direct_bt/DBTGattCharacteristic.cxx b/java/jni/direct_bt/DBTGattCharacteristic.cxx
index f376618e..84483d6a 100644
--- a/java/jni/direct_bt/DBTGattCharacteristic.cxx
+++ b/java/jni/direct_bt/DBTGattCharacteristic.cxx
@@ -165,7 +165,15 @@ jboolean Java_direct_1bt_tinyb_DBTGattCharacteristic_writeValueImpl(JNIEnv *env,
 jboolean Java_direct_1bt_tinyb_DBTGattCharacteristic_configNotificationIndicationImpl(JNIEnv *env, jobject obj,
                         jboolean enableNotification, jboolean enableIndication, jbooleanArray jEnabledState) {
     try {
-        GATTCharacteristic *characteristic = getInstance<GATTCharacteristic>(env, obj);
+        GATTCharacteristic *characteristic = getInstanceUnchecked<GATTCharacteristic>(env, obj);
+        if( nullptr == characteristic ) {
+            if( !enableNotification && !enableIndication ) {
+                // OK to have native characteristic being shutdown @ disable
+                DBG_PRINT("Characteristic's native instance has been deleted");
+                return false;
+            }
+            throw IllegalStateException("Characteristic's native instance deleted", E_FILE_LINE);
+        }
         JavaGlobalObj::check(characteristic->getJavaObject(), E_FILE_LINE);
 
         if( nullptr == jEnabledState ) {
diff --git a/java/jni/helper_base.hpp b/java/jni/helper_base.hpp
index a1f8bbdc..18ae8679 100644
--- a/java/jni/helper_base.hpp
+++ b/java/jni/helper_base.hpp
@@ -116,6 +116,13 @@ T *getInstance(JNIEnv *env, jobject obj)
 }
 
 template <typename T>
+T *getInstanceUnchecked(JNIEnv *env, jobject obj)
+{
+    jlong instance = env->GetLongField(obj, getInstanceField(env, obj));
+    return reinterpret_cast<T *>(instance);
+}
+
+template <typename T>
 void setInstance(JNIEnv *env, jobject obj, T *t)
 {
     if (t == nullptr) {