客户端服务器通信c ++中收到的垃圾值

Question

我试图在 Windows 上编写一个简单的客户端服务器程序。我的服务器端代码如下所示。

#include <iostream>
#include <windows.h>
#include <winsock2.h>
#include <stdlib.h>
#include <stdio.h>

using namespace std;

int main()
{
    WSADATA WSAData;
    SOCKET serverSock, clientSock;
    SOCKADDR_IN serverAddr, clientAddr;

    WSAStartup(MAKEWORD(2,0),&WSAData);
    serverSock=socket(AF_INET,SOCK_STREAM,0);

    serverAddr.sin_addr.s_addr=INADDR_ANY;
    serverAddr.sin_family=AF_INET;
    serverAddr.sin_port=htons(5555);

    bind(serverSock,(SOCKADDR *)&serverAddr, sizeof(serverAddr));
    listen(serverSock,0);
    std::cout<<"Listening for incoming connections \n";

    char buffer[1024];
    //memset(buffer,0,sizeof(buffer));
    int clientAddrsize= sizeof(clientAddr);
    if(clientSock = ( accept(serverSock,(SOCKADDR *)&clientAddr, &clientAddrsize ) != INVALID_SOCKET) )
    {
        std::cout<<"Client connected \n";
        recv(clientSock,buffer,int(strlen(buffer)),0);
        std::cout<<"Client Says: " << buffer <<"\n" ;
        Sleep(10000);
        //memset(buffer,0,sizeof(buffer));
        closesocket(clientSock);
        closesocket(serverSock);
        WSACleanup();
    }
    return 0;
}

客户端代码如下：

#include <iostream>
#include <winsock2.h>
#include <stdlib.h>
#include <stdio.h>

using namespace std;

int main()
{
    WSADATA WSAData;
    SOCKET clientSock;
    SOCKADDR_IN addr;

    WSAStartup(MAKEWORD(2,0),&WSAData);
    clientSock=socket(AF_INET,SOCK_STREAM,0);

    addr.sin_addr.s_addr=inet_addr("127.0.0.1");
    addr.sin_family=AF_INET;
    addr.sin_port=htons(5555);

    connect(clientSock,(SOCKADDR *)&addr,sizeof(addr));
    std::cout <<"Connected to the server \n";
    //memset(buffer,0,sizeof(buffer));
    char buffer[1024]={'h','e','l','l','o','.','\0'};
    send(clientSock,buffer,int(strlen(buffer)),0);
    std::cout<<"Message sent \n" ;
    Sleep(10000);
    closesocket(clientSock);
    WSACleanup();
    std::cout<<"Socket closed \n";
    return 0;
}

我在服务器端看到以下输出：

 Listening for incoming connections
 Client connected
 Client Says: R☺

我期待的是Client Says: Hello.。不知道为什么我得到垃圾值。

谢谢

Answer 1

char buffer[1024];

这在自动范围内声明了一个本地 char 数组。该数组没有初始化任何东西，并且包含随机垃圾。

recv(clientSock,buffer,int(strlen(buffer)),0);

这会将buffer 传递给strlen。但是，buffer 包含垃圾，这是未定义的行为，最终结果可能是随机的，介于此时立即崩溃或来自 strlen 的一些奇怪的返回值之间的任何事情。

另外，recv 的返回值完全被忽略，根本不检查。

所有套接字系统调用都可能失败，每个套接字系统调用send、recv 和所有其他的都可能失败。此外，即使成功，recv 也不会向您提供任何保证，send 的所有内容都将在一个镜头中成为recved .

例如，如果 send 发送了 10 个字节，则对 recv 的第一次调用可能会返回 1 或 5，表示只有前一个或五个字节会立即被 recved。您不保证recv 将一次性收到这 10 个字节。否则，recv 将（显然）需要再次调用以接收其余消息。

您需要解决所有这些问题才能拥有可靠的客户端/服务器应用程序：

1. 检查所有系统调用的返回值。我所说的关于recv 的所有内容也适用于send。尝试send 10 字节可能会返回 7，表示仅发送了前 7 个字节，然后您必须实现适当的逻辑来发送剩余的字节。

2. 修复未定义的行为，将未初始化的值传递给strlen。