Geek asks and answers

AttachCurrentThread crashing

The application crashes if the following function is called:

void on_log(JavaVM* vm, int level, const char *data)
{
  printOnAndroid(level, data);
  pthread_t loggerThread;
  pthread_create(&loggerThread, NULL, attachThreadToJavaVMAndPrint, data);
  pthread_join(loggerThread, NULL);
}

void attachThreadToJavaVMAndPrint(JavaVM* vm, const char *data)
{
  int isThreadAttached = attachJNIEnvToThread(vm);
  if (isThreadAttached == 1)
  {
     JNIEnv* env;
     (*vm)->GetEnv(vm, &env, APP_JNI_VERSION);

     jclass thisClass = (*env)->GetObjectClass(env, _loggerObject);
     jmethodID methodId = (*env)->GetMethodID(env, thisClass, "logFromC","(Ljava/lang/String;)V");
     if (methodId != NULL)
     {
         jstring message = (*env)->NewStringUTF(env, data);
         (*env)->CallVoidMethod(env, _loggerObject, methodId, message);
     }
     (*vm)->DetachCurrentThread(vm);
  }
}

void printOnAndroid(int level, const char* data)
{
   __android_log_print(ANDROID_LOG_DEBUG, DEBUG_TAG, "XXXX %i, data %s", level, data);
}

int attachJNIEnvToThread(JavaVM* vm)
{
  JNIEnv* env;
  JavaVMAttachArgs args;
  args.version = APP_JNI_VERSION;
  args.name = NULL;
  args.group = NULL;
  if ((*vm)->GetEnv(vm, &env, APP_JNI_VERSION) == JNI_EDETACHED)
  {
    jint attachResponse = (*vm)->AttachCurrentThread(vm, &env, &args);
    if ((*vm)->GetEnv(vm, &env, APP_JNI_VERSION) != JNI_OK)
    {
        return 0;
    }
  }
  return 1;
 }

There is a macro:

  #define APP_JNI_VERSION JNI_VERSION_1_2

According to the logs, a crash occurs when attachJNIEnvToThread (..) is called

EDIT:

It works successfully on linux .. so maybe there is something specific for Android that I don't know about.

EDIT 2:

The signature of the following has been changed:

  void * attachThreadToJavaVMAndPrint(void* dataArg)
  {
     JavaVM* vm = _vm;
     const char* data = (const char *)dataArg;
     int isThreadAttached = attachJNIEnvToThread(vm);
     if (isThreadAttached == 1)
     {
       JNIEnv* env;
       (*vm)->GetEnv(vm, &env, APP_JNI_VERSION);

       jclass thisClass = (*env)->GetObjectClass(env, _loggerObject);
       jmethodID methodId = (*env)->GetMethodID(env, thisClass, "logFromC","(Ljava/lang/String;)V");
       if (methodId != NULL)
       {
          jstring message = (*env)->NewStringUTF(env, data);
          (*env)->CallVoidMethod(env, _loggerObject, methodId, message);
       }
       (*vm)->DetachCurrentThread(vm);
     }
     return 0;
  }

This changes the log a bit, and the crash happens in the attachThreadToJavaAndPrint (..) file

EDIT 3:

It was agreed. The final changes included Edit 2 + some changes in the java code (not visible here (there were errors in it)).

+3
source share
2 answers

First make sure the vm parameter is correct in void on_log(JavaVM* vm, int level, const char *data). I saved vm as a global variable in JNIEXPORT jint JNICALL JNI_OnLoad(JavaVM *vm, void *reserved), and my code may work:

JNIEXPORT jint JNICALL JNI_OnLoad(JavaVM *vm, void *reserved)
{
                g_vm = vm;
}


{
    ret = g_vm->AttachCurrentThread( (JNIEnv **) &env, NULL);
}
+1

- . , .

, VM*, data, .

:

struct params
{
    JavaVM *vm;
    char *data;
};

void on_log(JavaVM* vm, int level, const char *data)
{
  struct params params = {vm, data};
  printOnAndroid(level, data);
  pthread_t loggerThread;
  pthread_create(&loggerThread, NULL, attachThreadToJavaVMAndPrint, &params);
  pthread_join(loggerThread, NULL);
}

void * attachThreadToJavaVMAndPrint(void* arg)
{
 struct params *params = arg;
 JavaVM* vm = params->vm;
 const char* data = params->data;
 ...

: JNIEnv :

int attachJNIEnvToThread(JavaVM* vm)
{
  JNIEnv* env;

  return (*vm)->AttachCurrentThread(vm, &env, NULL) ? 1 : 0;
}
+1

More articles:


All Articles