Question

我认为是的，但我发现的前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;
}

Answer 1

是。（只是这样看起来没有答案。）

Answer 2

关注内存泄漏是件好事，但是，在这种情况下，没有泄漏（原始示例）。 “常量字符串”是一个文字字符串;它不是从堆中分配的。

所以，不，你不需要清理传递的char *（原始示例）。

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

我是否需要清理传递给NewStringUTF的char *？

2 个答案: