C++ 错误消息 : invalid pointer

Question

我有一个函数可以将一个字符串附加到另一个字符串:

char* strjoin(char* str1,const char* str2) {
  long len1 = strlen(str1);
  long len2 = strlen(str2);
  char* result = (char*)malloc(len1+len2+1);

  memcpy(result,str1,len1+1);
  memcpy(result+len1,str2,len2+1);

  free(str1); <--------- program crashes here with error: invalid pointer
  return result;
}

调用上面函数的代码是这样的:

char* str = "John";
str = strjoin(str,"\x20");
str = strjoin(str,"Doe");

在上面的函数 strjoin 中，我为新字符串分配了内存，所以我释放了旧的 str1。为什么str1是无效指针？

Answer 1

在 join 的第一次调用中，您将指向字符串文字的指针(它是只读的)传递给 free。你不应该调用 free 除非你已经分配了被释放的内存。

您可以使用 strdup 在调用方中修复此问题:

char* str = strdup("John");
str = strjoin(str,"\x20");
str = strjoin(str,"Doe");