核心 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));

总结

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

相关推荐
汤愈韬3 小时前
ACL概述、ACL原理、基本ACL应用及配置
网络·网络协议·网络安全
微露清风3 小时前
系统性学习Linux-第二讲-基础开发工具
linux·运维·学习
不会代码的小猴4 小时前
Linux环境编程第六天笔记--system-V IPC
linux·笔记
阳光九叶草LXGZXJ4 小时前
达梦数据库-学习-48-DmDrs控制台命令(同步之Manager、CPT模块)
linux·运维·数据库·sql·学习
诸神缄默不语4 小时前
Linux命令行教程
linux
i建模6 小时前
如何在Arch Linux中重设忘记的root密码
linux·运维·服务器
码刘的极客手记7 小时前
VCAP4-DCA Beta 考试体验分享与 esxcli 自动化实战(第二、三部分)
网络·esxi·vmware·虚拟机
kida_yuan7 小时前
【Linux】运维实战笔记 — 我常用的方法与命令
linux·运维·笔记
@syh.8 小时前
【linux】进程控制
linux
朝风工作室8 小时前
开源一款轻量级跨网段 IPC 设备发现与网络配置工具
网络·开源