感谢提交 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.

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

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

CI 状态：

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

变更概要

本 PR 修复静态检查告警（1 个文件，+1/-1）：StrSafeCopy 增加 len == 0 边界防护。

void StrSafeCopy(char *dst, unsigned len, const char *src)
{
-    if (dst == NULL) {
+    if (dst == NULL || len == 0) {
        return;
    }
    if (src == NULL) {
        ...

问题详情

💡建议

1. 修复正确，但建议确认函数内部对 len - 1 的使用

位置：wifi/relation_services/common/wifi_hal_common_func.c（StrSafeCopy）

描述：新增 len == 0 防护是正确的必要修复：若函数内部存在 len - 1（如 strncpy(dst, src, len - 1) / dst[len - 1] = '\0'）类操作，len == 0 时 len - 1 按 unsigned 回绕为 0xFFFFFFFF，将导致缓冲区溢出。静态检查告警正是由此触发。

影响：修复后消除了 len == 0 时的回绕风险；无功能回归（len == 0 时本就无可用空间，提前返回语义正确）。

修复建议：确认函数体内所有 len 运算均受该守卫保护；建议将参数类型与运算显式化，进一步规避回绕类告警：

void StrSafeCopy(char *dst, unsigned int len, const char *src)
{
    if (dst == NULL || src == NULL || len == 0) {
        return;
    }
    /* 若存在 len - 1 类运算，可改为显式判断：
     * if (len <= 1) { dst[0] = '\0'; return; }
     */
    ...
}

2. src == NULL 分支行为需确认

描述：src == NULL 时函数行为（清空 dst 或直接返回）未在 diff 中可见。若为直接返回，dst 将保持未初始化状态，调用方可能继续使用脏缓冲区。

影响：潜在未初始化缓冲区使用（若调用方未自行清零）。

修复建议：确认该分支行为；若为直接返回，建议在返回前将 dst 置空：

if (src == NULL) {
    if (len > 0) {
        dst[0] = '\0';
    }
    return;
}

3. 无测试覆盖

该工具函数为核心复制接口（HAL 层通用），建议补充单测覆盖 dst == NULL / len == 0 / src == NULL / 截断场景。

4. 返回值语义

void 返回值使调用方无法感知复制是否成功/截断。可评估改为返回状态码（如 int）或保持现状（本次为最小告警修复，可不扩展）。

总体评分

审查结论

建议操作：✅ 可以合并

说明：

• 问题 1 为本 PR 核心修复点，方向正确、语义无回归；
• 问题 2/3 为可选的后续增强，不阻塞合并。

优点：

• 精准修复静态检查告警（len == 0 回绕风险）
• 变更最小（1 文件，+1/-1）
• DCO 检查通过

start build

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

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a84510d64650f998b2d3574/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.

验证结果已超过12小时，之前验证结果无效，自动重新触发构建，请关注最新验证结果

start build

本地或库上代码有更新,全量重新构建,重置所有关联PR的验证状态
门禁构建开始,包含静态检查、代码编译和测试【dayu200_tdd编译, hispark_taurus_Linux编译, dayu600_7885编译, ohos-host_mini_tdd编译, dayu200测试, dayu200编译, hispark_pegasus_3863测试, hispark_pegasus_3863编译, hispark_pegasus测试, master_inner_build编译, hispark_pegasus编译, dayu600_7885测试, x86_64_virt编译】,预计在60分钟内完成,门禁结果会同步发送到注册邮箱。您可以通过如下链接跟踪门禁进展:http://dcp.openharmony.cn/workbench/cicd/detail/6a91346064650f998bfd3450/runlist

代码门禁通过
您可以通过如下链接查看门禁报告:http://dcp.openharmony.cn/workbench/cicd/detail/6a91346064650f998bfd3450/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.