感谢提交 Pull Requests！如果您提交的PR已经开发完毕，请评论 "start build" 触发门禁，更多交互操作，请访问OpenHarmony社区支持命令清单。如果需要调整订阅PR、Issue的变更状态，请访问订阅链接。


Thanks for submitting the pull request. If your Pull Request has already been developed, you can leave a "start build" comment to trigger the gated system. For more commands, please visit OpenHarmony Command List. If you need to change the subscription of a Pull Request or Issue, please visit the link.

start build

首次触发
门禁构建开始,包含静态检查、代码编译和测试【hispark_pegasus_3863编译, hispark_taurus_Linux编译, x86_64_virt编译, hispark_pegasus_3863测试, dayu600_7885测试, dayu200编译, ohos-host_mini_tdd编译, dayu200测试, hispark_pegasus编译, hispark_pegasus测试, dayu600_7885编译, dayu200_tdd编译, master_inner_build编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a8664ea64650f998bc9490c/runlist

PR 审查报告：#5561 - 告警修复

仓库：OpenHarmony/communication_wifi
PR 地址：https://gitcode.com/openharmony/communication_wifi/merge_requests/5561
作者：王旭 (tradeWindToNorth)
目标分支：master
PR SHA：780ae598d2f550f7805cb2db27ff3ed5db98691c
状态：open
标签：waiting_on_author、dco检查成功

CI 状态：

• ✅ DCO 检查成功
• ✅ 冲突检查通过

变更概要

本 PR 修复静态检查告警（1 个文件，+1/-1）：GetConnectedStationInfo 循环增加 staNumber 上限约束。

bool DhcpdInterface::GetConnectedStationInfo(const std::string &ifaceName, std::...)
{
    ...
    GetConnectedStaInfo(ifaceName, staNumber, staInfos, &staSize);
-    for (int i = 0; i < staSize; i++) {
+    for (int i = 0; i < staSize && i < staNumber; i++) {
        StationInfo info;
        info.deviceName = staInfos[i].deviceName;
        info.bssid = staInfos[i].macAddr;

问题详情

💡建议

1. 修复方向正确，需确认 staNumber/staSize 语义

位置：wifi/services/wifi_standard/wifi_framework/wifi_toolkit/net_helper/dhcpd_interface.cpp（GetConnectedStationInfo）

描述：GetConnectedStaInfo(ifaceName, staNumber, staInfos, &staSize) 中，staNumber 通常为数组容量（输入），staSize 为实际填充数量（输出）。正常情况下 staSize <= staNumber 恒成立；静态检查无法证明该不变量，故告警 staInfos[i] 可能越界（若 staSize > staNumber 将发生缓冲区越界读）。新增 i < staNumber 约束后：

• staSize <= staNumber 正常时：行为不变；
• staSize > staNumber 异常时：只遍历前 staNumber 项，避免越界读。

影响：修复消除了潜在越界读；正常路径无行为变化。防御性截断（丢弃多余项）在异常场景下可接受。

修复建议：建议在异常场景（staSize > staNumber）补一条告警日志，便于现场定位：

GetConnectedStaInfo(ifaceName, staNumber, staInfos, &staSize);
if (staSize > staNumber) {
    WIFI_LOGE("staSize %{public}d exceed staNumber %{public}d", staSize, staNumber);
}
for (int i = 0; i < staSize && i < staNumber; i++) {
    ...
}

2. 可读性优化（可选）

i < staSize && i < staNumber 可等价改写为 i < std::min(staSize, staNumber)，语义更直观（需注意 staSize 类型，若为指针输出需先取值）。可接受现状。

3. 无测试

建议补充 staSize > staNumber 场景的单元测试（mock GetConnectedStaInfo 返回值），验证不越界且结果正确。

总体评分

审查结论

建议操作：✅ 可以合并

说明：

• 问题 1 为本 PR 核心修复点，方向正确、防御有效；
• 建议补充异常场景日志与测试作为后续增强，不阻塞合并。

优点：

• 精准消除潜在缓冲区越界读（staInfos[i] 受 staNumber 容量约束）
• 变更最小（1 文件，+1/-1）
• DCO 检查通过

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a8664ea64650f998bc9490c/runlist

静态检查:

您好，Committer @wshixjr @bjtu_wujun_transportation @f00651013 @chengguohong ，请分配检视人员检视该PR，可以通过命令"assign [@someone_id]"分配检视人员，也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @wshixjr @bjtu_wujun_transportation @f00651013 @chengguohong . Please assign someone to review the PR. You can assign a reviewer by using the command "assign [@someone_id]", or you can comment "assign" to review the PR by yourself.

start build

首次触发
门禁构建开始,包含静态检查、代码编译和测试【hispark_pegasus_3863编译, master_inner_build编译, hispark_taurus_Linux编译, hispark_pegasus测试, dayu600_7885编译, dayu200测试, dayu600_7885测试, hispark_pegasus_3863测试, dayu200编译, ohos-host_mini_tdd编译, dayu200_tdd编译, x86_64_virt编译, hispark_pegasus编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a869d2564650f998bdc3b92/runlist

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a869d2564650f998bdc3b92/runlist

静态检查: