11

我认为是的,但我发现的前 12 个示例都做了一些不具说明性的事情,例如

JNIEXPORT jstring JCALL Java_com_foo_dumbImpl(JNIEnv* env, jobject thisObj)
{
  return (*env)->NewStringUTF(env, "constant string"); 
}

所以为了后代我会问:这很糟糕,是吗?

JNIEXPORT jstring JCALL Java_com_foo_dumbImpl(JNIEnv* env, jobject thisObj)
{
  char *leak = malloc(1024);
  leak[0] = '\0';
  return (*env)->NewStringUTF(env, leak); 
}

...并且应该是:

JNIEXPORT jstring JCALL Java_com_foo_dumbImpl(JNIEnv* env, jobject thisObj)
{
  char *emptystring = NULL;
  jstring r = NULL;
  emptystring = malloc(1024);
  emptystring[0] = '\0';
  r = (*env)->NewStringUTF(env, emptystring); 
  free(emptystring);
  emptystring = NULL;
  return  r;
}
4

2 回答 2

9

是的。(只是为了让这看起来没有答案。)

于 2009-05-20T19:43:07.200 回答
3

关注内存泄漏很好,但是,在这种情况下,没有泄漏(原始示例)。“常量字符串”是文字字符串;它不是从堆中分配的。

所以,不,您不需要清理传递的 char * (原始示例)。

您编辑的示例更适合说明您的观点。在编辑的示例中,是的,您需要清理传递的字符串。

于 2010-12-29T01:55:30.937 回答