SIGINT 上的 C++ 程序内存泄漏

aza*_*zat 3 c c++ valgrind memory-leaks

我有一个程序,用作套接字客户端,这是代码

#include <stdio.h>
#include <signal.h>
#include <iostream>
#include <stdlib.h>
#include <string.h>
#include <string>
#include <sys/socket.h>
#include <netdb.h>

using namespace std;

void signalHandler(const int signal) {
    cout << "SIGINT handled" << endl;
    exit(EXIT_SUCCESS);
}

void error(const char *msg) {
    perror(msg);
    exit(0);
}

const string readStr(int descriptor) {
    int n = 0;
    string cmd = "";

    char buffer[256];
    memset(buffer, 0, sizeof(buffer));

    while ((n = read(descriptor, buffer, 255)) != 0) {
        if (n < 0) {
            error("Error reading string");
        }

        // full string - just copy
        if (n == 255) {
            cmd += buffer;
            memset(buffer, 0, sizeof(buffer));
        }
        else {
            cmd += buffer;
            break;
        }
    }

    return cmd;
}

int main(int argc, char** argv) {
    signal(SIGINT, signalHandler);

    int fd, port = (argc >= 3 ? atoi(argv[2]) : 11212), n;
    fd = socket(AF_INET, SOCK_STREAM, 0);
    if (fd < 0) {
        error("ERROR opening socket");
    }

    struct hostent *server = (argc >= 2 ? gethostbyname(argv[1]) : gethostbyname("localhost"));
    if (server == NULL) {
        error("ERROR no such host");
    }

    struct sockaddr_in addr;
    memset(&addr, 0, sizeof(addr));

    addr.sin_family = AF_INET;
    bcopy(server->h_addr, &addr.sin_addr.s_addr, server->h_length);
    addr.sin_port = htons(port);

    if (connect(fd, (struct sockaddr*)&addr, sizeof(addr)) < 0) {
        error("ERROR connecting");
    }

    string tmp;
    while (1) {
        cout << "Please enter the message: \n< " << flush;
        tmp = readStr(0);
        n = send(fd, tmp.c_str(), tmp.size(), 0);
        if (n < 0) {
            perror("ERROR writing to socket");
        }

        tmp = readStr(fd);
        cout << "> " << tmp << endl;
    }
    close(fd);

    return EXIT_SUCCESS;
}
Run Code Online (Sandbox Code Playgroud)

当我发送 SIGNINT 来处理(CTRL + C)时,valgrind 说:

^CSIGINT 已处理
==3798==
==3798== 堆摘要:
==3798== 退出时使用:1 个块中 39 个字节
==3798== 总堆使用量:45 次分配、44 次释放、已分配 4,778 字节
==3798==
==3798== 1个块中的39个字节可能在丢失记录1 of 1中丢失
==3798== 在 0x402471C: 运算符 new(unsigned int)
(vg_replace_malloc.c:255)
==3798== 0x40DBB64:std::string::_Rep::_S_create(无符号整数,
无符号整型,std::分配器常量&)(在
/usr/lib/libstdc++.so.6.0.15)
==3798== 0x1BFF403:???
==3798==
==3798== 泄漏摘要:
==3798==肯定丢失了:0个块中的0个字节
==3798==间接丢失:0个块中的0个字节
==3798==可能丢失:1个块中39个字节
==3798== 仍然可达:0 个块中的 0 个字节
==3798== 被抑制:0 个块中的 0 个字节
==3798==
==3798== 对于检测到和抑制的错误的计数,请使用以下命令重新运行:-v
==3798== 错误摘要:来自 1 个上下文的 1 个错误(已抑制:来自 9 个上下文的 20 个错误)

如果程序正确退出,例如如果 replate while(1)by int i = 0; while(i++ <= 2),则不会出现内存泄漏

那么有人可以帮我解决这个问题吗?

Jam*_*lis 5

当您std::exit从信号处理程序调用时,堆栈不会展开,并且像in 中的std::string对象这样的局部变量不会被销毁。tmpmain

由于tmp在程序终止之前不会被销毁,因此它分配的内存永远不会被释放,因此会发生泄漏。

  • 最好的做法是不要调用“std::exit”,而是让信号处理程序通知您的程序(可能通过某些消息队列或全局状态)它应该停止执行并让它干净地退出。 (2认同)