如何在列表内的句柄上使用 free？-> C->窗口API

Question

我有一个 C 语言列表，如下所示：

typedef struct _node
{
   int number;
   DWORD threadID;
   HANDLE threadH;
   struct *_node next;
} *node;

您也有这样的列表：

node new_node = malloc(sizeof(node));

正如您可能已经猜到的那样，该列表将存储线程的信息，包括它们的处理程序和 Id。当我尝试执行此操作时，我仍然遇到麻烦：

free(new_node);

每次我尝试执行此操作时，我都会遇到意外错误，VS 说存在数据损坏。我已经尽可能地确定了，当我尝试使用 free 句柄时，我发现问题存在。 我在 MSDN 上搜索了如何执行此操作，但我唯一能找到的是关闭线程的函数（这不是这里的目的，因为我希望线程运行，只需从列表中删除它的记录）。

问题是：我应该如何从内存中释放句柄？ （考虑到这只是句柄值的副本，活动句柄不会被删除）。

编辑：这是从列表中插入节点的函数：

int insereVisitanteLista(node* lista, DWORD threadID, HANDLE threadH, int num_visitante)
{
    node visitanteAnterior;
    node novoVisitante = (node)malloc(sizeof(node));

    if(novoVisitante == NULL)
        return 0;

    novoVisitante->threadID = threadID;
    novoVisitante->threadH = threadH;
    novoVisitante->number = num_visitante;
    novoVisitante->next = NULL;

    if(*lista == NULL)
    {
        *lista = novoVisitante;
        return 1;
    }

    visitanteAnterior = *lista;

    while(visitanteAnterior->next != NULL)
        visitanteAnterior = visitanteAnterior->next;

    visitanteAnterior->next =novoVisitante;
    return 1;
}

这​​是删除节点的函数：

int removeVisitanteLista(node * lista, DWORD threadID)
{
    node visitanteAnterior = NULL, visitanteActual;

    if(*lista == NULL)
        return 0;

    visitanteActual = *lista;

    if((*lista)->threadID == threadID)
    {
        *lista = visitanteActual->next;
        visitanteActual->next = NULL;
        free(visitanteActual);

        return 1;
    }

    while(visitanteActual != NULL && visitanteActual->threadID != threadID)
    {
        visitanteAnterior = visitanteActual;
        visitanteActual = visitanteActual->next;
    }

    if (visitanteActual == NULL)
        return 0;

    visitanteAnterior->next = visitanteActual->next;
    free(visitanteActual);

    return 1;
}

Answer 1

您尝试释放的节点到底是什么？这是一个指向 struct _node 的指针吗？如果是，您之前分配过吗？如果不是，则不需要free，否则您必须检查节点是否不NULL，并确保您不会多次free它。如果没有重现问题的最小工作示例，很难猜测您在做什么以及错误在哪里。我唯一可以建议的是阅读有关 C 语言内存管理的内容。此资源可能会有所帮助。

更新：

代码中的节点是指向 _node 的指针。所以 sizeof (node) 是指针的大小，它是 4 或 8 字节（取决于体系结构）。例如，您分配了 8 个字节，但假设您有一个指向更大的结构的指针。结果，你破坏了内存，并且程序的行为变得不确定。因此，将节点 novoVisitante = (node)malloc(sizeof(node)) 更改为节点 novoVisitante = (node)malloc(sizeof(_node)) 应该可以解决问题。

Answer 2

您没有向我们展示您调用 free() 的上下文，因此我需要进行一些推测，但我首先担心的是您没有提到在删除节点之前从列表中删除该节点。

首先通过修改前一个（或头）节点的下一个字段来取消链接节点。如果仍然出现错误，则说明您通过写入已分配的内存结构之一的末尾或类似的内容以某种方式损坏了内存。

另外，我假设节点是一个指针。您确实没有提供太多有关您正在做的事情的信息。