问题描述

在 gcc/common/config/aarch64/aarch64-common.cc 中，read_cpuinfo() 使用固定的 char cpuinfo[1024] 缓冲区读取 /proc/cpuinfo，但读取后没有根据 fread() 返回值补 NUL 终止，也没有处理字段超出前 1023 字节的情况，随后直接把该缓冲区传给 strstr() 解析 CPU implementer 和 CPU part。

这种实现存在两个明显风险：一是当读取内容刚好填满缓冲区或末尾没有 NUL 时，后续字符串搜索会越过有效数据范围继续扫描；二是当相关字段未出现在前 1023 字节内时，会被错误判定为“未找到”，导致平台识别不稳定。

根因分析

1. fread(cpuinfo, sizeof(char), sizeof(cpuinfo) - 1, fp); 的返回值没有保存，也没有根据实际读取长度追加 cpuinfo[nread] = '\0'。
2. 后续 strstr(cpuinfo, "CPU implementer") 和 strstr(cpuinfo, "CPU part") 依赖的是 C 风格 NUL 终止字符串，而当前缓冲区并没有被可靠终止。
3. 解析策略只看固定大小的前 1023 字节，既没有循环读取，也没有按行解析，因此当 /proc/cpuinfo 较长或字段位置靠后时，结果会受到文件布局影响。

修复方向

建议把这段解析改成更稳妥的逐行读取或显式长度受控解析逻辑：

• 保存 fread() 返回值并在有效位置补 NUL，至少先消除越界字符串扫描风险。
• 更推荐改成逐行读取 /proc/cpuinfo，按键值对查找 CPU implementer 与 CPU part，避免依赖固定大小缓冲区。
• 如果继续保留一次性读取方案，也应当在字段未完整读入时显式处理，而不是静默回退成 INVALID_IMP。