C 语言错误 munmap_chunk(): invalid pointer Aborted (core dumped)

Question

我在我的项目中遇到了这个错误。相关代码如下： 我为指针“值”分配内存如下：

data->TSSet[0].values = (float *)malloc(sizeof(float)*10000);

然后在使用之后，我释放的值如下：

free(data->TSSet[0].values);

然后我遇到了错误 munmap_chunk(): invalid pointer Aborted (core dumped)。指针“values”是有效的，因为当我尝试打印出“values”中的内容时，如下所示：

for(i=0;i<TSSet[0].length;i++)
    printf("%f\n",TSSet[0].values[i]);

我得到了正确的输出。 为什么会发生错误？ 完整版代码如下：（我在关键部分加了cmets，不想看长代码直接跳到cmets） 在 main 函数中，我只是使用 ReadCSV(file, &data) 读取数据，然后立即调用 ResetData(&data)。 谢谢大家帮助我！！！！

void ReadCSV(char *file_path, DATA *Data){
    Data->tsname = (char*)malloc(sizeof(char)*MAXNAME);
    Data->TSSet = (TSItem*)malloc(sizeof(TSItem)*MAXNITEM);
    FILE *fp = fopen(file_path, "r");
    if(fp == NULL)
        return ;
    char line[MAXSTRL];
    char *save_ptr,*tk;
    int id = 0, num_class = 0, i,label;
    Data->ls_class = (int *)malloc(sizeof(int)*MAXC);
    Data->num_item = 0;
    Data->length = 0;
    while(fgets(line, sizeof(line), fp)) {
        tk = strtok_r(line,",", &save_ptr);
        if (tk == NULL)
            return ;
        label = atoi(trim(tk));
        Data->TSSet[id].label = label;
        Data->TSSet[id].num_nbr = 0;
        if(IsExist(label,Data->ls_class, num_class) == 0){
            Data->ls_class[num_class] = label;
            num_class++;
        }
        // malloc the memory for values
        Data->TSSet[id].values = (float *)malloc(sizeof(float)*MAXL);
        Data->TSSet[id].knn = (Neighbor*)malloc(sizeof(Neighbor)*MAXK);
        i=0;
        // read in content of values from files
        while(tk!=NULL){
            tk = strtok_r(NULL,",", &save_ptr);
            Data->TSSet[id].values[i] = strtof(trim(tk),NULL);
            i++;
            if(strlen(save_ptr) == 0)
                break;
        }
        Data->TSSet[id].predlbl = -100;
        Data->TSSet[id].conf = 0.0;
        Data->TSSet[id].entropy = 0.0;
        Data->TSSet[id].length = i;
        id++;
    }
    if(fp == NULL)
        printf("NULL!!!!!!!!!!!!!!!!!!!!!\n");
    fclose(fp);
    Data->num_item = id;
    Data->length = i;
    Data->num_class = num_class;
}

void ResetData(DATA *data){
    int i;
    free(data->tsname);
    for(i=0;i<data->num_item;i++){
    // error here
        free(data->TSSet[i].values);
        free(data->TSSet[i].knn);
    }
    free(data->TSSet);
    data->num_item = 0;
    data->num_class = 0;
    free(data->ls_class);
    data->length = 0;
    return;
}

Answer 1

对我来说，跳出页面的第一件事是您的外部 while 循环缺少边界检查。我会验证 id 不会变得太大......至少：

 while(fgets(line, sizeof(line), fp)) {
   断言（id 

如果 id 变得太大，您就会开始进入未定义行为的领域。 

malloc() 返回的每个区域都有一个与之关联的内部“标头”，至少提供要释放的可用内存字节。如果你走过去，可能会发生各种奇怪的事情，尤其是在 free() 调用时。