master

ascend-robot[feature]修复当前代码中的安全问题

36c2f38f创建于 17 小时前历史提交

文件	最后提交记录	最后更新时间
test_injection_event.cpp	[refactor]删除重复定义的结构体，从msopcom仓引入 Co-authored-by: yan_yw<yanyuwei2@huawei.com> # message auto-generated for no-merge-commit merge: !62 merge common into master [refactor]删除重复定义的结构体，从msopcom仓引入 Created-by: yan_yw Commit-by: yan_yw Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** 当前结构插桩等功能需保证msopprof仓和msopcom仓定义的结构体一致，难维护且不一致时功能出错，微重构该部分：删除msopprof仓重复定义的结构体，从common仓引入 - **修改方案：** 公共结构体和函数等从msopcom仓引入 - **修改内容：** 1、插桩公共结构体OperandRecord、OperandType等从dbi_defs.h移除 2、芯片定义公共结构体及函数ChipProductType 等从defs.h移除 3、通信枚举和结构体ProfPacketType 等从packet.h移除 4、ProfConfig结构体MessageOfProfConfig 等从data_format.h移除 5、移除后CmakeList及命名空间的修改 - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- https://gitcode.com/Ascend/msopcom/pull/68 ### 2. 功能验证 - [x] **功能自验截图**（请确保不体现个人信息） - [x] **冒烟是否通过** ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [X] **是否经过代码检视** - [x] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** [@MTQ999](https://gitcode.com/MTQ999) - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** NA ---- See merge request: Ascend/msopprof!62	3 个月前
test_packet.cpp	【refactor】适配opscommon头文件整改+微重构halHelper类中的全局变量 Co-authored-by: MTQ999<matianqi2@huawei.com> # message auto-generated for no-merge-commit merge: !73 merge rf into master 【refactor】适配opscommon头文件整改+微重构halHelper类中的全局变量 Created-by: MTQ999 Commit-by: MTQ999 Merged-by: ascend-robot Description: ### 1. 修改描述 - **修改原因：** opscommon仓微重构profconfig.h，调优组件进行相应修改+微重构halHelper类中的全局变量 - **修改方案：** opscommon仓微重构profconfig.h，调优组件进行相应修改，引用重构后的头文件 - **修改内容：** opscommon仓微重构profconfig.h，调优组件进行相应修改，引用重构后的头文件 - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- https://gitcode.com/Ascend/msopcom/pull/77 ### 2. 功能验证 - [ ] **功能自验截图**（请确保不体现个人信息）Y - [ ] **冒烟是否通过** Y ---- ### 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [ ] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- ### 4. 资料修改自检 - **资料修改：** NA ---- See merge request: Ascend/msopprof!73	3 个月前
test_process_manager.cpp	[feature]修复当前代码中的安全问题 Co-authored-by: zzzsss1234<zhanglei802@h-partners.com> # message auto-generated for no-merge-commit merge: !198 merge fix_safety into master [feature]修复当前代码中的安全问题 Created-by: zzzsss1234 Commit-by: zzzsss1234 Merged-by: ascend-robot Description: # 1. 修改描述 https://gitcode.com/Ascend/msot/issues/62 - **修改原因：** 当前代码中数据读取存在越界风险，且kernelname传递的路径也存在安全隐患 - **修改方案：** 1.修改当前数据读取逻辑，限制读取数据不越界 2.修改kernelname传递过程，确保数据传递安全 - **修改内容：** 1.csrc/op_profiling/profiling/device/data_parse/metric_data_handler.cpp 2.csrc/op_profiling/prof_injection/device_process_manager.cpp - [ ] **涉及代码双合**(贴上另一个PR链接)： ---- # 2. 功能验证 - [ ] **功能自验截图**（请确保不体现个人信息）  - [ ] **冒烟是否通过** pass ---- # 3. 代码检视 - **要求：** - 合入功能代码大于 200 行，需要sig会议申报代码检视议题，并在PR中标注会议。 - committer评估是否需要在sig会议进行代码检视。 - 参与检视的committer人员名单与检视时间。 - 大于 1000 行代码原则上不允许合入，需进行备案。 - [ ] **是否经过代码检视** - [ ] **是否具备UT测试用例看护** - [ ] **是否需要在sig会议中进行代码检视** - **检视committer人员名单与检视时间：** ---- # 4. 资料修改自检 - **资料修改：** ---- See merge request: Ascend/msopprof!198	17 小时前