JNI GetMethodID导致本机线程出错

时间:2014-09-11 08:09:53

标签: java android c++ android-ndk java-native-interface

在android中我使用pthread_create创建本机线程,然后在Callback过程中,调用FindClass来获取Java类。但它确实有效。我从android jni tips获得提示 我在FindClass from any thread in Android JNI

中找到了解决方案

我为我的项目修改它 [编辑]

JavaVM* gJvm = nullptr;
static jobject gClassLoader;
static jmethodID gFindClassMethod;

JNIEXPORT jint JNICALL JNI_OnLoad(JavaVM *pjvm, void *reserved) {
    gJvm = pjvm;  // cache the JavaVM pointer
    auto env = getEnv();
    //replace with one of your classes in the line below
    auto randomClass = env->FindClass("com/example/RandomClass");
    jclass classClass = env->GetObjectClass(randomClass);
    auto classLoaderClass = env->FindClass("java/lang/ClassLoader");
    auto getClassLoaderMethod = env->GetMethodID(classClass, "getClassLoader",
                                             "()Ljava/lang/ClassLoader;");
    gClassLoader = env->CallObjectMethod(randomClass, getClassLoaderMethod);
    gClassLoader = env->NewGlobalRef(gClassLoader);
    gFindClassMethod = env->GetMethodID(classLoaderClass, "loadClass",
                                    "(Ljava/lang/String;)Ljava/lang/Class;");

    //check. this is ok
    jclass cls = env->FindClass("com/example/data/DataTest");
    jmethodID methoID = env->GetMethodID(cls, "name", "()Ljava/lang/String;");
    LOG_INFO("cls is %p\n", cls);

    return JNI_VERSION_1_6;
}

JNIEnv* getEnv() {
    JNIEnv *env;
    int status = gJvm->GetEnv((void**)&env, JNI_VERSION_1_6);
    if(status < 0) {    
        status = gJvm->AttachCurrentThread(&env, NULL);
        if(status < 0) {        
        return nullptr;
        }
    }  
    return env;
}


jclass findClass(const char* name) {
     JNIEnv *env = getEnv();
     jclass resultClass = 0;
     if(env)
     {
        resultClass = env->FindClass(name);
        //it can not found class in native thread, use loadClass method
        if (!resultClass)
        {
            LOG_INFO("can not find the class");
            //return value is not null. 
            return static_cast<jclass>(env->CallObjectMethod(gClassLoader, gFindClassMethod, env->NewStringUTF(name))); 
        }
    }
    return resultClass;
}
.......
//thread callback 
void *proc(void *)
{
  JNIEnv *env = getEnv();
  jclass cls = findClass("com/example/data/DataTest");
  if (cls)
  {
    LOG_INFO("GetMethodID");
    //crash
    jmethodID methodID = env->GetMethodID(cls, "name", "()Ljava/lang/String;"); 
    LOG_INFO("proc tag is %p\n", tag);
  }
}
.....
pthread_create(&handle, NULL, proc, 0);
.....

程序退出env-&gt; GetMethodID。我收到这个错误:

Invalid indirect reference 0x40d8bb20 in decodeIndirectRef.

如果我从findClass中删除resultClass = env->FindClass(name);,那就没问题了。 &#34; proc标签是&#34;可以打印。

//correct 
jclass findClass(const char* name) {
     JNIEnv *env = getEnv();
     jclass resultClass = 0;
     if(env)
     {
        if (!resultClass)
        {
            return static_cast<jclass>(env->CallObjectMethod(gClassLoader, gFindClassMethod, env->NewStringUTF(name))); 
        }
    }
    return resultClass;
}

env->FindClass(name);env->CallObjectMethod to loadClass之间的任何混淆?

这是一个错误吗?如何解决问题呢?

3 个答案:

答案 0 :(得分:2)

不要这样做:

gClassLoader = env->CallObjectMethod(randomClass, getClassLoaderMethod);

特别是,永远不要使用本地引用(这是CallObjectMethod返回的内容)并将其存储在除局部变量之外的任何内容中。

如果要在获取本地引用的函数之外访问该值,则需要使用NewGlobalRef获取全局引用。一旦执行返回到该线程中的VM,本地引用就会失效。

参见&#34;本地和全球参考文献&#34; JNI Tips文档中的部分。

答案 1 :(得分:1)

我使用了一些不同的代码来获取实际的JNIEnv*指针(它基于SO在某处发布的代码示例)。事实证明这种方法可靠。

class BaseJNI
{
protected:
    BaseJNI(JNIEnv * env)
    {
        int ret = env->GetJavaVM(&jvm);
        if( ret != JNI_OK )
        {
            LOG_INFO("Could not get JavaVM: %d", ret);
            throw;
        }
    }

    JNIEnv * GetEnv()
    {
        JNIEnv * env;
        // double check it's all ok
        int ret = jvm->GetEnv((void **)&env, JNI_VERSION_1_6);
        if (ret == JNI_OK)
            return env;

        if (ret == JNI_EDETACHED)
        {
            LOG_INFO("GetEnv: thread is not attached, trying to attach");
            ret = jvm->AttachCurrentThread(&env, nullptr);
            if( ret == JNI_OK )
            {
                LOG_INFO("GetEnv: attach successful");
                return env;
            }

            LOG_INFO("Cannot attach JNI to current thread: %d", ret);
            return nullptr;
        }

        LOG_INFO("could not get JNI ENV: %d", ret);
        return nullptr;
    }

protected:
    JavaVM *jvm;
};

当你确实拥有一个有效的JNIEnv*指针时,应该在你的主线程中实例化这个类,然后它也可以在后台线程中使用。

您是否可以尝试使用上面的代码获取JNIEnv*指针并查看指针是否正确获取?

答案 2 :(得分:1)

抱歉,我犯了一个愚蠢的错误。

resultClass = env->FindClass(name);

env-&gt; FindClass(name)抛出NoClassDefFoundException并返回NULL。申请继续进行。巧合地,VM在jmethodID methodID = env->GetMethodID(cls, "name", "()Ljava/lang/String;");中止。 可能是一个解决方案是:

jclass findClass(const char* name) {
     JNIEnv *env = getEnv();
     jclass resultClass = 0;
     if(env)
     {
        resultClass = env->FindClass(name);
        jthrowable mException = env->ExceptionOccurred();
        if (mException )
        {
            env->ExceptionDescribe();
            env->ExceptionClear();
            return static_cast<jclass>(env->CallObjectMethod(gClassLoader, gFindClassMethod, env->NewStringUTF(name))); 
        }
    }
    return resultClass;
}

我在http://android.wooyd.org/JNIExample找到了一些有用的用法 谢谢!