Question

我正在开发一个使用带有2d数组的结构作为字段的程序。但由于某些原因evrytime我尝试使用free_planet函数我收到双重免费错误。使用程序作为valgrind似乎问题是从第49行到第51行的说明，但我真的不明白为什么。

#include <stdio.h>
#include <stdlib.h>
#include <errno.h>
typedef enum cell { SHARK, FISH, WATER } cell_t;

typedef struct planet {
    unsigned int nrow; 
    unsigned int ncol; 
    cell_t ** w;
    int ** btime;
    int ** dtime;
} planet_t;


planet_t * new_planet (unsigned int nrow, unsigned int ncol) {
    if ((int) nrow>=0 && (int) ncol>=0){
        int i,j;
        planet_t *a=malloc(sizeof(planet_t*)); 
        a->nrow=nrow;
        a->ncol=ncol;

        a->w=(cell_t**) malloc(nrow * sizeof(cell_t*));
        a->btime=(int**) malloc(nrow* sizeof(int*));
        a->dtime=(int**) malloc(nrow* sizeof(int*));

        for (i=0; i<nrow; i++){
            a->w[i]=(cell_t*) malloc(ncol * sizeof(cell_t));
            a->btime[i]=(int*) malloc(ncol*sizeof(int));
            a->dtime[i]=(int*) malloc(ncol*sizeof(int));
        }

        for (i=0; i<nrow; i++)
            for (j=0; j<ncol; j++)
                a->w[i][j]=WATER;
        return a;
    }
    errno=EINVAL;
    return NULL;
}

void free_planet (planet_t* p){
    int i;
    int nrow=p->nrow;   
    for (i=0;i<nrow;i++){
        if (p->w[i]!=NULL) free (p->w[i]);  
        if (p->btime[i]!=NULL) free (p->btime[i]);
        if (p->dtime[i]!=NULL) free (p->dtime[i]);
    }
    if (p->w!=NULL) free (p->w);
    if (p->btime!=NULL) free (p->btime);
    if (p->dtime!=NULL) free (p->dtime);
    if (p!=NULL) free (p);
    return;
}

int main(){
    planet_t *p; unsigned int x,y;

    scanf( "%u %u", &x, &y);
    p = new_planet(x,y);
    free_planet (p);
    return 0;
}

Answer 1

这是我看到的第一件事：

planet_t *a = malloc(sizeof(planet_t*));

当你应该为结构分配空间时，你正在为指针分配空间。所以它应该是：

planet_t *a = malloc(sizeof(planet_t));

或者：

planet_t *a = malloc(sizeof(*a));

解除分配2d数组的双重自由错误

1 个答案: