调用inet_ntoa处理IN_ADDR时原存储IP的变量值意外变化问题

核心异常来源是inet_ntoa的内置实现机制：

• 该函数内部使用进程全局的静态缓冲区存储转换后的点分十进制IPv4字符串，返回的char*指针直接指向这块静态内存
• 这块静态缓冲区是全局复用的，没有为每次调用单独分配独立内存，后续任意一次对inet_ntoa的调用，都会把新的IP字符串覆盖写入这块内存

对应你的代码和运行逻辑，异常的完整链路是：

1. 代码中char* IPName = inet_ntoa(WFIAddr.sin_addr); 拿到的根本不是独立存储的目标IP字符串，只是指向这块静态缓冲区的指针
2. 循环内每次执行printf(..., inet_ntoa(IPAddr))时，本质上触发了一次新的inet_ntoa调用，把当前遍历到的IP值写入同一块静态缓冲区，直接覆盖了之前存储的目标IP内容
3. 你的运行输出完全匹配这个机制：第一次打印IPName还是初始的10.0.4.3，调用一次inet_ntoa输出160.223.17.135之后，再读IPName就变成了刚写入的160.223.17.135，后续每遍历一个IP，IPName的值就跟着同步更新一次。

至于之前尝试用std::string存储没解决问题，基本是因为拷贝时机不对：没有在inet_ntoa返回的第一时间把字符串内容复制到独立内存，仍然保留了对全局静态缓冲区的引用，自然会被后续调用覆盖。

临时兼容方案（不改动整体接口逻辑） #

拿到inet_ntoa返回值的第一时间，把字符串内容拷贝到自己管理的独立内存，不要直接存储返回的指针：

// 申请16字节独立缓冲区（刚好存最长15字符的点分IPv4+字符串结束符）
char szTargetIP[16] = {0};
// 在inet_ntoa返回的瞬间就完成内容拷贝
strcpy_s(szTargetIP, inet_ntoa(WFIAddr.sin_addr));

如果用std::string存储，直接在构造时完成拷贝即可，不要延迟赋值：

// 正确写法：构造时直接拷贝inet_ntoa的返回字符串
std::string strTargetIP = inet_ntoa(WFIAddr.sin_addr);

长期推荐方案（替换弃用接口） #

直接使用系统提供的安全替代接口inet_ntop，该接口由调用者自行提供输出缓冲区，从机制上规避了全局静态缓冲区的覆盖问题，同时也是线程安全的：

// 预先定义足够长的缓冲区，INET_ADDRSTRLEN是系统定义的IPv4点分字符串最大长度（16）
char szTargetIP[INET_ADDRSTRLEN] = {0};
inet_ntop(AF_INET, &WFIAddr.sin_addr, szTargetIP, INET_ADDRSTRLEN);

// 遍历IP表的逻辑修改为
IN_ADDR IPAddr;
char szCurIP[INET_ADDRSTRLEN] = {0};
for (i = 0; i < (int)pIPAddrTable->dwNumEntries; i++) {
    IPAddr.S_un.S_addr = (u_long)pIPAddrTable->table[i].dwAddr;
    // 转换当前遍历的IP，输出到独立的szCurIP缓冲区
    inet_ntop(AF_INET, &IPAddr, szCurIP, INET_ADDRSTRLEN);
    // 此时szTargetIP的值完全不会被影响，可直接做字符串比较
    if (strcmp(szTargetIP, szCurIP) == 0) {
        ctn = FALSE;
        break;
    }
}

• 不要直接存储inet_ntoa的返回指针：这块静态缓冲区不属于调用者，也不会永久保存上一次的转换结果，后续调用必然会覆盖内容
• inet_ntoa的全局静态缓冲区设计也是它被标记为弃用的核心原因：既不是线程安全的，也极容易出现这类非预期的字符串覆盖问题
• 字符串比较时不要把inet_ntoa的返回值直接作为比较参数，否则两次调用的返回值会指向同一块内存，比较逻辑会完全失效

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