C 在释放内存时将 double 类型的变量转换为 char 类型的变量

Question

在处理库时，在提供的示例源代码中我发现了以下代码:

x     = (double *) malloc (narcs  * sizeof (double));
dj    = (double *) malloc (narcs  * sizeof (double));
pi    = (double *) malloc (nnodes * sizeof (double));
slack = (double *) malloc (nnodes * sizeof (double));

我在这里没有发现什么特别之处，但是在释放内存时，源代码示例执行以下操作:

free_and_null ((char **) &x);
free_and_null ((char **) &dj);
free_and_null ((char **) &pi);
free_and_null ((char **) &slack);

free_and_null 代码是:

static void
free_and_null (char **ptr)
{
    if ( *ptr != NULL ) {
    free (*ptr);
    *ptr = NULL;
    };
};

我的问题不是为什么在释放之前分配的内存时，会强制转换为 char 双指针。我的问题是为什么要这样做，使用自定义函数来释放内存，以及为什么选择 char ** 作为该函数的参数。

我知道这个问题肯定会发生，因为我对 C 语言的了解仍然有限，但无论如何，任何人都可以解释为什么这样做以及这是否是一个好的做法。

Answer 1

我不得不同意你的看法——这段代码有点奇怪。作者没有理由不能同样轻松地使用 void **。

此外，if (*ptr != NULL) 检查是不必要的，因为 free(NULL) 是完全合法的。大括号后的 ; 字符也不是必需的。

一个更简单的版本可能是这样的:

static void free_and_null(void **ptr)
{
    free(*ptr);
    *ptr = NULL;
}

这个函数存在的原因是确保释放的指针设置为NULL，这有助于检测释放后使用错误并完全避免双重释放错误。

编辑:正如 StarPilot 在下面建议的那样，检查 ptr 本身是否为 NULL 可能是个好主意:

static void free_and_null(void **ptr)
{
    if (ptr)
    {
        free(*ptr);
        *ptr = NULL;
    }
}