6

我正在尝试在 linux 下编译和用 C 编写的代码，并收到以下错误消息：

glibc 检测到 malloc()：内存损坏

我不知道为什么......

substring() 只是通过给出起始索引和长度来返回原始字符串的一部分。例如 substring("this is example",0,4) = "this";

char *substring(char* str, int start, int length) {
    char *newString = (char *)malloc(length * sizeof(char));
    int i, x = 0;
    int end=start+length-1;
    for(i = start ; i <= end; i++){
        newString[x++] = str[i];
    }
    newString[x] = '\0';
    return newString;
}

getCharIndexFirst() 只返回指定字符的第一次出现的索引 getCharIndexLast() 只返回指定字符的最后一次出现的索引

以下是主要功能：

//consoleCommand has the form of 'send MESSAGE ID', has the value from stdin

int firstSpace = getCharIndexFirst(consoleCommand,' ');
int lastSpace = getCharIndexLast(consoleCommand,' ');
int len = strlen(consoleCommand);

char *header = substring(consoleCommand,0,firstSpace);
printf("header is: %s\n",header);
char *cmd = substring(consoleCommand,firstSpace+1,lastSpace-firstSpace-1);
printf("command is: %s\n",cmd); // the code only runs up to here and output the error..
char *socketstr = substring(consoleCommand,lastSpace+1,len-lastSpace-1);
printf("socket is: %s\n",socketstr);

这里有更多信息：consoleCommand 通常是标准输入，格式为“发送消息 ID”，当消息长度为 12 个字符时发生错误...例如“发送此消息 4”，“此消息”是 cmd并且长度为 12 个字符，这给了我错误！它适用于任何其他长度，我尝试过 3、4、24 ......

任何提示将不胜感激，谢谢！

4

2 回答 2

13 
newString[x] = '\0';

此时x等于length，这意味着您在分配的内存末尾写入 1 个字符。您需要为另外一个字符分配空间。

于 2013-09-27T17:56:41.587 回答
5

您没有为终止'\0'字符分配任何空间，因此您溢出分配以写入该字符。您还需要在分配中计算此字符：

char *newString = (char *)malloc((length + 1) * sizeof(char));
于 2013-09-27T17:55:31.463 回答