释放字符指针

Question

我有一个在程序执行期间被多次调用的函数。在上述函数中，我有一个动态字符指针，我多次调整其大小。

我的问题是：我需要在函数结束之前释放这个指针吗？

void functionName()
{
 char *variable = (char *) malloc(0);
    //variable is resized with realloc x number of times

 //should free be called here?
 return;
}

我还应该注意，我尝试释放指针，但是，当我这样做时，gdb 会发出警告。

Answer 1

是的，你必须释放它，否则你会泄漏内存。您的代码应该如下所示：

void function(void) 
{
    char *variable = (char *)malloc(0);

    variable = realloc(variable, 10);
    // do something with new memory

    variable = realloc(variable, 20);
    // do something with more new memory

    free(variable);  // clean up
}

我认为调用 malloc(0) 有点奇怪。

Answer 2

需要指出的几点：

我看不到你如何在代码中使用 realloc() ，但是如果你像这样使用它，那就错了：

variable = realloc(variable, amount);

当它无法分配更多内存时，< code>realloc() 返回 NULL 但保持原始指针不变。在上面的行中，这意味着 variable 为 NULL 并且我们失去了对其指向的内存的访问权限，但该内存尚未被释放。正确的习惯用法是这样的：

void *tmp = realloc(variable, amount);
if(tmp)
  {
    // success! variable invalid, tmp has new allocated data
    variable = tmp;
  }
else
  {
    // failure! variable valid, but same size as before, handle error
  }

您应该使用第二个的原因是，使用 realloc() 时，失败是很糟糕的，但在许多情况下是可以恢复的，这与 malloc() 不同。 code> 其中失败通常意味着“停止一切并死亡”。

这是一个更有争议的问题，但是否应该像您一样转换 malloc() 和 realloc() 的返回值是值得怀疑的。考虑一下：

// functionally identical in C
char *a = malloc(10);
char *b = (char *)malloc(10);

在 C++ 中，必须进行强制转换，因为在 C++ 中，void * 无法隐式转换为其他指针类型。 （我认为这是一个语言错误，但这不是我判断的地方。）如果您的代码是 C++，那么您无论如何都应该使用 new 和 delete 。如果您的代码是 C 代码，但需要使用 C++ 编译器进行编译（由于某些愚蠢的原因），您别无选择，只能进行强制转换。如果您不需要使用 C++ 编译器编译 C 代码（这类似于必须在 Python 解释器中运行 Ruby 代码），请继续执行以下几点，这就是我认为您不应该进行强制转换的原因。

1. 在 C89 中，如果在没有声明的情况下使用函数，它将被隐式声明为返回 int。比如说，如果我们忘记了#include并且调用了malloc()，那么没有强制转换的版本将导致编译器错误（从不允许将 int 转换为 char *），而带有强制转换的版本会（错误地）告诉编译器“我知道这听起来很疯狂，但无论如何都要强制转换它。”大多数编译器都会向您发出有关 malloc() 等内置函数的隐式（或不兼容）声明的警告，但强制转换确实会使查找变得更加困难。

2. 假设您有一些数据：

   float *array = (float *)malloc(10 * sizeof(float));
   

   后来，您发现需要更精确的数据，并且必须将其设为 double 数组。在上面的行中，您需要更改的不同位置不超过 3 个：

   double *array = (double *)malloc(10 * sizeof(double));
   

   如果，另一方面，你写了：

   float *array = malloc(10 * sizeof *array);
   

   您只需将 1 个位置的 float 更改为 double 即可。此外，始终使用 sizeof *obj 而不是 sizeof(type) 并且从不使用强制转换意味着稍后调用 realloc() 可以工作< em>没有任何改变，而使用强制转换和显式类型名称则需要找到调用realloc的任何地方并更改强制转换和sizeof。另外，如果您忘记了，请执行以下操作：

   double *array = (float *)malloc(10 * sizeof(float));
   

   在大多数平台上，array 现在只是一个包含 5 个元素的数组，假设对齐未关闭并且编译器不会抱怨您正在分配 float * 到 double *。有些人认为编译器发出的警告很有帮助，因为它指出了可能不正确的行。但是，如果我们避免 sizeof(type) 并避免强制转换，我们可以看到这些行不会不正确，因此让编译器注意它们是浪费时间我们可以用来编程。

Answer 3

从手册页来看：

如果 size 为 0，则 malloc() 返回 NULL，或者稍后可以成功传递给 free() 的唯一指针值。

所以，我相信答案是“是”:)。

Answer 4

是的，您需要调用一次 free() 来释放内存块。您不需要为您正在执行的后续 reallocs() 调用 free，即使它们返回不同的地址/指针。内存管理器知道不再需要旧块并将其 free() 。

Answer 5

您应该能够在最后调用 free(variable) 。如果 realloc 必须移动数据才能调整其大小，它会在内部调用 free，您无需担心这一点。

另外，在初始化变量时，您可以将其设置为NULL，而不是调用malloc； realloc 将像第一次使用 malloc 一样工作。

Answer 6

看看我对与内存管理相关的几个问题给出的一些答案：

• 为什么内存泄漏很常见？
• 这有什么意义malloc(0)？
• C 字符串混淆？
• 需要帮助解决分段错误。
• C stdlib realloc 问题？
• C：从分隔字符串创建字符串数组。

以上所有内容都指出了显而易见的是，对于每个 malloc 都有一个空闲空间，如果没有，就会发生内存泄漏，因此当您使用完指针变量时，必须释放内存那就是mallocd。

希望这有帮助，
此致，
汤姆.

Answer 7

int main(int argc, char *argv[])
{
        char *p = malloc(sizeof(argv[1]));

        p = argv[1];
        printf("%s\n", p);
        free(p);
        return 0;
}

我收到 glibc 错误

hello
*** glibc detected *** ./a.out: munmap_chunk(): invalid pointer: 0x00007fff66be94c6 ***
======= Backtrace: =========
/lib/x86_64-linux-gnu/libc.so.6(+0x7eb96)[0x7f38dca1db96]
./a.out[0x4005ed]
/lib/x86_64-linux-gnu/libc.so.6(__libc_start_main+0xed)[0x7f38dc9c076d]