已关闭
[Bug-Report|缺陷反馈]: log代码白盒扫描问题修复 #1067
GuoWenbo创建于  17 天前关闭于  8 小时前
GuoWenbo成员
17 天前 创建

Describe the current behavior / 问题描述 (Mandatory / 必填)

使用http://skills-market.dipnet.test.huawei.com/skill/code-bug-scanner 提供的skills扫描出log相关代码,
得到的扫描结果为:112abe3554cb4f01a568895ae831c1b9.xlsx
根据严重程度和置信度进行分类,然后按照严重程度逐个分析Critical级别的缺陷

Environment / 环境信息 (Mandatory / 必填)

NA

Steps to reproduce the issue / 重现步骤 (Mandatory / 必填)

NA

Describe the expected behavior / 预期结果 (Mandatory / 必填)

NA

NA

Special notes for this issue/备注 (Optional / 选填)

likedislike
GGuoWenbo成员
17 天前 添加了label:bug-report
GGuoWenbo成员
17 天前 关联了pull request:fix: 修复log代码白盒扫描Critical级缺陷(空指针/竞态/UAF/越界/语义反转) (#1067)
GuoWenbo成员
4 天前 评论:

另外有一个代码缺陷:DlogReportSendEndMsg函数内g_plogClient判空后,DlogReportUnLock的调用是多余的

likedislike
GGuoWenbo成员
4 天前 关联了pull request:fix: 修复log白盒扫描High级缺陷批1-slogd内存安全(栈溢出/越界写/UAF/解码越界) (#1067)
此处折叠了5条事件消息 查看更多
Cchenhuan1061620成员
3 天前 删除了label:wait-feedback
GuoWenbo成员
3 天前 评论:

PR合入后有报错 runtime/src/dfx/log/syslog/common/log_print_selflog.c:170:23: error: comparison of integer expressions of different signedness: 'int' and 'unsigned int' [-Werror=sign-compare]
170 | if (retryTime >= SELF_LOG_ROTATE_MAX_RETRY) {
单独提个PR快速修复一下

likedislike
GGuoWenbo成员
3 天前 关联了pull request:fix: 修复log_print_selflog.c符号比较告警(retryTime int改uint32_t) (#1067)
GuoWenbo成员
3 天前 评论:

PR5有编译报错
CMakeFiles/slogd.dir//////////runtime/src/dfx/log/syslog/slog/log_server/log_mgr/buffer_mgr/slogd_buffer.c.o.d -o CMakeFiles/slogd.dir//////////runtime/src/dfx/log/syslog/slog/log_server/log_mgr/buffer_mgr/slogd_buffer.c.o -c /home/phisik3/jenkins/workspace/work_code/runtime/src/dfx/log/syslog/slog/log_server/log_mgr/buffer_mgr/slogd_buffer.c
../../../../../../../../../../../../../runtime/src/dfx/log/syslog/slog/log_server/log_mgr/buffer_mgr/slogd_buffer.c:180:26: error: use of undeclared identifier 'g_bufMgrRegMutex'
(void)ToolMutexLock(&g_bufMgrRegMutex);
^
../../../../../../../../../../../../../runtime/src/dfx/log/syslog/slog/log_server/log_mgr/buffer_mgr/slogd_buffer.c:187:28: error: use of undeclared identifier 'g_bufMgrRegMutex'
(void)ToolMutexUnLock(&g_bufMgrRegMutex);
^
2 errors generated.

likedislike
GuoWenbo成员
3 天前 评论:

PR5 编译报错已修复(PR #5827 提交 6e7c2b83e):

根因:注册锁 g_bufMgrRegMutex 的声明被放在 #ifdef STATIC_BUFFER 块内,而使用点 SlogdBufferInit 在 #endif 之后的公共代码段。slogd 的 rc/esl 产品变体(ascend310rc/310Brc/helper310p/950+RC/960esl 等)不定义 STATIC_BUFFER,声明被预处理掉而使用点保留,报 undeclared identifier。ep_slogd_utest 恒定义该宏(CMakeLists L190/240),故本地 UT 与 CI 均未拦截。

修复:声明移出条件编译至文件作用域(紧邻被保护的 g_slogdBufMgr 头指针),两种产品配置下均可见;已回放工程实际编译命令验证含/不含 -DSTATIC_BUFFER 均编译通过(修复前无宏配置 exit=1 精确复现本报错)。

另外:上午评论提出的 log_print_selflog.c 符号比较告警(retryTime int vs SELF_LOG_ROTATE_MAX_RETRY unsigned),修复提交已备好(分支 fix-selflog-sign,retryTime int→uint32_t 单行修复),正在创建独立 PR。

likedislike
GGuoWenbo成员
3 天前 关联了pull request:fix: 修复log_print_selflog.c符号比较告警(retryTime int改uint32_t) (#1067)
chenhuan1061620成员
1 天前 评论:

🔔 @GuoWenbo您好,关联PR #5422 已合并,请在2天内检视,超期将视为检视通过,Issue将自动关闭。

likedislike
chenhuan1061620成员
13 小时前 评论:

🔔 @GuoWenbo您好,关联PR #5818 已合并,请在2天内检视,超期将视为检视通过,Issue将自动关闭。

likedislike
chenhuan1061620成员
13 小时前 评论:

🔔 @GuoWenbo您好,关联PR #5820 已合并,请在2天内检视,超期将视为检视通过,Issue将自动关闭。

likedislike
chenhuan1061620成员
13 小时前 评论:

🔔 @GuoWenbo您好,关联PR #5821 已合并,请在2天内检视,超期将视为检视通过,Issue将自动关闭。

likedislike
chenhuan1061620成员
13 小时前 评论:

🔔 @GuoWenbo您好,关联PR #5842 已合并,请在2天内检视,超期将视为检视通过,Issue将自动关闭。

likedislike
ykl999ykl999成员
8 小时前 issue状态由 进行中 改变为 已解决
ykl999ykl999成员
8 小时前 关闭了 issue
ykl999
ykl999成员
8 小时前 评论:

pr已全部上库,问题已解决

likedislike
GGuoWenbo成员
7 小时前 关联了pull request:fix: 修复log白盒扫描Medium/Low级缺陷批A-内存安全(越界/溢出/TOCTOU/边界) (#1067)
GGuoWenbo成员
5 小时前 关联了pull request:fix: 修复log白盒扫描Medium/Low级缺陷批B-功能正确性(回滚/配额/解析/边界) (#1067)