我有一个函数将字符串附加到另一个字符串:
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无效指针?
答案 0 :(得分:5)
在join
的第一次调用中,您将指向字符串文字的指针传递给free
,该指针是只读的。除非已经分配了正在释放的内存,否则不应该调用free
。
您可以使用strdup
:
char* str = strdup("John");
str = strjoin(str,"\x20");
str = strjoin(str,"Doe");