Question

所以我还有另一个新手问题。此函数用于读取传入文件中的所有字节，将它们存储在堆中，然后将地址存储到传入的内容中的那些字节中。参数和传入的长度＆＃39;长度＆＃39;参数。

bool load(FILE* file, BYTE** content, size_t* length)
{
if (file == NULL)
{
    return false;
}

//array to hold the bytes??
BYTE* buffer = malloc(sizeof(BYTE));

//To hold the number of bytes currently loaded
size_t size = 0;


//Pointer to keep track of where I am adding the data to in buffer

//get all the bytes from the file and put them in the buffer
for(int i = fgetc(file); i != EOF; i = fgetc(file))
{
    buffer[size] = (char) i; 
    size++;
    buffer = realloc(buffer, size + 1);
}

//dereference length
*length = size;

//derefernce content
*content = buffer;

//free(buffer);
return true;
}

所以之前这个函数所属的大型程序没有用，但是当我注释掉

时

free(buffer);

在底部打电话，我的程序开始完美运行。当我遇到双重免费错误时，我有动力对此发表评论。所以我的问题是：为什么在这个实例中调用free会导致错误？

我的直觉告诉我，这是因为数据

*content

指向现在＆＃34;删除＆＃34;所以我的计划不起作用。此外，在代码中的某个地方我也免费提供内容*，这是双重免费错误的来源。但出于某种原因，我倾向于认为数据实际上并没有被删除＆＃34;。

很抱歉，如果这很多，我一直对记忆分配，免费和指针感到困惑，并试图加深理解。

Answer 1

分配内存（或一般资源）时，必须确保具有明确的所有权语义。谁拥有分配的资源并负责释放它？

对于分配资源的函数，理智的语义是如果成功则函数返回分配的资源，除非另有指定，否则调用者拥有该资源。如果函数失败，调用者不必执行任何清理。

您的load()函数分配缓冲区。在其大部分功能体中，load()拥有该缓冲区。在它返回成功之前，它实际上将该缓冲区的所有权传递给调用者（通过content输出参数）。如果load()在分配缓冲区后出现故障点，那么它应该沿着该故障路径调用free(buffer)。

我也不得不指出代码中的一些问题：

BYTE* buffer = malloc(sizeof(BYTE));

您应该测试malloc是否失败。此外，sizeof (BYTE)没有用，因为根据定义，它是1。

buffer = realloc(buffer, size + 1);

这是不好的做法。如果realloc失败，您将丢失buffer的旧值并且会泄漏内存。最好这样做：

BYTE* tmp = realloc(buffer, size + 1);
if (tmp == NULL)
{
    free(buffer);
    return false;
}
buffer = tmp;

最后，每次将缓冲区增加1个字节效率非常低。更典型的做法是将缓冲区大小加倍或使其增长量更大。

为什么不在此实例中使用free（）

1 个答案: