jni-wrapper 在 Windows 上使 JVM 崩溃
jni-wrapper crashes JVM on Windows
我写了一个 JNI-Wrapper,但直到今天才在 Linux 上使用它。
现在我为 windows 编译它并且 JVM 在这个本地方法中崩溃:
如果我删除 ReleaseStringUTFChars
和 free
调用它也适用于 Windows,但有趣的是为什么它适用于 linux 但不适用于 windows?(我使用 Windows 10-64 位)。根据我的经验,这个调用是强制性的,在我正确释放它们之前有一些内存泄漏(在 linux)
void onMessageReceived(char* topic, char* timestamp, char* id, char* value) {
JNIEnv * g_env;
int getEnvStat = (*g_vm)->GetEnv(g_vm, (void **) &g_env, JNI_VERSION_1_8);
if (getEnvStat == JNI_EDETACHED) {
if ((*g_vm)->AttachCurrentThread(g_vm, (void **) &g_env, NULL) != 0) {
puts("Failed to attach");
fflush(stdout);
}
}
if (methodHandleMessage) {
} else {
jclass clazz = (*g_env)->GetObjectClass(g_env, store_callback);
methodHandleMessage = (*g_env)->GetMethodID(g_env, clazz, "handleMessage", "(Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;)V");
}
jstring j_topic = (*g_env)->NewStringUTF(g_env, topic);
jstring j_timestamp = (*g_env)->NewStringUTF(g_env, timestamp);
jstring j_value = (*g_env)->NewStringUTF(g_env, value);
jstring j_id = (*g_env)->NewStringUTF(g_env, id);
(*g_env)->CallVoidMethod(g_env, store_callback, methodHandleMessage, j_topic, j_timestamp, j_id, j_value);
//HERE IS THE PROBLEM:
(*g_env)->ReleaseStringUTFChars(g_env, j_topic, topic);
(*g_env)->ReleaseStringUTFChars(g_env, j_timestamp, timestamp);
(*g_env)->ReleaseStringUTFChars(g_env, j_value, value);
(*g_env)->ReleaseStringUTFChars(g_env, j_id, id);
//
(*g_vm)->DetachCurrentThread(g_vm);
}
和
void rep_actor(zsock_t *pipe, void *args) {
zsock_signal(pipe, 0);
while (!zsys_interrupted) {
char* timestamp;
char* sender;
char* command;
char* message;
zsock_recv(reply, "ssss", ×tamp, &sender, &command, &message);
char* result = onRequestReceived(timestamp, sender, command, message);
zsock_send(reply, "s", result);
//HERE IS THE PROBLEM:
free(timestamp);
free(sender);
free(command);
free(message);
free(result);
//
}
}
错误发生是因为你释放内存两次。第一次 - 通过在 onMessageReceived()
内部调用 ReleaseStringUTFChars()
,第二次 - 在 onMessageReceived()
外部调用每个释放的指针 free()
。
重要
对于在 java 环境中创建的对象(调用 env->New*()
方法),您不需要手动释放内存。在你的情况下:
// j_topic - it's a pointer for object inside of java heap
// garbage collector will free this memory
jstring j_topic = env->NewStringUTF(topic);
ReleaseStringUTFChars()
的调用在这里不合适。这仅在您使用 GetStringUTFChars()
:
从 java String
创建 原生字符串 时使用
// j_topic - string inside of java heap
const char* native_str = env->GetStringUTFChars(j_topic, isCopy);
// your logic for native_str array
// informs the VM that the native code no longer needs access to native_str
env->ReleaseStringUTFChars(j_topic, native_str);
在 javadoc 中查看方法 GetStringUTFChars()
和 ReleaseStringUTFChars()
- 你应该只将它们一起使用。
我写了一个 JNI-Wrapper,但直到今天才在 Linux 上使用它。
现在我为 windows 编译它并且 JVM 在这个本地方法中崩溃:
如果我删除 ReleaseStringUTFChars
和 free
调用它也适用于 Windows,但有趣的是为什么它适用于 linux 但不适用于 windows?(我使用 Windows 10-64 位)。根据我的经验,这个调用是强制性的,在我正确释放它们之前有一些内存泄漏(在 linux)
void onMessageReceived(char* topic, char* timestamp, char* id, char* value) {
JNIEnv * g_env;
int getEnvStat = (*g_vm)->GetEnv(g_vm, (void **) &g_env, JNI_VERSION_1_8);
if (getEnvStat == JNI_EDETACHED) {
if ((*g_vm)->AttachCurrentThread(g_vm, (void **) &g_env, NULL) != 0) {
puts("Failed to attach");
fflush(stdout);
}
}
if (methodHandleMessage) {
} else {
jclass clazz = (*g_env)->GetObjectClass(g_env, store_callback);
methodHandleMessage = (*g_env)->GetMethodID(g_env, clazz, "handleMessage", "(Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;Ljava/lang/String;)V");
}
jstring j_topic = (*g_env)->NewStringUTF(g_env, topic);
jstring j_timestamp = (*g_env)->NewStringUTF(g_env, timestamp);
jstring j_value = (*g_env)->NewStringUTF(g_env, value);
jstring j_id = (*g_env)->NewStringUTF(g_env, id);
(*g_env)->CallVoidMethod(g_env, store_callback, methodHandleMessage, j_topic, j_timestamp, j_id, j_value);
//HERE IS THE PROBLEM:
(*g_env)->ReleaseStringUTFChars(g_env, j_topic, topic);
(*g_env)->ReleaseStringUTFChars(g_env, j_timestamp, timestamp);
(*g_env)->ReleaseStringUTFChars(g_env, j_value, value);
(*g_env)->ReleaseStringUTFChars(g_env, j_id, id);
//
(*g_vm)->DetachCurrentThread(g_vm);
}
和
void rep_actor(zsock_t *pipe, void *args) {
zsock_signal(pipe, 0);
while (!zsys_interrupted) {
char* timestamp;
char* sender;
char* command;
char* message;
zsock_recv(reply, "ssss", ×tamp, &sender, &command, &message);
char* result = onRequestReceived(timestamp, sender, command, message);
zsock_send(reply, "s", result);
//HERE IS THE PROBLEM:
free(timestamp);
free(sender);
free(command);
free(message);
free(result);
//
}
}
错误发生是因为你释放内存两次。第一次 - 通过在 onMessageReceived()
内部调用 ReleaseStringUTFChars()
,第二次 - 在 onMessageReceived()
外部调用每个释放的指针 free()
。
重要
对于在 java 环境中创建的对象(调用 env->New*()
方法),您不需要手动释放内存。在你的情况下:
// j_topic - it's a pointer for object inside of java heap
// garbage collector will free this memory
jstring j_topic = env->NewStringUTF(topic);
ReleaseStringUTFChars()
的调用在这里不合适。这仅在您使用 GetStringUTFChars()
:
String
创建 原生字符串 时使用
// j_topic - string inside of java heap
const char* native_str = env->GetStringUTFChars(j_topic, isCopy);
// your logic for native_str array
// informs the VM that the native code no longer needs access to native_str
env->ReleaseStringUTFChars(j_topic, native_str);
在 javadoc 中查看方法 GetStringUTFChars()
和 ReleaseStringUTFChars()
- 你应该只将它们一起使用。