已合并
refactor: SetKaraokeParameters 参数改为枚举全链路穿透 #16722
refactor: SetKaraokeParameters 参数改为枚举全链路穿透 #16722
已合并
acorwa创建于 6 天前
acorwa
acorwa
6 天前

IssueNo:
https://gitcode.com/openharmony/multimedia_audio_framework/issues/13950

Description:

静态扫描截图:

通用规范自检:

自检项 自检结果
成员变量命名,统一采用xxxx_ NA
换行优先使用4格缩进 Y
*和&运算符统一跟随之后的变量 Y
避免使用未明确位宽的整型,选择使用int32_t, uint32_t等类型 Y
时间常量命名加后缀_MS, _US等,按照用途命名而非值 NA

安全编码自检:

自检项 自检结果
外部传入的指针使用前是否判空 NA
外部传入数据,直接用于内存申请、数组长度、索引、循环条件前,必须进行合法性校验 Y
成员或局部变量,尤其是指针、资源描述符,bool变量必须赋初值 Y
内存分配后必须判断是否成功 NA
禁止打印内存地址、文件路径、设备名、MAC地址等敏感信息,如有需要,使用private修饰 Y
有符号、无符号整数之间运算、比较,确保不会出现溢出、反转、除0 NA
有无符号不允许混合计算,必须提前进行强转 Y
map\vector\list\set等对象存在多线程并发访问场景必须加锁 Y

进程安全自检:

自检项 自检结果
SA新增idl接口,需要根据功能控制可访问范围,通过权限、UID校验等方式校验 NA
SA新增idl接口,必须同时新增fuzz用例,并进行fuzz测试 Y
类反序列化结果用于内存申请、数组长度、循环条件前,必须进行合法性校验 Y
新增动态共享库或可执行文件是否开启硬件PAC(gn文件配置 branch_protector_ret = "pac_ret") NA
新增动态共享库或可执行文件是否开启前向CFI(gn文件配置 cfi = true, cfi_cross_dso = true, debug = false) NA

XTS Result:

TDD Result:

Fuzz Result:

likedislike
Pull Request已成功合入, 合并人@openharmony_ci
(感谢 acorwa 的贡献)
acorwaacorwa
6 天前 关联了issue:[Bug]: refactor: SetKaraokeParameters 参数改为枚举全链路穿透
萤火虫
萤火虫
6 天前 评论:

扫描任务已触发,请前往CMC静态分析增强系统查看结果。如超15分钟无结果,请联系张俊文

likedislike
openharmony_ciopenharmony_ci成员
6 天前 添加了label:waiting_on_author
openharmony_ci
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.

likedislike
openharmony_ciopenharmony_ci成员
6 天前 添加了label:dco检查成功
此处折叠了71条消息 查看更多
openharmony_ciopenharmony_ci成员
3 天前 添加了label:merged
openharmony_ciopenharmony_ci成员
2 天前 删除了label:编译成功
openharmony_ciopenharmony_ci成员
2 天前 删除了label:静态检查成功
openharmony_ciopenharmony_ci成员
2 天前 删除了label:冒烟测试成功
openharmony_ci
openharmony_ci成员
2 天前 评论:

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

likedislike