0

为了解释我的问题，这是极简主义的 main.cc：

#include "./main.h"

int main(int argc, char *argv[]) {
    char *buffer = new char[12];
    char *output = new char[12];
    FILE *input  = fopen("file.test", "r");

    while ( read_stdin(buffer, 12, input, output) ) {
        // Operations on output
        // (...)
    }

    fclose(input);
    delete[] output; output = 0;
    delete[] buffer; buffer = 0;

    return 0;
}

和 main.h：

#include <cstdio>
#include <cstring>

inline bool read_stdin(char *tmp_buffer, const size_t &len, FILE *input, char *&output) {
    output = fgets(tmp_buffer, len, input);
    if ( output != NULL ) {
        char *lf = strchr(output, '\n');
        if ( lf != NULL ) {
            *lf = '\0';
        }
        return true;
    }
    return false;
}

函数 read_stdin() 可以从 STDIN 读取，它解释了它的名字。

好吧，一切都按预期工作，但 valgrind 告诉我这样的事情：

==6915== 12 bytes in 1 blocks are definitely lost in loss record 1 of 1
==6915==    at 0x4C29527: operator new[](unsigned long) (in /usr/lib/valgrind/vgpreload_memcheck-amd64-linux.so)
==6915==    by 0x4008A2: main (main.cc:6)

我编译为g++ -O0 -g main.cc -o test

我知道这12个字节是“输出”，但是为什么会丢失一些字节？我使用 delete[]，即使 STDIN 或输入​​没有任何内容，输出也会为 NULL 对吗？

我误解了为什么还有这12个字节，我错在哪里？

先感谢您 ：）

编辑

感谢 Vaughn Cato、Dietmar Kühl 和 Richard J. Ross III，我改变了台词：

output = fgets(tmp_buffer, len, input);
    if ( output != NULL ) {

至

if ( fgets(output, len, input) != NULL ) {
4

1 回答 1

7

您已替换output为不同的指针，因此您不会删除分配的相同内容：

output = fgets(tmp_buffer, len, input);

我不确定为什么read_stdin有output参数。如果您只需要检查 fgets 的结果，那么您可以使用局部变量：

inline bool read_stdin(char *tmp_buffer, const size_t &len, FILE *input) {
    char *output = fgets(tmp_buffer, len, input);
    .
    .
    .
}
于 2013-01-04T00:20:50.350 回答