WinSock c ++ inet_ntop始终显示204.204.204.204（并且accept（）并未失败）

Question

我正在尝试制作Winsock服务器，并且我想在客户端连接时在服务器上显示客户端的ip，但这就是问题所在。 每次尝试连接时，都会显示204.204.204.204。 我尝试连接另一台计算机，但结果相同。 导致本地主机

之后，我开始在这个网站上寻找与我有相同问题的人，我发现有几个与我有相同问题的人，但是他们所有人的accept或inet_ntop函数均无法正常工作。 因此，我检查了这两个函数，均未返回错误。 也许我很傻，但是我真的不知道出了什么问题。 （顺便说一句英语不是我的母语，所以请告诉我您是否注意到我的英语，或者我的英语还不错）

代码中不起作用的部分

sockaddr_in from;
    int clientlen = sizeof(from);
    // accept
    SOCKET client = accept(server, (SOCKADDR*)&client, &clientlen);
    if (client == INVALID_SOCKET)
    {
        std::cout << "Error in accept(): " << WSAGetLastError << std::endl;
        WSACleanup();
    }
    else
    {

        char clientIp[17];
        if (inet_ntop(AF_INET, &from.sin_addr, clientIp, 17) == NULL)
        {
            std::cout << "Can't get the client's ip: " << WSAGetLastError() << std::endl;
        }

        std::cout << "ip connected: " << clientIp << std::endl;

整个代码（如果需要）

#include <iostream>
#include <WinSock2.h>
#include <WS2tcpip.h>
#include <string>

#pragma comment(lib, "ws2_32.lib")

int main()
{
    std::cout << "--- Tcp/ip Server ---" << std::endl;
    WSADATA wsa;
    WSAStartup(MAKEWORD(2, 2), &wsa);

    SOCKET server = socket(AF_INET, SOCK_STREAM, 0);
    if (server == INVALID_SOCKET)
    {
        std::cout << "error in SOCKET(): "<< WSAGetLastError() << std::endl;
        WSACleanup();
    }
    sockaddr_in s;
    s.sin_family = AF_INET;
    s.sin_addr.s_addr = INADDR_ANY;
    s.sin_port = htons(52000);

    // bind
    if (bind(server, (sockaddr*)&s, sizeof(s)) == SOCKET_ERROR)
    {
        std::cout << "Error: bind()" << std::endl;
    }
    //listen
    if (listen(server, SOMAXCONN) == SOCKET_ERROR)
    {
        std::cout << "Error in listen(): " << WSAGetLastError() << std::endl;
        WSACleanup();
    }
    sockaddr_in from;
    int clientlen = sizeof(from);
    // accept
    SOCKET client = accept(server, (SOCKADDR*)&client, &clientlen);
    if (client == INVALID_SOCKET)
    {
        std::cout << "Error in accept(): " << WSAGetLastError << std::endl;
        WSACleanup();
    }
    else
    {

        char clientIp[17];
        if (inet_ntop(AF_INET, &from.sin_addr, clientIp, 17) == NULL)
        {
            std::cout << "Can't get the client's ip: " << WSAGetLastError() << std::endl;
        }

        std::cout << "ip connected: " << clientIp << std::endl;

        // the code isn't finished yet

        system("pause");
        WSACleanup();
    }
    return 0;
}

Answer 1

您在accept()的第二个参数中传递了错误变量的地址。

您正在传递将要向其分配accept()结果的SOCKET client变量的地址。 当在同一条语句中声明和初始化变量时，C ++允许使用变量的地址。 但这不是您想要的。 您需要改为通过变量传递sockaddr_in from的地址：

sockaddr_in from;
int clientlen = sizeof(from);
// accept
SOCKET client = accept(server, (SOCKADDR*)&from, &clientlen); // <-- &from instead of &client

你离开你from变量初始化，和你的编译器填充未初始化的变量具有0xCC （十进制204）字节在调试模式下，所以这就是为什么你最终看到204.204.204.204 （十六进制0xCC 0xCC 0xCC 0xCC ）从inet_ntop()时，你不from正确初始化from变量。