已合并
Fix farfield #12589
已合并
WMXZ_J创建于 12 天前
WMXZ_J
12 天前

相关的Issue

https://gitcode.com/openharmony/communication_dsoftbus/issues/9843

原因(目的、解决的问题等)

Fix farfield

描述(做了什么,变更了什么)

Fix farfield

验证结果(新增、改动、可能影响的功能)

日志规范自检:

安全编码自检:

是否已执行L0用例

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 WMXZ_J 的贡献)
WWMXZ_J
12 天前 关联了issue:fix farfield
openharmony_ciopenharmony_ci成员
12 天前 添加了label:waiting_on_author
openharmony_ci
openharmony_ci成员
12 天前 评论:

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

likedislike
openharmony_ciopenharmony_ci成员
12 天前 添加了label:dco检查成功
frank_libing12 天前进行代码检视1
core/connection/proxy/br/br_proxy_manager.c
@@ -259,0 +247,4 @@
247+ CONN_CHECK_AND_RETURN_RET_LOGE(ret == SOFTBUS_OK, ret, CONN_PROXY,
248+ "createProxyConnectInfo fail, ret=%{public}d", ret);
249+ 
250+ struct ProxyConnection *existing = GetBrProxyChannelManager()->getProxyChannelByAddr(connectInfo->brMac);
frank_libing12 天前评论:

🟡 [中] 函数返回值直接使用 -> 解引用

分析: 函数返回的指针可能为空,直接使用 -> 解引用存在空指针解引用风险。建议先判空。

涉及代码:

struct ProxyConnection *existing = GetBrProxyChannelManager()->getProxyChannelByAddr(connectInfo->brMac);

(core/connection/proxy/br/br_proxy_manager.c:250)


此评论由 PR 自动审查脚本 (2026-10-09 12:35) 生成

likedislike
此处折叠了173条消息 查看更多
openharmony_ci
openharmony_ci成员
10 天前 评论:

您好,Committer @byx0010 @chen-botong @jzy_123 @zhang-haowen123 @michael4096 @jeosif @liangliang_ma @firstsummer @songzhengteng @tanyanying @wangJE ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @byx0010 @chen-botong @jzy_123 @zhang-haowen123 @michael4096 @jeosif @liangliang_ma @firstsummer @songzhengteng @tanyanying @wangJE . 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.

likedislike
openharmony_ciopenharmony_ci成员
10 天前 关闭了关联的issue
openharmony_ciopenharmony_ci成员
10 天前 合入了pull request,合并节点 SHA:652f88e0261ce4d0610f73cc451c26b84d2597af
openharmony_ciopenharmony_ci成员
10 天前 删除了label:waiting_for_review
openharmony_ciopenharmony_ci成员
10 天前 添加了label:merged