我来帮你排查这个棘手的问题——你获取到的IP总是类似x.127.0.0的奇怪值，而且每次还不一样，这大概率是栈缓冲区溢出导致的内存覆盖问题。

先看你贴的代码片段：

struct sockaddr_in client_sockaddr;
socklen_t tut = sizeof(sockaddr_in);
int valread; int new_socket;
new_socket = accept(sockFd, (struct sockaddr *)&client_sockaddr,&tut);
valread = read( new_socket , clientMsg, _maxBufSize);
std::cout << "ip: "<< inet_ntoa(client_sockaddr.sin_addr) << std::endl;

问题根源分析 #

你看到的异常IP，本质是client_sockaddr结构体的内存被意外篡改了。最可能的触发点是read()调用：如果clientMsg是在栈上定义的（比如char clientMsg[_maxBufSize];），而_maxBufSize设置得过大，或者客户端发送的数据长度超过了clientMsg实际能容纳的空间，read()写入的数据就会“越界”，直接覆盖栈上相邻的client_sockaddr变量，把原本正确的IP数据冲成了随机的异常值——也就是你看到的x.127.0.0这类奇怪地址。

顺带提一句：inet_ntoa()确实是个过时的函数，线程不安全，但你这里是紧接着read()就打印，所以它不是导致问题的主因。

具体解决方案 #

1. 修复缓冲区溢出问题

   • 先确认clientMsg的实际分配大小：如果是栈分配，用固定的合理值，比如：

     #define MAX_BUF_SIZE 1024
     char clientMsg[MAX_BUF_SIZE];
     valread = read(new_socket, clientMsg, MAX_BUF_SIZE);
     

   • 绝对不要让_maxBufSize超过clientMsg的实际容量，也可以考虑用动态分配（比如malloc）来避免栈空间不足的问题。

2. 不要忽略系统调用的返回值

   • 每次调用accept()和read()后都要检查返回值：比如new_socket如果是-1，说明连接失败，直接跳过后续操作；valread如果是-1，说明读取出错，也不要继续处理。这能避免很多异常情况。

3. 换成更安全的inet_ntop()

   • inet_ntoa()已经被淘汰了，inet_ntop()支持IPv6，而且线程安全，用法很简单：

     char ip_str[INET_ADDRSTRLEN];
     inet_ntop(AF_INET, &client_sockaddr.sin_addr, ip_str, INET_ADDRSTRLEN);
     std::cout << "ip: " << ip_str << std::endl;
     

4. 初始化结构体避免脏数据

   • 定义client_sockaddr时直接初始化，防止栈上的脏数据干扰：

     struct sockaddr_in client_sockaddr = {0};
     

验证方法 #

修改完代码后，先让客户端发送少量数据，看看能不能正确打印出192.168.42.80。如果还是有问题，可以打印client_sockaddr.sin_addr.s_addr的十六进制值，对比实际IP的网络字节序（192.168.42.80转成网络字节序是0x502AA8C0），看看是否匹配，进一步确认内存有没有被篡改。

内容的提问来源于stack exchange，提问作者PAvel Tchoudakov