将指针初始化为在函数中动态分配的指针

Question

当我试图存储从套接字接收的数据时，我将一个指向指针的指针传递给一个函数以动态分配它。它适用于一个请求，第二个请求通常会导致段错误。 Valgrind 抱怨：条件跳转或移动取决于引用我的响应指针的未初始化值。

如何初始化指针或者我可以做些什么来保证这个安全？在main函数中释放它是否正确？

int main(int argc, char **argv) {
    char * response;
    char readbuf[BUFFSIZE + 1] = "";
    //here I read some data into readbuff which I will send to the server below
    handle_request_data(readbuf, &response);
    //do some stuff with response, send to another socket
    free(response); // can I do that?
}

int handle_request_data(char * readbuf, char ** response) {
    //create tcp socket, connect to it and send readbuf to server

    int recv_total = 0;
    char buffer[BUFFSIZE + 1] = "";
    *response = malloc(BUFFSIZE + 1);
    while ((tmpres = recv(sock_tcp, buffer, BUFFSIZE, 0)) > 0) {

        if (recv_total > 0) {
            //need more memory for buffer
            *response = realloc(*response, BUFFSIZE + recv_total + 1);
        }
        memcpy(*response + recv_total, buffer, tmpres);
        recv_total += tmpres;
    }
}

感谢您的帮助！

Answer 1

1) Q: 将指针初始化为在函数中动态分配的指针

警告：您应该只在函数内部初始化一次指针。如果您在其他地方（重新）初始化它，您可能会面临内存泄漏或其他灾难的风险。

2) Q：在main函数中释放是否正确？

当然:)。但同样的警告：只释放它一次。并且，理想情况下，养成释放它后将其设置为 NULL 的习惯。

3) Q: 对第一个请求进行查找，第二个请求通常会导致 seg 错误。

我的猜测是您踩代码中某处的指针。

问：“缓冲区”在哪里定义？你确定你没有在 recv() 中覆盖它（并破坏你的“响应”指针）吗？

问：段错误究竟发生在哪里？你能在调试器中单步调试那部分代码吗？

请发回你找到的东西！

Answer 2

首先，简单的：

在main函数中释放它是否正确？

为什么不呢？反正你什么也做不了。

第二，困难的部分：嗯，你应该尝试使用调试器来调试你的代码。乍一看，这似乎很好，但我们永远不知道。你在使用多线程吗？您确定不存在竞争条件吗？等等。

Answer 3

什么是缓冲区？ handle_request_data 内部未定义。全球？ 你的意思是 readbuf 吗？未使用此参数。