Message :重试获取查杀原因-ability_ability

已关闭

Message :重试获取查杀原因 #20322

zhengdongdong12创建于 6 天前关闭于 1 天前从zhengdongdong12/ability_ability_runtime_7358： master 合入到 openharmony/ability_ability_runtime： master

Message :重试获取查杀原因 #20322

已关闭

zhengdongdong12创建于 6 天前关闭于 1 天前

从zhengdongdong12/ability_ability_runtime_7358 master合入到openharmony/ability_ability_runtime master

zhengdongdong12

6 天前

IssueNo: https://gitcode.com/openharmony/hiviewdfx_hicollie/issues/518

Description: 重试获取查杀原因

稳定性自检：

自检项	自检结果
涉及跨进程调用的相关操作需要抛至主线程或加锁防止并发	OK
成员变量进行赋值或创建需要排查并发	OK
谨慎在lambda表达式中使用引用捕获	OK
谨慎在未经拷贝的情况下使用外部传入的string、C字符串	OK
map\vector\list\set等stl模板类使用时需要排查并发	OK
谨慎考虑加锁范围	OK
在IPC通信中谨慎使用同步通信方式	OK
禁止传递this指针至其他模块或线程（特别是eventhandler任务）	OK
禁止将外部传入的裸指针在内部直接构造智能指针	OK
禁止多个独立创建的智能指针管理同一地址	OK
禁止在析构函数中抛异步任务	OK
禁止js对象在非js线程（例如在IPC线程）创建、使用或销毁	OK
禁止在对外接口中未经判空直接使用外部传入的指针	OK
禁止接口返回局部变量引用	OK
禁止在信号函数中加锁	OK
禁止在关键流程（SA启动、应用启动等主流程）执行耗时的操作	OK
禁止将同一个cpp编译在不同的so中	OK

安全编码自检：

自检项	自检结果
裸指针避免通过隐式转换构造为sptr	OK
json对象在取值之前必须先判断类型，避免类型不匹配	OK
序列化时必须对传入的数组大小进行校验，避免出现超大数组	OK
避免使用未明确位宽的整型，选择使用int8_t、uint8_t等类型	OK
外部传入的路径要做规范化校验，对路径中的.、..、../等特殊字符严格校验	OK
指针变量、表示资源描述符的变量、bool变量必须赋初值	OK
readParcelable获取的对象使用前需要判空	OK
分配和释放内存的函数需要成对出现	OK
申请内存后异常退出前需要及时进行内存释放	OK
内存申请前必须对内存大小进行合法性校验	OK
内存分配后必须判断是否成功	OK
禁止使用realloc、alloca函数	OK
禁止打印文件路径、口令等敏感信息，如有需要，使用private修饰	OK
禁止打印内存地址	OK
整数之间运算时必须严格检查，确保不会出现溢出、反转、除0	OK
禁止对有符号整数进行位操作符运算	OK
禁止对指针进行逻辑或位运算	OK
循环次数如果收外部数据控制，需要检验其合法性	OK
禁止使用内存操作类危险函数，需要使用安全函数	OK
谨慎使用不可重入函数	OK
必须检查安全函数的返回值，并进行正确处理	OK
禁止仅通过TokenType类型判断绕过权限校验	OK

TDD Result: OK

XTS Result: OK

是否已执行L0用例

已验证
不涉及。如不涉及，请写明理由

AI检视评分（使用本地代码检视skills扫描）:

当前Pull Request已关闭, 关闭人@zhengdongdong12

afwk_helper成员

6 天前评论：

开始进行AI检视！

AI review has been started, please wait...

afwk_helper成员

6 天前评论：

check type	result	report
start ai_review	pass	-

openharmony_ci成员

6 天前添加了label：waiting_on_author

openharmony_ci成员

6 天前评论：

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

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

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

hwliujinwei3 天前进行代码检视1

services/appmgr/src/app_mgr_service_inner.cpp

6580+}

6581+

6582+void AppMgrServiceInner::ReportProcessKillEvent(int32_t pid, std::string versionCode, std::string versionName,

6583+ std::shared_ptr<AppRunningRecord> appRecord, const AppfreezeManager::ProcessKillInfo& killInfo)

hwliujinwei3 天前评论：

[重要] ReportProcessKillEvent 中保留了 bool foreground = ... 计算但不再使用(旧代码将其传给 GetProcessKillReason 的第4参数，现已移除)。同时新流程中 killInfo 的 foreground 字段不再被设置(零初始化为 false)，而后续 hisyseventReport->InsertParam(EVENT_KEY_FOREGROUND, killInfo.foreground) 仍依赖该字段。应在 GetProcessKillReason 中或调用方设置 killInfo.foreground。

hwliujinwei3 天前进行代码检视2

services/appdfr/src/appfreeze_manager.cpp

13821367 if (killId == INVALID_KILL_ID) {

13831368 killInfo.killReason = INVALID_KILL_REASON;

13841369 killInfo.killMsg = killMsg + " " + std::string(INVALID_KILL_REASON) + ":" + std::to_string(killId);

13851370 } else if (killId < 0) {

hwliujinwei3 天前评论：

[重要] GetProcessKillReason 中条件 if (res || killInfo.killReason == INVALID_KILL_REASON) 逻辑反转。当 GetExitKernelReason 成功(res==true)时返回 false 触发重试，但实际上成功获取到原因时应返回 true 直接上报。应改为 if (!res || killInfo.killReason == INVALID_KILL_REASON)。

zhengdongdong12

1 天前评论：

res是true是获取过程失败，这个情况不重试；res获取过程成功，但是没有获取到原因，有可能是set还未来的及。所以增加重试机制。

hwliujinwei3 天前进行代码检视1

services/appmgr/src/app_mgr_service_inner.cpp

6582+void AppMgrServiceInner::ReportProcessKillEvent(int32_t pid, std::string versionCode, std::string versionName,

6583+ std::shared_ptr<AppRunningRecord> appRecord, const AppfreezeManager::ProcessKillInfo& killInfo)

6584+{

65606585 bool foreground = appRecord->GetState() == ApplicationState::APP_STATE_FOREGROUND ||

hwliujinwei3 天前评论：

[次要] ReportProcessKillEvent 中保留了 AAFwk::EventInfo eventInfo; SetKilledEventInfo(appRecord, eventInfo); 但 SendAppEvent 已移至 SendProcessKillEvent 中调用，此处 eventInfo 创建后未使用，属于死代码，建议清理。

hwliujinwei3 天前进行代码检视2

services/appdfr/src/appfreeze_manager.cpp

1437+ return;

1438+ }

1439+ fdsan_exchange_owner_tag(sysloadFd, 0, FREEZE_DOMAIN);

1440+ KillInfo info = {0};

hwliujinwei3 天前评论：

[次要] SetExitKernelReason 中 int res = ioctl(...) 前有一行仅含空格的行，存在多余空白字符，建议清理。

zhengdongdong12

1 天前评论：

无需清理

zhengdongdong12

1 天前关闭了 pull request