fix(m52): r1 评审 P2-1 class 缺失 WARN 去刷屏 + 两个 nit(snprintf clamp / 恒真断言)
P2-1(必修):`multi_font: class %s font %s missing, fallback global` 与上一轮修掉的 enabled 行同在 meow_font_reload_multifont() 里、被 main_menu.c:1385 每 2s 调一次; 命中「某个 font.class_* 非空但文件缺失/不可读」时最多 6 行/2s(3 行/s),比 enabled 行凶 6 倍,会让「空转 60s 日志不增长」在 class 缺失形态下失效。 改法:循环内只记「本轮缺失类别位图」,循环外经新纯函数 mf_missing_warn_mask() 做跃迁 判定(缺失集合不变 → 静默;新类别缺失 → 1 条;恢复后再缺失 → 再 1 条)。回退行为不变 (缺失类别照旧回退全局),日志文本/参数不变。另加 _Static_assert 钉住类别数 ≤32(位图宽度)。 nit:wd_log() 的 snprintf 返回值显式 clamp(原「buf≤511+前缀≤23<544」是隐式不变量); test_wd_fw 的 "scan finds self" 原断言带 `|| np > 0` 恒真、等于没测 → 改为真查表断言 getpid() 在 wd_rg_pid[] 中且 comm 非空、并与 /proc/self/stat 的 pid/ppid 交叉核对。 验证:test_wd_fw 128 pass/0 fail(原 124,+4 来自强化后的自扫断言); test_mf_logsig 21 pass/0 fail(原 13,+8 为 P2-1 验收:单类缺失 60s@2s 只 1 条、 缺失集合变化再 1 条、恢复后再缺失再 1 条);launcher 宿主测试 ALL PASS; 三处交叉编译 0 warning(watchdog 100,116 B/25f76950、launcherd 902,044 B/75629078、 audiod 68,732 B/ebb9a7ac 未变=bind hunk 未动)。
This commit is contained in:
1 parent
166b32d0e5
commit
0d6c56100f
5 files changed
+93
-4
No files matched your search
@@ -181,6 +181,10 @@ static void wd_log(const char *fmt, ...)
|
||||
va_end(ap);
|
||||
char line[544];
|
||||
int l = snprintf(line, sizeof line, "%llu [watchdog] %s\n", now_ms(), buf);
|
||||
/* M52 r2(r1 评审 nit):显式 clamp,别把「buf ≤511 + 前缀 ≤23 < 544」当隐式不变量——
|
||||
* 日后调大 buf 或调小 line 就是栈越界读。snprintf 返回「本应写入的长度」,可能 ≥ sizeof line。 */
|
||||
if (l < 0) l = 0;
|
||||
else if (l >= (int)sizeof line) l = (int)sizeof line - 1;
|
||||
if (wd_log_fd >= 0) {
|
||||
if (write(wd_log_fd, line, l) < 0) wd_log_fd = -1;
|
||||
return;
|
||||
|
||||
Reference in new issue
Block a user