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():无效指针中止(核心转储)。指针“values”是有效的，因为当我尝试打印“values”中的内容时，如下所示:

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

我得到了正确的输出。为什么会发生错误？完整版代码如下:(关键部分我加了注释，如果不想看长代码就直接跳到注释处)在主函数中，我只是使用 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)) {    assert( id < MAXC  );  /* (recommend a more "noisy" error message) */   ... }

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

每个 malloc() 返回的区域都有一个与其关联的内部“ header ”，至少给出要释放的空闲内存字节。如果你仔细观察这个，可能会发生各种奇怪的事情，特别是在 free() 调用上。