wandering1
5224f21622
三处都由本轮 P1 新增的主机单测照出，用户逐条裁定后实施。共同性质：都不是改动引入
的回归，而是**从没被执行到的分支**——生产路径上要么没有调用者、要么没走边界，所以
编译器（0 错 0 警）、cppcheck、真机日志对它都无话可说。
1) 缺陷一（选 a：改名，行为零变化）
lftpd_string_trim 的尾部循环起点落在 strlen(p) 那一格 NUL，isspace(0) 为假 →
循环体一次都不执行，"尾部空白从未被裁"，而名字承诺两头都去。改名
lftpd_string_ltrim + 删掉那条死循环 + 在 .h/.c 写明"只去前导"。不去尾才是对的：
RFC 959 以命令终止符 CRLF 界定命令，参数（PASS 口令、含空格的文件名）的尾随
空格属于参数本体，去掉等于静默改数据；CRLF 已由 lftpd_inet_read_line 截断。
调用点 lftpd.c 同步改名。全库无遗留真引用（余 6 处旧名均为解释改名历史的散文）。
2) 缺陷二（按推荐修）
net_select 的 timeout = {0,0} 换算出 timeout_ms = 0，而退出条件写着
if ((timeout_ms > 0) && (已用时 >= timeout_ms)) → 超时永不生效，退化成死等；
POSIX 语义是"只查一轮立即返回"。去掉 timeout_ms > 0 这个错误前置即可。就绪
检查在本判断之前，故有就绪时仍立刻带计数返回；同文件 net_poll_events 的
timeout >= 0 本来就对，现在两条对齐。风险面：拿它当"非阻塞查一遍"会让 netTask
回不到主循环，而 netTask 在 IWDG required_mask 里（串行 socket 栈唯一执行者）。
3) 缺陷三（选 ①：按 POSIX 置位）
exceptfds 是恒为空的死参数：循环开头 p_sock == NULL → continue 先跳过，而
check_socket_error() 只有"表项为空 / fd 越界"才返回 1，越界的 i 又不在本循环
范围内 → 调用方按 POSIX 直觉用它收异常，永远什么都收不到，且返回码不报错。
把异常判定提到 continue 之前：表项不存在的 fd 计入 exceptfds 与返回计数，并更新
函数注释。选 ① 会改变返回计数语义，故补一条"未请求异常集时不计数"的用例，
确保只查可读的老调用方不被误伤（TC-NETSEL-012b）。
测试同步翻转：原 TC-NETSEL-008 / TC-NETSEL-012 是"现状桩、改了会响"，现在改成正向
契约，并新增 TC-NETSEL-012c（异常位只落到被请求的那个集合）；TC-LFTSTR-005 从
"尾部不裁"的现状登记改为正向契约（尾随空格必须保留）。net_select 22/22、
lftpd_io 33/33 全绿。
@build：0 错误 0 警告。实测体积 ER_IROM1 232,512 B（快照 232,536 → **−24 B**）、
Flash 233,704 B（−20 B）、RW_IRAM1 75,272 B（+8 B 是上一提交 net_cfg 那个字节，
本批零新增）、**RW_CCM 57,376 B 不变**；四档百分比 22.2/57.4/87.9/22.3 全不变。
Code 减少 36 B 即删掉的死循环。
覆盖率台账：门禁如实报了一次 lftpd_string.c 已覆盖行 8 → 7（**代码少了一行**，不是
覆盖退化），已 --update-baseline 重登；现 22/22 套件、line=3003/3446（87.1%）、
func=236/259（91.1%）、tested_files=26、untested_files=118。
文档：主机单测覆盖率口径 §9 新增"三处死路径的处置"小节（裁定、改法、影响、钉住
它的用例一一对应），并把两处"等裁定"的登记改为已处置。
自查：coding-standard / test-cppcheck / id-check --strict（无台账外新账）/ doc-links
（35 处 = 存量）全过；八个改动文件与 HEAD 逐项比对 BOM/CR/U+FFFD/行尾无变化
（这几个文件本就不带 BOM，未被动过）。pre-commit --all-files 仍只有三条失败且与
本分支无关：mem-numbers / mem-budget（.map 与 docs/内存管理 受管快照漂移，增量见
上，按并行分支约定不在分支里 --write，等 Lead 合并后统一回填）与 ui-golden
（simulator/tests/ref/*.bin 在 .gitignore 内，新建 worktree 天生没有黄金参考）。
19 lines
693 B
C
19 lines
693 B
C
#include <string.h>
|
||
#include <ctype.h>
|
||
|
||
#include "private/lftpd_string.h"
|
||
|
||
char *lftpd_string_ltrim(char *s)
|
||
{
|
||
char *p = s;
|
||
/* 只去行首空白：返回跳过后的指针，不搬移缓冲。
|
||
* 原先此处的"去尾部空白"循环，起点是 strlen(p) 那一格 NUL，isspace('\0') 为假
|
||
* → 从未执行过；而 FTP 参数的尾随空格属于参数本体（见本文件头与
|
||
* private/lftpd_string.h 的说明），2026-10-23 删掉这条死代码并改名为 *_ltrim，
|
||
* 让函数名与实际行为一致（**行为零变化**）。 */
|
||
for (int i = 0, len = (int)strlen(s); i < len && isspace((int)s[i]); i++) {
|
||
p++;
|
||
}
|
||
return p;
|
||
}
|