diff options
Diffstat (limited to 'java/jni')
| -rw-r--r-- | java/jni/direct_bt/DBTGattCharacteristic.cxx | 10 | ||||
| -rw-r--r-- | java/jni/helper_base.hpp | 7 |
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) { |