将 char** 传递给 void** 函数参数时，与指针类型不兼容的指针类型警告

如何解决将 char** 传递给 void** 函数参数时，与指针类型不兼容的指针类型警告

我正在尝试实现一个安全释放函数，该函数擦除分配的内存，释放它，然后还将指向分配区域的指针设置为 NULL，因此该指针不能在释放后重用，也不能被双重释放相同的功能。为了实现这一点，我使用了一个指向指针的参数，它允许我覆盖指向已分配内存的指针。

问题是 GCC 抱怨指针类型不兼容（“但它适用于我的机器”）；我没想到会有这样的警告。我的理解是任何指针都可以隐式转换为 void*，因此我猜测指针的 address 也可以转换为 void**。

与此同时，我将 secure_free() 重写为宏，解决了警告，但我想知道编译器为什么会抱怨。

文件securefree.c

#include <stdlib.h>
#include <stdint.h>
#include <string.h>

#define STRING_BUFFER_LEN 10

/**
 * Securely erases a heap-allocated memory section,frees it and sets its
 * pointer to NULL to avoid use-after-free and double-free.
 */
static void secure_free(void** p_p_data,size_t length_in_bytes)
{
    if (p_p_data == NULL || *p_p_data == NULL)
    { return; }
    memset(*p_p_data,length_in_bytes);
    free(*p_p_data);
    *p_p_data = NULL;
}

int main(void)
{
    // Allocate some data
    char* my_string = calloc(STRING_BUFFER_LEN,sizeof(char));
    if (my_string == NULL) { return 1; }
    // Use the allocated space in some way
    my_string[0] = 'a';
    my_string[1] = 'b';
    // Free using the dedicated function
    secure_free(&my_string,STRING_BUFFER_LEN);
    return 0;
}

使用 GCC 编译（Rev6，由 MSYS2 项目构建，10.2.0）：

$ gcc securefree.c -o securefree
securefree.c: In function 'main':
securefree.c:29:17: warning: passing argument 1 of 'secure_free' from incompatible pointer type [-Wincompatible-pointer-types]
   29 |     secure_free(&my_string,STRING_BUFFER_LEN);
      |                 ^~~~~~~~~~
      |                 |
      |                 char **
securefree.c:11:32: note: expected 'void **' but argument is of type 'char **'
   11 | static void secure_free(void** p_p_data,size_t length_in_bytes)
      |                         ~~~~~~~^~~~~~~~

编辑：宏版本看起来像这样

#define secure_free_macro(ptr,len) if ((ptr) != NULL) { \
        memset((ptr),(len)); free(ptr); (ptr) = NULL; }

解决方法

您尝试做的事情无法移植，因为不同的指针类型可以有不同的表示；并且要将空指针分配给该值，您必须将指向指针的first 转换为指向实际指针变量的有效类型 的指针——这是不可能的.

但是你可以做的是使用宏，它和任何宏一样好，而且使用起来要简单得多：

#define secure_free(x) (free(x),(x) = 0)

这在没有 & 的情况下有效。

,

C 允许将任何指针隐式强制转换为 void* 作为显式异常。请注意，void 和 char 是不兼容的类型。因此 void*、char*、void** 和 char** 也不兼容。这就是编译器发出警告的原因。

要绕过此问题，请更改函数签名以使用 void*：

void secure_free(void* ptr,size_t length_in_bytes) {
   void **p_p_data = (void**)ptr;
   ...
}

为了保护参数是指向指针的指针，可以使用宏：

#define secure_free(x,s) ((void)sizeof **(x),secure_free((x),(s)))

• 表达式 **(x) 将无法编译，因为 x 不是指向指针的指针。
• sizeof 防止对 x 中的 **(x) 求值以避免副作用
• (void) 使编译器停止抱怨未使用的值
• 逗号运算符(X,Y)，只返回Y的值，即secure_free(...)的返回值
• 对宏使用与函数相同的名称允许将 secure_free 扩展为宏，仅当它用作函数时。这允许使用 secure_free 作为指向函数的指针

额外说明。在代码中

    memset(*p_p_data,length_in_bytes);
    free(*p_p_data);

编译器可能会优化出 memset()。我建议强制转换为 volatile void * 以强制编译器生成清除代码。

编辑

此外，由于 memset 丢弃了 volatile 限定符，因此可能会使用循环清除内容。