基于Winsock2的C++多线程TCP服务器客户端重连崩溃问题

核心崩溃原因分析 #

1. 多线程下std::vector的非同步访问 #

服务器在客户端线程中修改存储客户端信息的std::vector（移除断开的客户端），同时主线程可能在遍历、查询该容器，全程未加锁同步。这种场景下极易触发容器迭代器失效或数据竞争，直接导致服务器崩溃。

2. closesocket后的无效资源引用 #

• 客户端断开后，调用closesocket但未彻底清理相关内存/引用，后续代码可能误操作已释放的SOCKET句柄。
• 重启客户端时，操作系统可能复用旧的SOCKET描述符，导致服务器逻辑混淆，将新连接识别为旧客户端的残留资源。

3. 线程退出的资源泄漏 #

客户端处理线程退出时，未正确释放线程相关资源，或线程对象未完成销毁，引发内存冲突。

具体修复步骤 #

1. 给客户端列表添加线程同步机制 #

在Clients类中引入互斥锁，确保所有对std::vector的读写操作都在锁保护下执行：

// Clients类内新增
#include <mutex>
std::mutex client_mtx;

// 遍历客户端列表时加锁
void ShowOnlineClients() {
    std::lock_guard<std::mutex> lock(client_mtx);
    for (const auto& client : clients) {
        // 输出客户端IP、SOCKET等信息
    }
}

// 移除客户端时加锁
void RemoveClient(SOCKET target_sock) {
    std::lock_guard<std::mutex> lock(client_mtx);
    auto it = std::remove_if(clients.begin(), clients.end(), 
        [target_sock](const ClientInfo& info) {
            return info.sock == target_sock;
        });
    clients.erase(it, clients.end());
}

2. 规范客户端断开处理流程 #

当recv()返回0或SOCKET_ERROR时，严格遵循加锁移除→关闭SOCKET→退出线程的顺序：

int recv_len = recv(client_sock, buffer, BUFFER_SIZE, 0);
if (recv_len <= 0) {
    // 先加锁移除客户端记录，避免其他线程访问无效资源
    client_list->RemoveClient(client_sock);
    // 关闭SOCKET句柄
    closesocket(client_sock);
    // 正常退出线程
    return 0;
}

3. 避免SOCKET复用的逻辑冲突 #

不要仅用SOCKET描述符作为客户端的唯一标识，给每个新连接分配独立的UUID或递增ID，结合IP+端口+ID来区分客户端，彻底规避SOCKET复用带来的混淆。

4. 修正线程管理逻辑 #

• 使用_beginthreadex创建线程时，确保传递的参数（如客户端信息指针）生命周期覆盖线程全程，避免线程启动后参数已失效。
• 线程退出时调用_endthreadex释放线程资源，不要直接返回。

复现验证建议 #

1. 启用VS调试模式，服务器崩溃时查看调用栈，定位到崩溃的具体代码行（大概率是std::vector操作或SOCKET操作的位置）。
2. 添加日志输出，记录客户端断开、移除、新连接的每一步操作，对比崩溃前的日志序列，确认触发条件。

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