C++ socket服务端server.cpp读取char*类型接收值为空问题求助

排查步骤 #

• 检查read返回值n的数值：你当前代码仅判断了n<0的场景，未覆盖其他返回值情况，在read之后新增打印cout << "read返回值为：" << n << endl;即可确认问题：
  • 若n>0：说明实际已读到数据，大概率是客户端发送的数据末尾带了换行、回车、空格等多余字符，导致和本地存储的用户名匹配失败，看似是空值实则有内容
  • 若n=0：说明客户端建立连接后未发送数据就主动断开了连接，需排查客户端发送逻辑
  • 若n=-1：新增perror("read error");打印错误码，确认是信号中断、socket状态异常还是其他系统层面问题
• 抓包确认数据传输：使用tcpdump -i lo port 你的服务端口抓本地环回口的包，确认客户端连接建立后确实发送了数据，排除客户端侧问题
• 打印实际收到的二进制内容：将读到的内容转十六进制打印，确认是否存在不可见字符干扰逻辑判断：

for(int j=0;j<n;j++){
    printf("%02x ", (unsigned char)publicKeyResponse[j]);
}
printf("\n");

修复方案 #

1. 完善read逻辑，处理边界情况 #

bzero(publicKeyResponse, 256);
n = read(t, publicKeyResponse, sizeof(publicKeyResponse)-1); // 预留1个字节放结束符，避免字符串溢出
if (n < 0) {
    perror("read error");
    close(t); // 出错关闭当前连接socket，避免资源泄漏
    continue; // 继续监听下一个连接，不要直接退出整个服务
} else if (n == 0) {
    cout << "客户端已断开连接" << endl;
    close(t);
    continue;
}
// 去掉末尾的换行、回车符，避免匹配失败
while(n > 0 && (publicKeyResponse[n-1] == '\n' || publicKeyResponse[n-1] == '\r')) {
    publicKeyResponse[--n] = '\0';
}

2. 修复现有逻辑漏洞 #

你原有用户名匹配逻辑存在错误，会重复发送does not exist响应，同时处理完连接未主动关闭socket，修复参考：

else
{
    bool found = false;
    for(int i = 0; i < data.size(); i++)
    {
        if (data.at(i).first == publicKeyResponse)
        {
            const string& pKey = data.at(i).second;
            n = write(t, pKey.c_str(), pKey.size());
            found = true;
            break; // 找到匹配项后直接跳出循环
        }
    }
    if(!found)
    {
        n = write(t, "does not exist.", 15);
    }
    close(t); // 处理完请求后关闭当前连接
}

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