提供跨应用通信功能-startup

已开启

提供跨应用通信功能 #2950

斌创建于 6 天前从Love_BIN/startup_appspawn： master 合入到 openharmony/startup_appspawn： master

提供跨应用通信功能 #2950

已开启

斌创建于 6 天前

从Love_BIN/startup_appspawn master合入到openharmony/startup_appspawn master

斌

6 天前

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

提供跨应用通信功能

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

提供跨应用通信功能

测试用例（新增、改动、可能影响的功能）

合并受阻

斌

6 天前关联了issue：提供跨应用通信功能

斌

6 天前评论：

start build

openharmony_ci成员

6 天前添加了label：waiting_on_author

openharmony_ci成员

6 天前评论：

感谢提交 Pull Requests ！此PR未通过DCO校验。
校验失败可能原因：

1. 未签署“DCO协议”（开发者原创声明协议），在线签署、查看签署状态。

2. Commits 中未包含 Signed-off-by信息，参考FAQ处理。

修复上述问题后，在PR的评论框输入“check dco” ，单击”评论”，系统将再次进行DCO校验。

当前检测到如下Commits的Signed-off-by邮箱未签署DCO协议：

42c7c44

Thanks for submitting a pull request. This pull request has not passed the DCO check.
Possible causes:

1. You have not signed the Developer Certificate of Origin (DCO). Sign the DCO and check DCO status.

2. The commits do not contain the Signed-off-by information. To resolve this issue, see FAQs.

After resolving the preceding issues, enter check dco in the comment box of this pull request and click Comment. The system will check DCO status again.

The Signed-off-by emails in the following commits have not signed the DCO:

42c7c44

openharmony_ci成员

6 天前添加了label：dco检查失败

openharmony_ci成员

6 天前评论：

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

openharmony_ci成员

6 天前添加了label：静态检查失败

openharmony_ci成员

6 天前评论：

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

静态检查:

#	check type	result	report
1	codeCheck	noPass	>>>

编译测试:

#	Device	build result	package
1	hispark_taurus_LiteOS	pending	NA
2	hispark_taurus_Linux	pending	NA
3	ohos-sdk	pending	NA
4	dayu200	pending	NA
5	dayu200_tdd	pending	NA
6	master_inner_build	pending	NA
7	ohos-host_mini_tdd	pending	NA
8	dayu600_7885	pending	NA
9	x86_64_virt	pending	NA

斌

3 天前强制推送 1 个提交:3dd7d4ac-提供跨应用通信功能

openharmony_ci成员

3 天前删除了label：静态检查失败

openharmony_ci成员

3 天前评论：

代码有更新,重置PR验证状态

openharmony_ci成员

3 天前评论：

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

3 天前删除了label：dco检查失败

openharmony_ci成员

3 天前添加了label：dco检查成功

斌

3 天前评论：

static-check

openharmony_ci成员

3 天前评论：

静态检查:

#	check type	result	report
1	codeCheck	noPass	>>>

斌

2 天前强制推送 1 个提交:78666624-提供跨应用通信功能

openharmony_ci成员

2 天前评论：

斌

2 天前强制推送 1 个提交:24878c71-提供跨应用通信功能

斌

2 天前强制推送 1 个提交:dfcebb8b-提供跨应用通信功能

openharmony_ci成员

2 天前评论：

openharmony_ci成员

2 天前评论：

斌

2 天前强制推送 1 个提交:32c36248-提供跨应用通信功能

openharmony_ci成员

2 天前评论：

斌

2 天前强制推送 1 个提交:8472017f-提供跨应用通信功能

openharmony_ci成员

2 天前评论：

斌

2 天前强制推送 1 个提交:306752c1-提供跨应用通信功能

openharmony_ci成员

2 天前评论：

斌

2 天前评论：

static-check

openharmony_ci成员

2 天前评论：

静态检查:

#	check type	result	report
1	codeCheck	noPass	>>>

soulrequiem1 天前进行代码检视1

modules/sandbox/normal/sandbox_core.cpp

839+ SandboxCommonDef::g_ipcGroupSandboxPathPrefix + groupId;

840+ SharedMountArgs arg = {

841+ .srcPath = srcPath.c_str(),

842+ .destPath = sandboxPath.c_str()

soulrequiem1 天前评论：

high modules/sandbox/normal/sandbox_core.cpp:842

问题：MountIPCGroup 仅拒绝 groupGid == 0，任意非 0 gid 都会创建 root: 01771 目录并挂载给应用。消息流为应用 module.json5 声明 → BMS/AMS 组装 → appspawn TLV，应用侧声明数据可直达 appspawn。若上游缺少 gid 白名单校验（本仓不可见），恶意应用可声明敏感系统 gid（如 3000 shell 组），其进程经 setgroups 加入后获得该组在全系统 group-rw 目录的访问能力，不限于 IPC 共享内存，构成提权面。

证据：groupGid == 0 是唯一校验；对照组 MountAllGroup 的 DataGroup 源路径为系统生成的 dir+uuid，不含应用可选 gid。

建议：appspawn 作为最后执行者应做防御性校验：限定 groupGid 落在专用分配段（类似 isolated uid 段做法），或 neverallow 已知系统保留 gid。同时请 owner 确认 AMS/BMS 侧的 gid 分配机制是否存在白名单。

soulrequiem1 天前进行代码检视1

modules/sandbox/normal/sandbox_core.cpp

504+ APPSPAWN_LOGE("chmod or chown failed for %{public}s, errno %{public}d", path.c_str(), errno);

505+ if (stat(path.c_str(), &statBuff) < 0) {

506+ APPSPAWN_LOGE("stat %{public}s failed", path.c_str());

507+ } else if (statBuff.st_uid != uid || statBuff.st_gid != gid ||

soulrequiem1 天前评论：

medium modules/sandbox/normal/sandbox_core.cpp:507

问题：EnsureDirWithMode 中 chmod/chown 失败仅打日志后 return 0。IPCGroup 目录权限为 01771（sticky+setgid），属主/属组错误意味着共享内存的组内互访边界被破坏，但挂载照常完成，应用无感知。

证据：失败分支只 LOGE 后落到 return 0，调用方 EnsureDirWithMode(...) != 0 的检查在该路径永远不触发。

建议：IPCGroup 场景下 chmod/chown 失败应返回错误并跳过该项挂载，或至少提供可观测的上报手段。

soulrequiem1 天前进行代码检视1

modules/sandbox/normal/sandbox_core.cpp

834+ std::string srcPath = SandboxCommonDef::g_ipcGroupSrcPathPrefix + userId + "/group/" + groupId;

835+ if (EnsureDirWithMode(srcPath, 0, groupGid, SandboxCommonDef::IPC_GROUP_SRC_PATH_MODE) != 0) {

836+ continue;

837+ }

soulrequiem1 天前评论：

medium modules/sandbox/normal/sandbox_core.cpp:837

问题：atoi("3000000000") 溢出为截断值，atoi("-1") 可通过 != 0 检查后 cast 为 (gid_t)0xFFFFFFFF，两者都会创建错误属主的共享内存目录，与 F1 叠加会扩大攻击面。

证据：static_cast<gid_t>(atoi(groupGidItem->valuestring)) 无 errno 与范围检查。

建议：改用 strtol 并做 errno 与范围校验（0 < gid，字符串完全消费）。

soulrequiem1 天前进行代码检视1

modules/sandbox/normal/sandbox_core.cpp

845+ APPSPAWN_CHECK_ONLY_LOG(mountRet == 0, "mount ipcGroup %{public}s failed, ret %{public}d",

846+ srcPath.c_str(), mountRet);

847+ }

848+ cJSON_Delete(ipcGroupRoot);

soulrequiem1 天前评论：

low modules/sandbox/normal/sandbox_core.cpp:848

问题：EnsureDirWithMode 失败（mkdir 失败路径）时 continue，该项 IPC 组不挂载但整体 ret=0，应用启动成功，直到运行时通信失败才暴露。与数据非法（ret=-1 启动失败）的容错策略不一致。

证据：if (EnsureDirWithMode(...) != 0) { continue; } 不置 ret。

建议：建议区分环境错误与数据错误的日志级别，或确认该容错策略是有意设计并在注释中说明。

soulrequiem1 天前进行代码检视1

test/unittest/app_spawn_standard_test/app_spawn_sandbox_normal_test/app_spawn_sandbox_core_test.cpp

1818#include <memory>

1919#include <string>

2020#include <vector>

21+#include <unistd.h>

2122

2223#include "appspawn_server.h"

2324#include "appspawn_service.h"

541542 DeleteAppSpawningCtx(appProperty);

542543}

543544

545+/**

soulrequiem1 天前评论：

low test/unittest/app_spawn_standard_test/app_spawn_sandbox_normal_test/app_spawn_sandbox_core_test.cpp:545

问题：测试 10/11/15-18 真实创建 /mnt/sandbox/shm/100/group/* 且可能真实 bind mount（设备 root 下 DoAppSandboxMountOnce 非 stub 路径会成功），但仅 rmdir 一层且无 umount，重复执行残留挂载点与父目录。另外测试用硬编码 "IPCGroup" 字符串而非 MSG_EXT_NAME_IPC_GROUP 宏。

证据：MountIPCGroup_10/11 注释自述 mount will likely fail in test env，但设备 root 环境下路径可达时实际会成功。

建议：teardown 中 umount 沙箱内挂载点并递归清理父目录；字符串统一使用宏。

相关的Issue

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

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

测试用例（新增、改动、可能影响的功能）

合并受阻