Valgrind报告getaddrinfo泄漏内存?

6

我一直在我们的项目上运行Valgrind,但是Valgrind一直报告说从调用getaddrinfo中丢失了内存,尽管在函数底部调用了freeaddrinfo。有什么想法可能导致这种情况发生?

int tcp_connect(char *address, char *port)
{
//printf("%s\n ", address);
    int status, sockfd;
    struct addrinfo hints, *servinfo, *p;

    memset(&hints, 0, sizeof hints);
    hints.ai_family = AF_UNSPEC;
    hints.ai_socktype= SOCK_STREAM;
    status = getaddrinfo(address,port,&hints,&servinfo);
    if(status != 0){
        printf("tcpconnect: getaddrinfo failed\n");
    }
    for(p = servinfo; p != NULL; p = p->ai_next) {
        sockfd = socket(p->ai_family, p->ai_socktype, p->ai_protocol);
        if(sockfd == -1){
            printf("tcpconnect: socket failed\n");
            continue;
        }

        status = connect(sockfd, p->ai_addr, p->ai_addrlen);
        if(status == -1){
            close(sockfd);
            printf("tcpconnect: connect failed\n");
            continue;
        }
        break;
    }

    if (p == NULL){
        printf("tcpconnect: could not get a valid socket file descriptor\n");
        return -1;
    }

    freeaddrinfo(servinfo);
    //return socket
    return sockfd;
}

以下是valgrind的输出结果:
==7803== 384 bytes in 8 blocks are definitely lost in loss record 30 of 37
==7803==    at 0x4023CC8: malloc (vg_replace_malloc.c:236)
==7803==    by 0x413FE46: gaih_inet (in /lib/libc-2.7.so)
==7803==    by 0x4141CA1: getaddrinfo (in /lib/libc-2.7.so)
==7803==    by 0x804D20D: tcp_connect (tcpconnect.c:18)
==7803==    by 0x804D505: update_radio_data_t (radiodata.c:69)
==7803==    by 0x4073368: start_thread (in /lib/libpthread-2.7.so)
==7803==    by 0x4159CFD: clone (in /lib/libc-2.7.so)

我对这个内存泄漏感到困惑,因为我确保在底部调用了freeaddrinfo。这可能是Valgrind或freeaddrinfo的一个bug吗?


如果 p == NULL,则 freeaddrinfo 不会在末尾被调用。 - Ram
2个回答

9

你的错误路径(if (p == NULL))没有调用 freeaddrinfo,我建议将此调用移到检查语句之前。


4
if (p == NULL){
    printf("tcpconnect: could not get a valid socket file descriptor\n");
    return -1;
}

如果这个条件有一天变成真的,freeaddrinfo(servinfo) 将不会被调用。

网页内容由stack overflow 提供, 点击上面的
可以查看英文原文,
原文链接