Skip to content
This repository was archived by the owner on Jan 4, 2023. It is now read-only.

Commit 7f2a22f

Browse files
ptzafrirpetreeftime
authored andcommitted
Fix issue #121
boolean_cls should only be unreferenced after it has been used and not before. Signed-off-by Tzafrir Poupko <[email protected]>
1 parent b5e293b commit 7f2a22f

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

java/jni/BluetoothAdapter.cxx

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -322,9 +322,9 @@ void Java_tinyb_BluetoothAdapter_enablePoweredNotifications(JNIEnv *env, jobject
322322

323323
jclass boolean_cls = search_class(*jni_env, "java/lang/Boolean");
324324
jmethodID constructor = search_method(*jni_env, boolean_cls, "<init>", "(Z)V", false);
325-
jni_env->DeleteLocalRef(boolean_cls);
326325

327326
jobject result = jni_env->NewObject(boolean_cls, constructor, v ? JNI_TRUE : JNI_FALSE);
327+
jni_env->DeleteLocalRef(boolean_cls);
328328

329329
jni_env->CallVoidMethod(**callback_ptr, method, result);
330330
jni_env->DeleteLocalRef(result);
@@ -416,9 +416,9 @@ void Java_tinyb_BluetoothAdapter_enableDiscoverableNotifications(JNIEnv *env, jo
416416

417417
jclass boolean_cls = search_class(*jni_env, "java/lang/Boolean");
418418
jmethodID constructor = search_method(*jni_env, boolean_cls, "<init>", "(Z)V", false);
419-
jni_env->DeleteLocalRef(boolean_cls);
420419

421420
jobject result = jni_env->NewObject(boolean_cls, constructor, v ? JNI_TRUE : JNI_FALSE);
421+
jni_env->DeleteLocalRef(boolean_cls);
422422

423423
jni_env->CallVoidMethod(**callback_ptr, method, result);
424424
jni_env->DeleteLocalRef(result);
@@ -533,9 +533,9 @@ void Java_tinyb_BluetoothAdapter_enablePairableNotifications(JNIEnv *env, jobjec
533533

534534
jclass boolean_cls = search_class(*jni_env, "java/lang/Boolean");
535535
jmethodID constructor = search_method(*jni_env, boolean_cls, "<init>", "(Z)V", false);
536-
jni_env->DeleteLocalRef(boolean_cls);
537536

538537
jobject result = jni_env->NewObject(boolean_cls, constructor, v ? JNI_TRUE : JNI_FALSE);
538+
jni_env->DeleteLocalRef(boolean_cls);
539539

540540
jni_env->CallVoidMethod(**callback_ptr, method, result);
541541
jni_env->DeleteLocalRef(result);
@@ -670,9 +670,9 @@ void Java_tinyb_BluetoothAdapter_enableDiscoveringNotifications(JNIEnv *env, job
670670

671671
jclass boolean_cls = search_class(*jni_env, "java/lang/Boolean");
672672
jmethodID constructor = search_method(*jni_env, boolean_cls, "<init>", "(Z)V", false);
673-
jni_env->DeleteLocalRef(boolean_cls);
674673

675674
jobject result = jni_env->NewObject(boolean_cls, constructor, v ? JNI_TRUE : JNI_FALSE);
675+
jni_env->DeleteLocalRef(boolean_cls);
676676

677677
jni_env->CallVoidMethod(**callback_ptr, method, result);
678678
jni_env->DeleteLocalRef(result);

0 commit comments

Comments
 (0)