0

下面我有一个来自我的服务器代码和客户端代码的代码段。我的客户端向服务器发送一个整数，服务器成功接收，然后我向服务器发送一个长度为 str_len 的字符串。服务器中的第二次读取不起作用，它正在阻塞并且不读取任何内容。当我退出客户端时，服务器会打印出它没有读取任何内容。怎么了？

            //Server code

            bzero(buffer,256);
            n = read(newsockfd,buffer,255);
            if (n < 0) error("ERROR reading from socket");
            unsigned int *length = new unsigned int;
            memcpy(length, buffer, sizeof(int));
            cout << "Length : " << *length << endl;
            int len = *length + 1;
            char buffIn[len+1];
            bzero(buffIn,len);
            //ok msg?
            n = read(newsockfd,buffIn,len);
            if (n < 0) error("ERROR reading from socket");
            cout << "value of n" << n << endl;
            printf("Received : %s\n", buffIn);


            //client method

            void send(string req)
            {
                //Send string len
                unsigned int str_len = req.length();
                //str_len = 3000;
                write(socketFd, &str_len, sizeof(str_len));
                //Send string
                const char *str_req = req.c_str();
                printf("%s\n",str_req);
                cout << "Str len is : " << strlen(str_req) << endl;
                write(socketFd, str_req, strlen(str_req) + 1);
                cout << "write done " << endl;
            }
4

2 回答 2

5

考虑这一行：

n = read(newsockfd,buffer,255);

您永远不会随后检查 的精确值n。您检查函数是否失败，但不检查读取了多少字节。提示：在你的情况下，它超过sizeof (int).

试试这个：

n = read(newsockfd, buffer, sizeof(int));

Nb：在其他情况下，也可能小于sizeof(int)。你也必须处理这种情况。

于 2013-02-24T00:15:15.140 回答
0

您从套接字读取 255 个字节的 str 长度，从而拉入数据的前 251 个（假设为 32 位整数）。因此，当您要求字符串的其余部分时，您要求额外的 251 个字节未发送，因此读取调用阻塞。

将长度读入整数

unsigned int length;
n=read(newsockfd,&length, sizeof(length));

在您当前的代码片段中，您没有删除长度

unsigned int *length = new unsigned int;

这将导致小的内存泄漏。

正如 Rob 在他的回答中所说，您应该检查读取的返回字节数，而不仅仅是检查返回的错误

于 2013-02-24T00:20:33.490 回答