如何解决内存泄漏？

Question

我有

int main(void){
  while (1){
    char *input = NULL;
    char buf[1000];
    int read;
    size_t len;
    read = getline(&input, &len, stdin);

    if (-1 != read){
      sprintf(buf, "%s", input);
      lowercase(buf);; // custom function
      get_command(buf); // custom function already checked for memory leak
    }
    free(stdin);
    free(input);
    free(buf);
  }
  return 0;
}

通过valgrind运行此命令将返回：

HEAP SUMMARY
by 0x4022c2: main(prog.c:647) //  read = getline(&input, &len, stdin);

LEAK SUMMARY
still reachable: 120 bytes in 1 blocks

既然我释放了所有内容（stdin，input，buf），为什么它仍然给我内存泄漏？ 以及我该如何解决？

Answer 1

在注释中，您说您从函数get_command调用exit(0) 。 调用exit(x)行为就像程序从main返回x 。 这意味着您可以跳过main末尾的清理代码。

为了解决此问题，您可以从get_command返回一个值，例如对于常规操作为0，对于错误为-1或对于输入结束为1。 这意味着get_command中的exit(0)现在变为return 1 。

您的main循环如下所示：

int main(void)
{
    char *input = NULL;
    size_t len = 0;

    while (1) {
        if (getline(&input, &len, stdin) < 0) break;

        lowercase(input);
        if (get_command(input) == 1) break;
    }

    free(input);

    return 0;
}

请注意，我已修复了您的代码的其他一些问题：

• 不要使用固定大小的临时缓冲区。 getline可以读取任意长度的行。 如果将它们复制到临时缓冲区，至少要确保它适合。 毕竟，您已经获得了len信息。 最好直接使用input字符串。
• 每次调用getline时，请勿以NULL缓冲区开头。 这将减少分配的数量，因为仅当一行的行长于之前读取的每一行时，才会分配新的内存，这种情况不会经常发生。 这也意味着free应该在循环之后进行。