核心 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. 补充错误处理(增强鲁棒性)
  • 服务器路径拼接容错 :服务器handleReadRequest和handleWriteRequest中拼接文件路径时,处理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));

总结

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

相关推荐
智慧光迅AINOPOL3 分钟前
POF光电复合缆多业务承载原理
网络·安全·智能路由器
feng_you_ying_li20 分钟前
linux之网络层
linux·运维·网络
步行cgn27 分钟前
HTTP 请求报文结构详解:请求行、请求头、请求体各自的作用
网络·网络协议·http
未来之窗软件服务28 分钟前
Linux 运维日志删除的多种方式—东方仙盟
linux·运维·服务器·仙盟创梦ide·东方仙盟
小雉系统40 分钟前
BcoreOS实战:用一个升级包,让Rocky Linux跑进Debian服务器
linux·运维·服务器·debian
thethefighter41 分钟前
快速预览Word、WPS秒开,支持空格弹窗
linux·word·wps·快速预览·空格弹窗·nhdeepview
一只小鱼儿吖43 分钟前
内网穿透技术全景:原理、安全与实践
网络·安全
用户8351414049491 小时前
把日志和寄存器值喂给AI做故障定位,我的实际问法
linux·ubuntu
bj_bluewei_tech1 小时前
拯救者玩游戏掉帧卡顿,温度不高是什么原因?
网络·电脑·笔记本电脑·玩游戏
Ruiery2 小时前
Linux 6.6内核内存管理深度解析(十七):vmalloc — 内核要“虚拟连续“就够:物理页逐个凑,页表来连
linux·运维·服务器