fix : cpy adv devicename to advDevName-communication

已合并

fix : cpy adv devicename to advDevName #11310

wangJE创建于 2025年11月28日从wangJE/communication_dsoftbus_8074： master 合入到 openharmony/communication_dsoftbus： master

fix : cpy adv devicename to advDevName #11310

已合并

wangJE创建于 2025年11月28日

从wangJE/communication_dsoftbus_8074 master合入到openharmony/communication_dsoftbus master

wangJE成员

2025年11月28日

Pull Request已成功合入, 合并人@openharmony_ci

（感谢 wangJE 的贡献）

W wangJE成员

2025年11月28日关联了issue：coy adv broadcast device name to advDevName

openharmony_ci成员

2025年11月28日添加了label：waiting_on_author

openharmony_ci成员

2025年11月28日评论：

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

openharmony_ci成员

2025年11月28日添加了label：dco检查成功

W wangJE成员

2025年11月28日强制推送 1 个提交:ad7e8845-fix : cpy adv devicename to advDevName

此处折叠了44条消息查看更多

openharmony_dcp成员

2025年11月28日评论：

提交次数已达到或超过3次，为了保证上库效率和减少构建资源浪费，请点击链接观看学习视频，谢谢！https://dcp.openharmony.cn/workbench/video/videoDisplay

Marion成员

2025年11月29日评论：

⚠️ 🤖 AI 代码检视报告 ⚠️

总体评估: NEEDS_ATTENTION

问题统计:

总问题数: 3
严重问题: 0
高危问题: 2

摘要:
该变更添加了advDevName字段来存储广播设备名称，但存在未初始化缓冲区、潜在的整数溢出风险以及不一致的大小常量使用问题。

📊 详细报告
查看完整的审查详情，包括具体的问题描述、建议和代码位置：
🔗 查看详细报告

此评论由 OpenHarmony Insight 代码审查系统自动生成

Marion成员2025年11月29日进行代码检视1

adapter/common/net/bluetooth/broadcast/adapter/ble/src/softbus_ble_utils.c

485+ memcpy_s(dst->advDevName, sizeof(dst->advDevName), &advData[index + 1], len - 1) != EOK) {

486+ DISC_LOGE(DISC_BLE_ADAPTER, "copy adv device name failed");

487+ return SOFTBUS_MEM_ERR;

488+ }

Marion2025年11月29日评论：

🤖 AI 代码检视意见

类别: 🛡️ Reliability
严重级别: High
文件: adapter/common/net/bluetooth/broadcast/adapter/ble/src/softbus_ble_utils.c
行号: L484-L488

问题描述:
代码检查dst->advDevName[0] == '\0'但该缓冲区未被显式初始化。如果结构体在栈上分配，可能包含随机值，导致条件判断不可靠。在特定编译器优化或内存对齐情况下，可能误判或读取到脏数据。

修复建议:
确保SoftBusBcScanResult结构体在使用前被正确初始化。建议在结构体定义处添加构造函数或使用memset清零：

// 在分配SoftBusBcScanResult时
SoftBusBcScanResult scanResult = {0};  // 全部初始化为0

// 或者确保dst已被清零
memset(dst, 0, sizeof(SoftBusBcScanResult));

此评论由 OpenHarmony Insight Review Lite 自动生成

wangJE成员

2025年11月29日评论：

⚠️ 🤖 AI 代码检视报告 ⚠️

总体评估: NEEDS_ATTENTION

问题统计:

总问题数: 5
严重问题: 2
高危问题: 2

摘要:
代码变更在蓝牙广播数据结构中新增advDevName字段并实现了复制逻辑，但存在潜在缓冲区溢出风险和逻辑缺陷，需要修复以确保内存安全。

📊 详细报告
查看完整的审查详情，包括具体的问题描述、建议和代码位置：
🔗 查看详细报告

此评论由 OpenHarmony Insight 代码审查系统自动生成

Marion成员

1月24日评论：

⚠️ 🤖 AI 代码检视报告 ⚠️

总体评估: NEEDS_ATTENTION

问题统计:

总问题数: 4
严重问题: 0
高危问题: 1

摘要:
该PR添加了广播设备名称复制功能，但存在潜在的缓冲区越界风险和错误处理不一致问题。

📊 详细报告
查看完整的审查详情，包括具体的问题描述、建议和代码位置：
🔗 查看详细报告

此评论由 OpenHarmony Insight 代码审查系统自动生成

相关的Issue

原因（目的、解决的问题等）

描述（做了什么，变更了什么）

验证结果（新增、改动、可能影响的功能）

日志规范自检：

安全编码自检：

是否已执行L0用例

Pull Request已成功合入, 合并人@openharmony_ci