核心 Bug:客户端与服务器端口不匹配(导致请求无法送达)

Bug 分析报告

1. 核心 Bug:客户端与服务器端口不匹配(导致请求无法送达)
  • 问题描述 :客户端代码中定义的服务器端口为 69(static const int PORT = 69),而服务器代码中绑定的端口为 6969(static const int PORT=6969)。由于 UDP 通信中客户端必须发送数据到服务器实际监听的端口,端口不匹配会导致服务器无法接收客户端的下载 / 上传请求,最终表现为功能失败。
  • 定位代码
    • 客户端端口定义(TFTPClient类私有成员):

      cpp

      运行

      复制代码
      static const int PORT = 69;  // 客户端使用的服务器端口
    • 服务器端口定义(TFTPServer类私有成员):

      cpp

      运行

      复制代码
      static const int PORT=6969;  // 服务器实际监听的端口
2. 次要 Bug:客户端输入缓冲区处理不当(可能导致文件名读取失败)
  • 问题描述 :客户端run()函数中,用户输入选择(1/2/3)后调用waitForInput()清空缓冲区,但在doDownload()doUpload()中使用getline(cin, filename)读取文件名时,可能因缓冲区残留数据导致读取空字符串或错误内容。
  • 定位代码
    • run()中输入选择后调用waitForInput()

      cpp

      运行

      复制代码
      cin >> choice;   // 输入选项
      waitForInput(); // 吸收垃圾字符
    • waitForInput()实现(仅清空到换行符,但未考虑cin的状态):

      cpp

      运行

      复制代码
      void TFTPClient::waitForInput() {
          while (getchar() != '\n'); // 纯清空输入缓冲区直到换行符
      }
    • 若用户输入选择后有多余字符(如1 abc\n),waitForInput()会将abc也清空,导致后续getline需要重新输入,但更严重的是,若cin状态异常(如输入非字符),可能导致getline直接失败。

Bug 解决方案

1. 修复端口不匹配问题
  • 修改客户端端口 :将客户端TFTPClient类中的PORT修改为与服务器一致的 6969,确保客户端请求能发送到服务器监听的端口。

    cpp

    运行

    复制代码
    // 客户端TFTPClient类私有成员修改
    static const int PORT = 6969;  // 与服务器端口保持一致
2. 优化输入缓冲区处理(避免文件名读取失败)
  • 改进waitForInput()函数 :在清空缓冲区前先检查cin的状态,确保输入流正常,避免因输入异常导致后续getline失效。

    cpp

    运行

    复制代码
    void TFTPClient::waitForInput() {
        // 若cin处于错误状态,先清空错误标志
        if (cin.fail()) {
            cin.clear();
        }
        // 清空缓冲区直到换行符
        while (getchar() != '\n');
    }
  • getline前手动清空缓冲区 :尽管waitForInput()已处理,但为确保getline能正确读取文件名,可在doDownload()doUpload()getline前再做一次缓冲区检查(可选增强):

    cpp

    运行

    复制代码
    // doDownload()中读取文件名前添加
    cout << "请输入要下载的文件的文件名称:";
    cin.ignore(numeric_limits<streamsize>::max(), '\n'); // 强制清空残留缓冲区
    getline(cin, filename);
    
    // doUpload()中同理
    cout << "请输入要上传的文件名:";
    cin.ignore(numeric_limits<streamsize>::max(), '\n');
    getline(cin, filename);
3. 补充错误处理(增强鲁棒性)
  • 服务器路径拼接容错 :服务器handleReadRequesthandleWriteRequest中拼接文件路径时,处理root_dir/结尾的情况(避免出现//):

    cpp

    运行

    复制代码
    // 服务器拼接路径时修改
    string full_path;
    if (!root_dir.empty() && root_dir.back() == '/') {
        full_path = root_dir + filename;
    } else {
        full_path = root_dir + "/" + filename;
    }
  • 客户端超时处理 :客户端recvfrom时添加超时机制,避免因服务器无响应导致程序卡死(使用setsockopt设置SO_RCVTIMEO):

    cpp

    运行

    复制代码
    // 客户端构造函数中添加超时设置(如5秒超时)
    struct timeval timeout = {5, 0}; // 5秒
    setsockopt(sfd, SOL_SOCKET, SO_RCVTIMEO, &timeout, sizeof(timeout));

总结

主要问题是客户端与服务器端口不匹配,导致请求无法送达,修复端口一致即可解决核心功能失败问题。输入缓冲区处理和错误处理的优化可进一步提升程序稳定性。

相关推荐
领尚9 分钟前
openclaw 极简安装(Ubuntu 24.04 server)
linux·运维·ubuntu
迷途之人不知返27 分钟前
shell相关知识与Linux权限
linux
SPC的存折31 分钟前
3、主从复制实现同步数据过滤
linux·运维·服务器
SPC的存折33 分钟前
openEuler 24.03 MariaDB Galera 集群部署指南(cz)
linux·运维·服务器·数据库·mysql
芯智工坊42 分钟前
第19章 Mosquitto完整项目实战
网络·人工智能·mqtt·开源
SPC的存折1 小时前
MySQL 8.0 分库分表
linux·运维·服务器·数据库·mysql
cyber_两只龙宝1 小时前
【Oracle】Oracle之DQL中WHERE限制条件查询
linux·运维·数据库·云原生·oracle
22信通小白1 小时前
USRP初学者使用手册(基础配置及bug记录)——Linux+Clion(单台X310收发)
linux·运维·c++·5g·bug·信息与通信
网络安全许木1 小时前
自学渗透测试第14天(信息收集进阶与指纹识别)
linux·网络安全·渗透测试
灰子学技术2 小时前
Envoy 底层 TCP 交互、UDS 和事件驱动技术文档
网络·网络协议·tcp/ip