如何在C中正确分配结构数组哈希图中的项目

Question

在以下代码中，我正在使用malloc将新项目添加到哈希图中。我以为我已经检查了所有使用malloc的复选框，但是valgrind说我的内存泄漏。有人可以指出我出问题的地方吗？

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

typedef struct node
{
    char content[46];
    struct node* next;
}
node;

typedef node* hashmap_t;

int main (int argc, char *argv[]) {

    hashmap_t hashtable[1000];

    node *n = malloc(sizeof(node));
    if(n == NULL) return 0;

    hashmap_t new_node = n;

    new_node->next = malloc(sizeof(node));
    if(new_node->next == NULL) {
        free(n);
        return 0;
    }

    strncpy(new_node->content, "hello", 45);
    hashtable[10] = new_node;

    for(int y=0; y < 1000; y++) {
        if(hashtable[y] != NULL) {
            free(hashtable[y]->next);
            free(hashtable[y]);
        }
    }

    return 1;
}

Answer 1

在线

if(hashtable[y] != NULL)

您没有将hashtable初始化为任何值，它也被声明为局部变量。初始值应为某个垃圾值。因此，您不能假设数组的所有1000个元素的hashtable[y]是否为NULL。

您可以在声明例如时将结构初始化为零

hashmap_t hashtable[1000] = {0};

或您可以将其声明为全局变量。