Free() 函数导致程序崩溃

Question

我目前正在尝试释放分配的内存，但是这样做会导致程序崩溃。我是 C 和一般编程的新手，如果能在问题和任何其他可能源于我的经验不足的问题上获得帮助，我会很高兴。

Pool* allocatePool(int x);
void freePool(Pool* pool);
void store(Pool* pool, int offset, int size, void *object);

typedef struct _POOL
{
    int size;
    void* memory;
} Pool;

int main()
{
    printf("enter the number of bytes you want to allocate//>\n");
    int x;
    Pool* p;
    scanf("%d", &x);
    p=allocatePool(x);
    freePool(p);
    return 0;
}

/* Allocate a memory pool of size n bytes from system memory (i.e., via malloc()) and return a pointer to the filled data Pool structure */

Pool* allocatePool(int x)
{
    static Pool p;
    p.size = x;

    p.memory = malloc(x);
    printf("%p\n", &p);

    return &p;//return the address of the Pool
}

/* Free a memory pool allocated through allocatePool(int) */
void freePool(Pool* pool) 
{
    free(pool);
    printf("%p\n", &pool);
}

Answer 1

您必须释放已分配的相同内存。这里：

p.memory = malloc(x);

您将x 字节分配给p.memory。这意味着这里：

free(pool);
printf("%p\n", &pool);

您应该释放相同的内存。您尝试使用free 池对象，但它没有在堆上分配。在您的实现中，它是一个包含单个池的静态对象。尝试free 未在堆上分配的内存是未定义的行为，在您的情况下是崩溃。

上面的代码还显示了对free 的另一个误解：它不会更改分配内存的句柄。它仅将先前分配的字节标记为再次可用。您的程序必须防止通过您拥有其内存的指针freed 访问内存。

另外，&pool不是池对象的地址，而是局部指针变量pool的地址。

要修复您的崩溃，请将您的函数更改为：

void freePool(Pool *pool) 
{
    if (pool) free(pool->memory);
}