已开启
增加off日志打印 #20312
增加off日志打印 #20312
已开启
acdemicJava创建于 1 天前
acdemicJava
acdemicJava
1 天前

IssueNo:
https://gitcode.com/openharmony/ability_ability_runtime/issues/16077

Description:

稳定性自检:

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

安全编码自检:

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

TDD Result:

XTS Result:

是否已执行L0用例

AI检视评分(使用本地代码检视skills扫描):

likedislike
合并受阻
acdemicJavaacdemicJava
1 天前 关联了issue:[Bug]: 增加off日志打印
afwk_helper成员
1 天前 评论:

开始进行AI检视!

AI review has been started, please wait...

likedislike
afwk_helper成员
1 天前 评论:
check type result report
start ai_review pass -
likedislike
acdemicJava
acdemicJava
1 天前 评论:

start build

likedislike
afwk_helper成员
1 天前 评论:

门禁编译Tips:
1.触发门禁编译前先评论static-check进行静态检查
2.本地编译验证充分后再触发门禁
3.门禁编译时不要推送代码否则会终止上一次门禁编译。
失败超过3次将可能影响代码合入

likedislike
acdemicJava
acdemicJava
1 天前 评论:

pass

likedislike
afwk_helper成员
1 天前 评论:
check type result report
TDD check fail -

TDD检查失败。
TDD check failed.

请自行检视是否需要补充用例或联系以下人员检视后评论"force tdd"强制通过。若最新提交中已携带TDD,请评论"check tdd"重新触发检查。
Please:
1. Commit with TDD implementation, then leave a "check tdd" comment to re-trigger the TDD check process. or
2. Contact the following members to leave a "force tdd" comment.

duansizhao: @dsz2025
xialiangwei: @xialiangwei
wangkailong: @wkljy
liqiang: @jayleehw
管控详情:https://gitcode.com/dsz2025/AfwkHelper/blob/main/PR管控/PR管控说明.md

likedislike
openharmony_ciopenharmony_ci成员
1 天前 添加了label:waiting_on_author
afwk_helper成员
1 天前 评论:
AI代码检视(AI Code Check)
报告标题 报告来源 提交时间 评分 ID匹配 检查结果
- - - - - fail

AI代码检视失败。
AI Code Check failed.

未找到AI代码检视报告。请使用本地代码检视工具(skills/codecheck)对代码进行检视,生成检视报告后将其粘贴到PR评论或PR提交描述中。
No AI code review report found. Please run local code check skills (skills/codecheck), and paste the report into PR comments or PR description.

若已处理完毕,请评论 "check ai-code-check" 重新触发检查。
或联系以下人员评论 "force ai-code-check" 强制通过:
duansizhao: @dsz2025
xialiangwei: @xialiangwei
wangkailong: @wkljy
liqiang: @jayleehw
管控详情:https://gitcode.com/dsz2025/AfwkHelper/blob/main/PR管控/PR管控说明.md

likedislike
afwk_helper成员
1 天前 评论:

自定义PR管控检查报告 / Custom PR Control Check Report

检查项 检查结果 处理建议
TDD检查 ✗ 失败 请自行检视是否需要补充用例或联系以下人员检视后评论"force tdd"强制通过。若最新提交中已携带TDD,请评论"check tdd"重新触发检查。
检视意见 ✓ 通过 -
AI代码检视 ✗ 失败 未找到AI代码检视报告。请使用本地代码检视工具(skills/codecheck)对代码进行检视,生成检视报告后将其粘贴到PR评论或PR提交描述中。
likedislike
openharmony_ci
openharmony_ci成员
1 天前 评论:

感谢提交 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成员
1 天前 添加了label:dco检查成功
openharmony_ci
openharmony_ci成员
1 天前 评论:

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

likedislike
afwk_helper成员
1 天前 评论:

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

总体评估: NEEDS_ATTENTION

问题统计:

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

摘要:
该PR仅增加了日志打印逻辑,整体风险较低,但日志内容存在复制粘贴错误,在移除特定回调的分支中错误地记录为off_all,可能影响日志排查的准确性。

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


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

likedislike
openharmony_ciopenharmony_ci成员
1 天前 添加了label:编译成功
openharmony_ciopenharmony_ci成员
1 天前 添加了label:静态检查成功
openharmony_ciopenharmony_ci成员
1 天前 添加了label:冒烟测试成功
openharmony_ciopenharmony_ci成员
1 天前 通过测试
openharmony_ci
openharmony_ci成员
1 天前 评论:

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

静态检查:

# check type result report
1 codeCheck pass >>>

编译测试:
# Device build result test result package
1 ohos-sdk success NA >>>
2 dayu200 success success >>>
3 dayu200_tdd success NA >>>
4 master_inner_build failed(IGNORE)(联合门禁失败) NA >>>
5 mac-sdk success NA >>>
6 ohos-host_mini_tdd success NA >>>
7 dayu600_7885 success success >>>
8 x86_64_virt success NA >>>

likedislike
openharmony_ciopenharmony_ci成员
1 天前 删除了label:waiting_on_author
openharmony_ciopenharmony_ci成员
1 天前 添加了label:waiting_for_review
openharmony_ci
openharmony_ci成员
1 天前 评论:

您好,Committer @jayleehw @zhangyafei-echo @littlejerry1 @jsjzju @wkljy @yzkp @ccllee1 @yangxuguang-huawei @li-weifeng2024 @wendel ,请分配检视人员检视该PR,可以通过命令"assign [@someone_id]"分配检视人员,也可以直接评论"assign"分配给自己进行检视。


Hello, Committer @jayleehw @zhangyafei-echo @littlejerry1 @jsjzju @wkljy @yzkp @ccllee1 @yangxuguang-huawei @li-weifeng2024 @wendel . 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
littlejerry1
littlejerry1成员1 天前进行代码检视1
frameworks/js/napi/app/error_manager/js_error_manager.cpp
@@ -1511,6 +1517,8 @@ private:
15111517 auto res = CreateJsUndefined(env);
15121518 if (function == nullptr) {
15131519 ClearGlobalPromiseReference(env);
1520+ TAG_LOGI(AAFwkTag::JSNAPI, "SubEvent op = off_all, kit = AbilityKit, "
1521+ "event=globalUnhandledRejectionDetected");
15141522 return res;
15151523 }
15161524 if (!ValidateFunction(env, function)) {
@@ -1528,6 +1536,8 @@ private:
15281536 if (equals) {
15291537 NAPI_CALL(env, napi_delete_reference(env, iter.ref));
15301538 globalPromiseList.erase(iter);
1539+ TAG_LOGI(AAFwkTag::JSNAPI, "SubEvent op = off_all, kit = AbilityKit, "
1540+ "event=globalUnhandledRejectionDetected");
15311541 return res;
15321542 }
15331543 }
@@ -1581,6 +1591,7 @@ private:
15811591 napi_value result = nullptr;
15821592 NapiAsyncTask::Schedule("JSErrorManager::OnUnregisterErrorObserver",
15831593 env, CreateAsyncTaskWithLastParam(env, lastParam, nullptr, std::move(complete), &result));
1594+ TAG_LOGI(AAFwkTag::JSNAPI, "SubEvent op = off_all, kit = AbilityKit, event=error");
15841595 return result;
15851596 }
15861597 
@@ -1598,6 +1609,8 @@ private:
15981609 {
15991610 auto res = CreateJsUndefined(env);
16001611 if (argc == ARGC_ONE) {
1612+ TAG_LOGI(AAFwkTag::JSNAPI, "SubEvent op = off_all, kit = AbilityKit, "
littlejerry1
littlejerry11 天前评论:

仅增加日志

likedislike
littlejerry1littlejerry1成员
11 小时前 通过审查