test(NET/lftpd): lftpd 控制通道模块(35 条),FTP 一套四块全部进分母
新增 test/host/lftpd/,lftpd.c 覆盖 304/473 = 64.3%。做法是把生产源整个 #include 进 TU,于是命令表 commands[]、17 个 static cmd_* 与 handle_control_channel 都能直接驱动: 假 inet 层给"脚本化命令行队列 + 响应记录",假 io 层给"极简内存视图 + 可注入失败", 认证与配置当可注入黑盒(认证本身有 test/host/ftp_auth 的 10 条专测,不重复验)。 钉住的契约(都是真机上贵、却难按需复现的那几类): - STOR 传到一半写失败必须 **unlink 半成品**:否则 Flash 上永久留个坏文件,现场表现 是"升级包传坏了再也传不上去"。真机要卡准"传一半掐线"的时机,这里注一个短写就行。 - 数据通道是单连接 socket:LIST/RETR/STOR 用完必须关监听并把 data_socket 复位成 -1; accept 失败要报 425 不留残口,会话退出(QUIT 或对端断线)时 cleanup 也要关掉还没 用的被动监听。漏一次就是下一条命令拿着上一个号继续 accept,偶发连不上。 - 命令名缓冲只有 command_tmp[4+1],前置判断 if (index >= 5) 就是防 memcpy 越界; 恰好 5 个字符必须被拒(改一行这条就响)。 - 认证前置白名单:未登录只放行 USER/PASS/QUIT/FEAT/SYST/NOOP,其余一律 530;口令错 必须清 auth_user_ok 强制重走 USER;被锁与口令错都只回 530,不给攻击者区分提示; 配置读不到时 USER 无法匹配、PASS 直接 530(fail-closed)。 - 写开关([ftp] write_en=0)必须在**碰文件之前**挡住 STOR/DELE,且不去 accept 数据 通道;DELE 不许删目录。 - 227/229 编码:IP 经 net_htonl 后按高字节到低字节排(就是点分序),端口高/低两字节; 拿不到本机 IP 时要把刚建的监听关掉。FEAT 多行:首行 211-、四条能力无状态码、末行 211 结束(共 6 条响应)。 - 220 欢迎没送达(对端已走)就直接收工,不空转读命令;该留痕按真实分级算 INFO 而不是 ERROR(真实 private/lftpd_log.h 把 lftpd_log_peer 归到 INFO:任何人连 21 端口就走 就能无限制造固件级 ERROR,还会撞掉 @test 的 Forbidden 标记 [ERR])。 - 传输循环每轮喂狗(lftpd_task_wd_beat 计数可断言)。 主机侧两个必要的兼容处理(都写在桩注释里,不是随手改): 1. lftpd.c 会 include ff.h,而这份 FatFs 在 Windows 支路上把 winsock.h 拉进来,它带 "#define s_addr S_un.S_addr" 的 BSD 兼容宏,会把真实 net_types.h 里的字段声明 uint32_t s_addr; 展开成 uint32_t S_un.S_addr; 直接炸(实测 C2143 在 "." 前缺 ";")。 桩里只 #undef 这个污染宏,让固件那份结构体定义原样参与编译,被测代码读的还是真字段。 2. 引号 include 先从 lftpd.c 自己目录找 private/lftpd_log.h,桩目录赢不过它——于是 顺着用真实那份(它只是把 lftpd_log_* 映射成 DBG_*),只桩最底下的 dbg_log.h。这样 真实日志分级也一起被编译验证,比我自己重写一套更对。 自查(35 条里前两轮红了 4 条,全是**我的期望/前置态写错**,实现均正确):未登录时 FOO 被白名单以 530 拦掉测不到 502 分派;227 的 IP 顺序我把 htonl 前后搞反;FEAT 是 6 条响应我按 5 条索引;PASV 之后走完整会话会被 cleanup 关掉监听(那正是 006 要验的 行为),观察 PASV 留下的状态必须直调 cmd_pasv。 覆盖率与台账:套件 26 → 27 全绿;files_tested 30 → 31;未触及 .c 36 → 35;分母 4,072 → 4,545 行。**总体 line_rate 88.4% → 85.9% 是摊薄不是回退**(64.3% 的文件进 了分母),按 §5 口径要盯 files_tested/files_untested——它从 19 一路涨到 31。 恒等式 35 = 1(P1 剩 iec104_serv.c) + 5(P2) + 29(P3)。编号补建 TC-HOST-LFTPD (游标 034),族数 20 → 21。 自查门禁:test-cppcheck Passed;check_ids --strict 通过;doc-links 35 处(= 存量); 9 个文件 BOM/CR/U+FFFD/行尾逐项净。零生产代码改动,未跑 @build。--all-files 仍只有 mem-numbers / mem-budget(.map 台账漂移,等 Lead 合并后统一回填)与 ui-golden (ref/*.bin 不入库)三条失败,与本分支改动无关。
This commit is contained in:
@@ -1834,6 +1834,28 @@
|
||||
"subitem_ok": true,
|
||||
"owner_note": "B 层口径(2026-10-21 批 3 定:这些号在代码目录外零消费者,属主登记处 = 单测代码本身,不逐号补文档);2026-10-23 随覆盖率口径 §9 P1 FTP 批次补登记",
|
||||
"cursor_note": "游标按实测最大号收录(2026-10-23):越界即视为未登记的冒号牌"
|
||||
},
|
||||
{
|
||||
"name": "TC-HOST-LFTPD",
|
||||
"class": "TC",
|
||||
"title": "主机侧单测用例号:FTP 控制通道协议层(test/host/lftpd)",
|
||||
"pattern": "\\bTC-LFTPD-[0-9]{1,4}[a-z]?\\b",
|
||||
"digit_width": [
|
||||
2,
|
||||
4
|
||||
],
|
||||
"owner_docs_glob": [
|
||||
"test/host/lftpd/*"
|
||||
],
|
||||
"scope": "global",
|
||||
"frozen": false,
|
||||
"cursor": "TC-LFTPD-034",
|
||||
"cursor_owner_only": true,
|
||||
"check_dangling": true,
|
||||
"allow_orphans": true,
|
||||
"subitem_ok": true,
|
||||
"owner_note": "B 层口径(2026-10-21 批 3 定:这些号在代码目录外零消费者,属主登记处 = 单测代码本身,不逐号补文档);2026-10-23 随覆盖率口径 §9 P1 FTP 批次补登记",
|
||||
"cursor_note": "游标按实测最大号收录(2026-10-23):越界即视为未登记的冒号牌"
|
||||
}
|
||||
],
|
||||
"unmanaged_bare": {
|
||||
|
||||
@@ -160,6 +160,14 @@
|
||||
"lines_covered": 123,
|
||||
"lines_valid": 253
|
||||
},
|
||||
"Drivers/BSP/NET/lftpd/lftpd.c": {
|
||||
"func_rate": 0.8077,
|
||||
"funcs_covered": 21,
|
||||
"funcs_valid": 26,
|
||||
"line_rate": 0.6427,
|
||||
"lines_covered": 304,
|
||||
"lines_valid": 473
|
||||
},
|
||||
"Drivers/BSP/NET/lftpd/lftpd_auth.c": {
|
||||
"func_rate": 1.0,
|
||||
"funcs_covered": 2,
|
||||
@@ -302,6 +310,14 @@
|
||||
"lines_total": 1193,
|
||||
"ok": true
|
||||
},
|
||||
"lftpd": {
|
||||
"files": {
|
||||
"Drivers/BSP/NET/lftpd/lftpd.c": 473,
|
||||
"Drivers/BSP/NET/lftpd/lftpd_string.c": 7
|
||||
},
|
||||
"lines_total": 480,
|
||||
"ok": true
|
||||
},
|
||||
"lftpd_inet": {
|
||||
"files": {
|
||||
"Drivers/BSP/NET/lftpd/lftpd_inet.c": 127
|
||||
@@ -467,6 +483,7 @@
|
||||
"App/util/task_prof.c",
|
||||
"App/util/task_watchdog.c",
|
||||
"Drivers/BSP/GD5F2GQ5UE/nand_ftl.c",
|
||||
"Drivers/BSP/NET/lftpd/lftpd.c",
|
||||
"Drivers/BSP/NET/lftpd/lftpd_auth.c",
|
||||
"Drivers/BSP/NET/lftpd/lftpd_inet.c",
|
||||
"Drivers/BSP/NET/lftpd/lftpd_io.c",
|
||||
@@ -480,16 +497,16 @@
|
||||
],
|
||||
"totals": {
|
||||
"files_generated_excluded": 5,
|
||||
"files_tested": 30,
|
||||
"files_untested": 114,
|
||||
"func_rate": 0.9097,
|
||||
"funcs_covered": 272,
|
||||
"funcs_valid": 299,
|
||||
"line_rate": 0.8838,
|
||||
"lines_covered": 3599,
|
||||
"lines_valid": 4072,
|
||||
"modules_ok": 26,
|
||||
"modules_total": 26
|
||||
"files_tested": 31,
|
||||
"files_untested": 113,
|
||||
"func_rate": 0.9015,
|
||||
"funcs_covered": 293,
|
||||
"funcs_valid": 325,
|
||||
"line_rate": 0.8587,
|
||||
"lines_covered": 3903,
|
||||
"lines_valid": 4545,
|
||||
"modules_ok": 27,
|
||||
"modules_total": 27
|
||||
},
|
||||
"untested_sources": [
|
||||
{
|
||||
@@ -500,10 +517,6 @@
|
||||
"code_lines": 801,
|
||||
"path": "Drivers/BSP/CH395F/ch395f.c"
|
||||
},
|
||||
{
|
||||
"code_lines": 694,
|
||||
"path": "Drivers/BSP/NET/lftpd/lftpd.c"
|
||||
},
|
||||
{
|
||||
"code_lines": 689,
|
||||
"path": "Drivers/BSP/GD5F2GQ5UE/gd5f2gq5ue.c"
|
||||
|
||||
Reference in New Issue
Block a user