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

Posted 2023-02-22

【中文标题】C 语言错误 munmap_chunk(): invalid pointer Aborted (core dumped)

【英文标题】：C language error munmap_chunk(): invalid pointer Aborted (core dumped)

【发布时间】：2016-03-27 12:27:11

【问题描述】：

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

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;

【问题讨论】：

数据的类型是什么？

这是一个结构体，typedef struct char *tsname; TSItem *TSSet;诠释 num_item;整数类；诠释 *ls_class;整数长度； 数据;

请编辑帖子并将其粘贴到那里

仅仅因为您打印的数据看起来正确并不意味着指针实际上是有效的。如果没有Minimal, Complete, and Verifiable Example，就不可能说出问题所在。

顺便说一句，在 C 中你 don't have to cast the result of malloc。事实上，如果您忘记包含<stdlib.h>，它实际上可能很危险。

【参考方案1】：

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

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

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

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

【讨论】：

如果 id 上升到等于 MAXC 你可以得到你看到的行为。