11

I made hello world application from book Android apps for Absolute Beginners and Temperature Convertor app from here Both is running fine on Emulator but when I try to run it on Samsung Note 2 following error is coming on LogCat

02-08 07:22:18.665: E/dalvikvm(30944): JNI ERROR (app bug): accessed stale local reference 0xbc00021 (index 8 in a table of size 8)
02-08 07:22:18.665: E/dalvikvm(30944): VM aborting
02-08 07:22:18.665: A/libc(30944): Fatal signal 11 (SIGSEGV) at 0xdeadd00d (code=1), thread 30944 (oid.temperature)

Both applications do open shows layout with title but do not shows any other views in layout

Samples runs fine

device: note 2 Samsung-gt_n7100

IDE:Eclipse version 3.8

OS: 64bit Windows 7

Community
  • 1
  • 1
Mehmood Hassan
  • 111
  • 1
  • 2
  • 8

1 Answers1

66

Since android 4.0 garbage collector was changed. Now it moves object around during garbage collection, which can cause a lot of problems.

Imagine that you have a static variable pointing to an object, and then this object gets moved by gc. Since android uses direct pointers for java objects, this would mean that your static variable is now pointing to a random address in the memory, unoccupied by any object or occupied by an object of different sort. This will almost guarantee that you'll get EXC_BAD_ACCESS next time you use this variable.

So android gives you JNI ERROR (app bug) error to prevent you from getting undebugable EXC_BAD_ACCESS. Now there are two ways to avoid this error.

  1. You can set targetSdkVersion in your manifest to version 11 or less. This will enable JNI bug compatibility mode and prevent any problems altogether. This is the reason why your old examples are working.

  2. You can avoid using static variables pointing to java objects or make jobject references global before storing them by calling env->NewGlobalRef(ref).
    Perhaps on of the biggest examples here is keeping jclass objects. Normally, you'll initialize static jclass variable during JNI_OnLoad, since class objects remain in the memory as long as the application is running.

This code will lead to a crash:

static jclass myClass;

JNIEXPORT jint JNICALL JNI_OnLoad (JavaVM * vm, void * reserved) {  
    myClass = env->FindClass("com/example/company/MyClass");  
    return JNI_VERSION_1_6;  
}

While this code will run fine:

static jclass myClass;

JNIEXPORT jint JNICALL JNI_OnLoad (JavaVM * vm, void * reserved) {  
    jclass tmp = env->FindClass("com/example/company/MyClass");  
    myClass = (jclass)env->NewGlobalRef(tmp);
    return JNI_VERSION_1_6;  
}

For more examples see link provided by Marek Sebera: http://android-developers.blogspot.cz/2011/11/jni-local-reference-changes-in-ics.html

Alexey
  • 7,262
  • 4
  • 48
  • 68
  • 11
    Seriously though, this should go on Android's frontpage. I'm now wondering how many thousands of native apps could randomnly crash because of this issue, jesus... – RelativeGames Oct 17 '13 at 02:32
  • @Alexey mate i m new to android and facing the same problem http://stackoverflow.com/questions/21423036/sqlite3-exception-database-disk-image-is-malformed pls tell how can i implement your code.. i m rectifying it from last 3 weeks :( – Maveňツ Jan 31 '14 at 12:28
  • @ManishYadav The jni wrapper around sqlite library probably has the same error I'm talking about here, but I can't say for sure without looking at the jni (c-side) wrapper itself. You should really attach both java-side and c-side jni code to your question. Also, have you considered using android standart wrapper around sqlite? It's fast, and it'll suffice for almost anything (except localized search in non-english, custom indexes and few other rarely used things). – Alexey Feb 02 '14 at 15:45
  • @Alexey thanx a ton buddy for suggesting,pls find the code now i have uploaded the "c" files. – Maveňツ Feb 03 '14 at 05:27
  • 3
    Beautifully explained. Do I need to release global references in any kind of shutdown function? – Andrew Wyld Feb 13 '14 at 17:47
  • Ah, just looked up JNI_OnUnload and apparently it's never called in Android. – Andrew Wyld Feb 14 '14 at 10:25