Question

我正在使用C进行一些学习，并且无法识别内存泄漏情况。

首先，一些代码：

我的主要功能：

#define FILE_NAME "../data/input.txt"

char * testGetLine( FILE * );
int testGetCount(void);

int main(void)
{
    int count = 0;
    FILE * fptr;

    if ((fptr = fopen(FILE_NAME, "r")) != NULL) {
        char * line;
        while ((line = testGetLine(fptr)) != NULL) {            

            printf("%s", line);
            free(line); count++;
        }

        free(line); count++;

    } else {
        printf("%s\n", "Could not read file...");
    }

    // testing statements
    printf("testGetLine was called %d times\n", testGetCount());
    printf("free(line) was called %d times\n", count);

    fclose(fptr);
    return 0;
}

和我的 getline 功能：

#define LINE_BUFFER 500

int count = 0;

char * testGetLine(FILE * fptr)
{
    extern int count;

    char * line;
    line = malloc(sizeof(char) * LINE_BUFFER);
    count++;

    return fgets(line, LINE_BUFFER, fptr);
}

int testGetCount(void) {
    extern int count;
    return count;
}

我的理解是，每当我调用free函数时，我都需要调用testGetLine。根据我的统计，在一个包含四行的简单文本文件中，我需要免费调用 5 次。我在以下输出中用我的测试语句验证：

This is in line 01
Now I am in line 02
line 03 here
and we finish with line 04
testGetLine was called 5 times
free(line) was called 5 times

我遇到的问题是，valgrind说我alloc 6 次，我只打电话给free 5 次。这是valgrind的截断输出：

HEAP SUMMARY:
    in use at exit: 500 bytes in 1 blocks
  total heap usage: 6 allocs, 5 frees, 3,068 bytes allocated
500 bytes in 1 blocks are definitely lost in loss record 1 of 1
   at 0x4C2B3F8: malloc (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
   by 0x4007A5: testGetLine (testGetLine.c:13)
   by 0x400728: main (tester.c:16)
LEAK SUMMARY:
   definitely lost: 500 bytes in 1 blocks
   indirectly lost: 0 bytes in 0 blocks
     possibly lost: 0 bytes in 0 blocks
   still reachable: 0 bytes in 0 blocks
        suppressed: 0 bytes in 0 blocks

我觉得我缺少内存管理方面的东西。 valgrind说我正在使用的第6个内存分配在哪里？我应该如何释放它？

跟进实施Adrian的回答

testGetLine调整：

char * testGetLine(FILE * fptr)
{
    extern int count;

    char * line;
    line = malloc(sizeof(char) * LINE_BUFFER);
    count++;

    if (fgets(line, LINE_BUFFER, fptr) == NULL) {
        line[0] = '\0';
    }

    return line;
}

循环调整时

main：

while ((line = testGetLine(fptr))[0] != '\0') {            

    printf("%s", line);
    free(line); count++;
}

free(line); count++;

Answer 1

fgets返回说明：

成功时，该函数返回str。如果文件结尾是在尝试读取字符时遇到，eof指示符是设（feof）。如果在任何字符被读取之前发生这种情况，那么返回的指针是一个空指针（并且str的内容保留不变）。如果发生读取错误，则错误指示符（ferror）为设置并返回一个空指针（但指向的内容 str可能已经改变了。）

当fgets没有读取任何内容时，它不会返回您使用malloc的char *。

因此，您上次通话中的malloc未被释放。一段时间之后的声明无法正常工作。

解决方案：更改您的回报并返回line代替：

char * testGetLine(FILE * fptr)
{
    extern int count;

    char * line;
    line = malloc(sizeof(char) * LINE_BUFFER);
    count++;
    fgets(line, LINE_BUFFER, fptr);
    return line;
}

C内存泄漏和Valgrind输出

1 个答案: