新线程中的Java环境变量导致错误

我的应用程序需要将java层的audiorecord录制的样本发送到JNI

但为了避免多次内存操作（复制），我在本机层初始化期间创建了缓冲区，并将其传递给java层，如下所示

应用程序.c

#include <jni.h>
#include <stdio.h>
#include <string.h>
#include <pthread.h>

#include <android/log.h>
#include "Application.h"

JNIEnv *Native_env;
jobject native_obj;

jmethodID WriteID, ReadID;
jclass JavaClass;
jshortArray ShortArray;

FILE *read_fptr;
//jsize start = 0;
//jsize leng = 0;
jshort NativeArray[1920];

void Update_NativeJNVVariables(JNIEnv *env, jobject obj)
{
    Native_env = (JNIEnv * )env;
    native_obj = obj;

    //ShortArray = (*Native_env)->NewShortArray(Native_env, 1920);

    //leng = (*Native_env)->GetArrayLength(Native_env,ShortArray);

    //__android_log_print(ANDROID_LOG_ERROR, "Update_NativeJNVVariables", "Length of array is %d", leng);

    memset(NativeArray, 0x00, sizeof(NativeArray));

    //(*Native_env)->SetShortArrayRegion(Native_env,ShortArray, start, leng, NativeArray);

    //jclass LocalClass;
    //LocalClass = (*Native_env)->GetObjectClass(Native_env,native_obj);
    //WriteClass = (*Native_env)->NewGlobalRef(Native_env,LocalClass);

    JavaClass = (*Native_env)->FindClass(Native_env, "com/consilient/tech/uday/javaapi/MainActivity");

    //LocalClass = (*Native_env)->GetObjectClass(Native_env,native_obj);
    //ReadClass = (*Native_env)->NewGlobalRef(Native_env,LocalClass);

   if(JavaClass != NULL)
   {
       WriteID = (*Native_env)->GetMethodID(Native_env,JavaClass,"WriteAndroidPCM","([B)V");
   }

   if(JavaClass != NULL)
   {
       //ReadID = (*Native_env)->GetMethodID(Native_env,JavaClass,"ReadAndroidPCM","([SI)V");
       ReadID = (*Native_env)->GetMethodID(Native_env,JavaClass,"ReadAndroidPCM","(I)V");
   }

    read_fptr = fopen("/sdcard/outputs/read_file.pcm","wb");
    if(read_fptr == NULL)
   {
        __android_log_print(ANDROID_LOG_ERROR, "UPdateJavaEnv", "Read FIle Cannot Be opened");
   }
}

void ReadPCMSamples(short *Buffer, int no_of_samples)
{
    //    jboolean isCopy = JNI_TRUE;
    //    jshort *ArrayPointer = (*Native_env)->GetShortArrayElements(Native_env,ShortArray, &isCopy);

    if(ReadID)
        (*Native_env)->CallVoidMethod(Native_env,JavaClass,ReadID,no_of_samples);

    //fwrite(ArrayPointer, sizeof(short), 160, read_fptr);

    //(*Native_env)->ReleaseShortArrayElements( Native_env, ShortArray, ArrayPointer, 0 );

    //(*Native_env)->GetDirectBufferAddress(Native_env,native_obj);

    //GetByteArray(Buffer, no_of_samples);
}

/*
 * Class:     com_consilient_tech_uday_javaapi_MainActivity
 *  Method:    StartNativeThread
 * Signature: ()V
 */
JNIEXPORT void JNICALL Java_com_consilient_tech_uday_javaapi_MainActivity_FillNativeBuffer
    (JNIEnv *env, jobject obj, jshortArray Buffer, int no_of_samples)
{
      jboolean isCopy = JNI_TRUE;
      jshort *ArrayPointer = (*Native_env)->GetShortArrayElements(Native_env, Buffer, 0);
/*
        for (int i = 0; i < no_of_samples; i++)
        {
            NativeArray[i] = ArrayPointer[i];
        }
*/
         (*Native_env)->ReleaseShortArrayElements( Native_env, Buffer, ArrayPointer, 0 );
    }

int ifReceivedExit = 0;

void Start()
{
    while(ifReceivedExit == 0)
    {
        ReadPCMSamples(NativeArray,160);
    }
}

void Stop()
{
    ifReceivedExit = 1;
}

pthread_t threadID;

void StartThread()
{
int res  = 0;
res = pthread_create(&threadID, NULL, Start, NULL);
if (res != 0)
    __android_log_print(ANDROID_LOG_ERROR, "Thread Creation", "Failed %s", strerror(res));
else
    __android_log_print(ANDROID_LOG_ERROR, "Thread Creation", "Success"); // Thread ID %x res %d", threadID, res);
}

void StopThread()
{
    Stop();
    if(JavaClass)
        (*Native_env)->DeleteGlobalRef(Native_env,JavaClass);

    if(read_fptr)
       fclose(read_fptr);
}

jint JNI_OnLoad(JavaVM* vm, void* reserved)
{
    return JNI_VERSION_1_6;
}

主要活动.c

#include <jni.h>

#include "com_consilient_tech_uday_javaapi_MainActivity.h"
#include "Application.h"

/*
* Class:     com_consilient_tech_uday_javaapi_MainActivity
* Method:    InitializeJNI
* Signature: ()V
*/
JNIEXPORT void JNICALL Java_com_consilient_tech_uday_javaapi_MainActivity_InitializeJNI
    (JNIEnv *env, jobject obj)
{
Update_NativeJNVVariables(env, obj);
}

/*
* Class:     com_consilient_tech_uday_javaapi_MainActivity
* Method:    StartNativeThread
* Signature: ()V
*/
JNIEXPORT void JNICALL Java_com_consilient_tech_uday_javaapi_MainActivity_StartNativeThread
    (JNIEnv *env, jobject obj)
{
StartThread();
}

/*
* Class:     com_consilient_tech_uday_javaapi_MainActivity
* Method:    StartNativeThread
* Signature: ()V
*/
JNIEXPORT void JNICALL Java_com_consilient_tech_uday_javaapi_MainActivity_StopNativeThread
    (JNIEnv *env, jobject obj)
{
StopThread();
}

/*
* Class:     com_consilient_tech_uday_javaapi_MainActivity
* Method:    GetByteArrayFromNative
* Signature: (SI)V
*/
JNIEXPORT void JNICALL Java_com_consilient_tech_uday_javaapi_MainActivity_GetByteArrayFromNative
    (JNIEnv *env, jobject obj, jint no_of_samples)
{
//GetByteArray(no_of_samples);
}

#包括
#包括“com\u consilient\u tech\u uday\u javaapi\u main activity.h”
#包括“Application.h”
/*
*类别：com\u consilient\u tech\u uday\u javaapi\u main活动
*方法：初始化JNI
*签字：（）五
*/
JNIEXPORT void JNICALL Java\u com\u Consistent\u tech\u uday\u javaapi\u Main Activity\u InitializeJNI
（JNIEnv*env，jobject对象）
{
更新本地JNV变量（环境、obj）；
}
/*
*类别：com\u consilient\u tech\u uday\u javaapi\u main活动
*方法：StartNativeThread
*签字：（）五
*/
JNIEXPORT void JNICALL Java\u com\u Consistent\u tech\u uday\u javaapi\u Main Activity\u StartNativeThread
（JNIEnv*env，jobject对象）
{
StartThread（）；
}
/*
*类别：com\u consilient\u tech\u uday\u javaapi\u main活动
*方法：StartNativeThread
*签字：（）五
*/
JNIEXPORT void JNICALL Java_com_Consistent_tech_uday_javaapi_Main Activity_StopNativeThread
（JNIEnv*env，jobject对象）
{
止动螺纹（）；
}
/*
*类别：com\u consilient\u tech\u uday\u javaapi\u main活动
*方法：GetByteArrayFromNative
*签字：（SI）V
*/
JNIEXPORT void JNICALL Java\u com\u Consistent\u tech\u uday\u javaapi\u Main Activity\u GetByteArrayFromNative
（JNIEnv*env、jobject obj、jint样本编号）
{
//GetByteArray（无样本）；
}

当我试图找到bug时，它在GetShortArrayRegion崩溃，导致了分段错误

---

谢谢

谢谢Michael提供的链接

问题在于缓存Java本机环境变量（如您所指定的）

不应该缓存环境，而可以使用NewGlobalRef方法使类成为全局的，这将有助于在其他线程中使用该类。方法ID也可以使我成为全局的

最好在JNI_OnLoad中获取类和MethodID，并为它们创建全局引用

当我们创建新线程时，我们需要将新线程附加到JavaVM。

---

并获取JNI Env以进行进一步处理。

Native_Env来自哪里？你不应该缓存

JNIEnv

指针，以防万一。你也不应该试图将

ArrayPointer

传递给

ReadAndroidPCM

。Java方法不知道什么是

jshort*

。有一个函数Update_JNI_Variables（），从Java层调用，它用接收到的环境变量和对象变量更新本机环境变量和本机对象。“有一个函数Update_JNI_Variables（），从Java层调用。”呃。为什么不从需要执行JNI调用的本机函数调用

GetEnv

？（并确保将当前线程连接到VM，以防它被分离）这听起来像是一个奇怪的设计。既然你的问题中没有包含这些代码，就不可能排除这不是你问题的一部分。因此，我不认为这个问题现在能够得到可靠的回答。

#include <jni.h>

#include "com_consilient_tech_uday_javaapi_MainActivity.h"
#include "Application.h"

/*
* Class:     com_consilient_tech_uday_javaapi_MainActivity
* Method:    InitializeJNI
* Signature: ()V
*/
JNIEXPORT void JNICALL Java_com_consilient_tech_uday_javaapi_MainActivity_InitializeJNI
    (JNIEnv *env, jobject obj)
{
Update_NativeJNVVariables(env, obj);
}

/*
* Class:     com_consilient_tech_uday_javaapi_MainActivity
* Method:    StartNativeThread
* Signature: ()V
*/
JNIEXPORT void JNICALL Java_com_consilient_tech_uday_javaapi_MainActivity_StartNativeThread
    (JNIEnv *env, jobject obj)
{
StartThread();
}

/*
* Class:     com_consilient_tech_uday_javaapi_MainActivity
* Method:    StartNativeThread
* Signature: ()V
*/
JNIEXPORT void JNICALL Java_com_consilient_tech_uday_javaapi_MainActivity_StopNativeThread
    (JNIEnv *env, jobject obj)
{
StopThread();
}

/*
* Class:     com_consilient_tech_uday_javaapi_MainActivity
* Method:    GetByteArrayFromNative
* Signature: (SI)V
*/
JNIEXPORT void JNICALL Java_com_consilient_tech_uday_javaapi_MainActivity_GetByteArrayFromNative
    (JNIEnv *env, jobject obj, jint no_of_samples)
{
//GetByteArray(no_of_samples);
}