0

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

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 指针无效?

4

1 回答 1

5

join您的第一次调用中,将一个指向字符串文字的指针(它是只读的)传递给free. free除非您分配了正在释放的内存,否则您不应该调用。

您可以使用以下方法在调用者中修复此问题strdup

char* str = strdup("John");
str = strjoin(str,"\x20");
str = strjoin(str,"Doe");
于 2012-08-23T04:15:39.947 回答