c - 释放内存时堆损坏 C

Question

这是我的代码：

int load_data(char *line, int *vertex, int *edge)
{
    char *ch_number = NULL;
    char *error = NULL;

    *vertex = (int)strtol(line, &error ,10);
    if((error[0] != '-') && (error[1] != '>')) return 0;

    ch_number = (char*)malloc(sizeof(char) * (strlen(error) - 2));

    memcpy(ch_number, &error[2], strlen(error) - 2);
    ch_number[strlen(error) - 2] ='\0';

    *edge = (int)strtol(ch_number, &error ,10);

    if(error[0] != '\0') 
    {
        free(ch_number);
        return 0;   
    }
    free(ch_number);
    return 1;
}

调试器显示 free(ch_number); 使堆损坏。我究竟做错了什么？这是使用的示例：

load_data("15643->45545546",&vertex,&edge);

Answer 1

C 数组是从零开始的，所以这条线

 ch_number[strlen(error) - 2] ='\0';

在结束后写入一个字节ch_number。这样做的影响是不确定的，但听起来你已经写入了堆管理器使用的保护词来准确检测这种损坏。

如果要向 中添加空终止符ch_number，则需要分配一个额外的内存字节

ch_number = malloc(strlen(error) - 1);