内存泄漏结构字符 C

Question

我想我这里有一些内存泄漏。
也许有人可以告诉我在这种情况下如何处理泄漏的正确方法。

#include <stdio.h>
#include <stdlib.h>

struct adresse {
    char *name;
    int nummer;
};

int main() {

    int size = 2;
    struct adresse *a = (struct adresse *) malloc(sizeof(struct adresse) * size);

    for (int i = 0; i < size; i++) {
        a[i].name = "Testname";
        a[i].nummer = 123;
    }

    for (int i = 0; i < size; i++) {
        printf("%s, %d\n", a[i].name, a[i].nummer);
    }

    free(a);

    return 0;
}

Answer 1

我想我这里有一些内存泄漏。也许有人可以告诉我在这种情况下如何处理泄漏的正确方法。

在您编写的任何动态分配内存的代码中，对于分配的任何内存块，您都有 2 个职责：(1)始终保留指向起始地址的指针内存块，因此，(2) 当不再需要它时可以释放。

如果您分配它，请跟踪它，并在不再需要它时free 它。如果你没有分配它，你不能free它。如果你已经释放它，你不能再次free它。

" 如果参数与内存管理函数先前返回的指针不匹配，或者如果空间已通过调用 free 或 realloc 被释放，则行为未定义。"

C11 §7.22.3.3 The free function [p2] (n1570 draft)

你唯一分配的是a，然后你释放a -- 你没有内存泄漏。

内存使用/错误检查示例

您可以通过使用内存使用/错误检查程序（例如 Linux 上的valgrind -- 每个操作系统都有类似的程序）轻松确认是否存在内存泄漏。它们使用简单，只需通过它运行您的程序即可：

$ valgrind ./bin/leak
==23745== Memcheck, a memory error detector
==23745== Copyright (C) 2002-2015, and GNU GPL'd, by Julian Seward et al.
==23745== Using Valgrind-3.11.0 and LibVEX; rerun with -h for copyright info
==23745== Command: ./bin/leak
==23745==
Testname, 123
Testname, 123
==23745==
==23745== HEAP SUMMARY:
==23745==     in use at exit: 0 bytes in 0 blocks
==23745==   total heap usage: 1 allocs, 1 frees, 32 bytes allocated
==23745==
==23745== All heap blocks were freed -- no leaks are possible
==23745==
==23745== For counts of detected and suppressed errors, rerun with: -v
==23745== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 0 from 0)

始终确认您已释放所有已分配的内存并且没有内存错误。

(注意，取决于您的操作系统和valgrind 的实现，它可能会显示系统为您的进程分配的内存在程序退出时仍在使用。并非所有valgrind 的实现提供适当的抑制文件来屏蔽操作系统为您的进程而不是您分配的所有内存。只要您确认您分配的内存已被释放 - 您已经完成了您的工作。）

从上面的valgrind 输出中可以看出，您没有错误，并且所有分配的内存都已释放。 （干得好！）

Answer 2

您也应该 malloc 测试名称字符串。这使得释放所有内容变得更加复杂，因为您需要先释放所有 Testname 字符串，然后释放整个结构内存。

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

struct adresse {
  char *name;
  int nummer;
};

int main() {

    int size = 2;
    struct adresse *a = (struct adresse *) malloc(sizeof(struct adresse) * size);

    for (int i = 0; i < size; i++) {
      char *str = "Testname";
      size_t len = strlen(str);
      a[i].name = malloc(len+1);
      memset(a[i].name, 0, len+1);
      memcpy(a[i].name, str, len);
      a[i].nummer = 123;
    }

    for (int i = 0; i < size; i++) {
        printf("%s, %d\n", a[i].name, a[i].nummer);
        free(a[i].name);
    }

    free(a);

    return 0;
}