152 Star 679 Fork 532

OpenHarmony / kernel_liteos_m

 / 详情

los_memory.c 变量都未进行初始化,将来出了问题不好定位

已拒绝
任务
创建于  
2020-09-11 10:19

LITE_OS_SEC_TEXT_INIT UINT32 OsMemSystemInit()
{
UINT32 ret; //here
UINT32 memSize; //here
m_aucSysMem0 = (UINT8 *)(UINTPTR)(((UINT32)(UINTPTR)m_aucSysMem0 + (OS_MEM_NODE_ALIGN_SIZE - 1)) &
~(OS_MEM_NODE_ALIGN_SIZE - 1));
memSize = ((UINT32)g_sysMemAddrEnd) - OS_SYS_NOCACHEMEM_SIZE - (UINT32)(UINTPTR)m_aucSysMem0;
ret = LOS_MemInit(m_aucSysMem0, memSize);
PRINT_INFO("LiteOS heap memory address:0x%x,size:0x%x\n", m_aucSysMem0, memSize);
return ret;
}

评论 (5)

陈国 创建了任务
陈国 关联仓库设置为OpenHarmony/kernel_liteos_m
展开全部操作日志

这里可以不进行初始化,使用前有赋值操作即可。否则会冗余初始化。

提交代码issue的时候,格式能不能搞好啊 :sweat:

华为的编程规范不允许重复初始化,这个你放心,如果后面修改导致变量未初始化而被使用,很容易在白盒检查时拦截到

参见上述评论,非问题,不优化,issue关闭

Denny 任务状态待办的 修改为已完成
Denny 任务状态已完成 修改为已拒绝

登录 后才可以发表评论

状态
负责人
项目
里程碑
Pull Requests
关联的 Pull Requests 被合并后可能会关闭此 issue
分支
开始日期   -   截止日期
-
置顶选项
优先级
预计工期 (小时)
参与者(6)
8046261 mgy917 1599794627 1564930 dennyshen 1615514438
C
1
https://gitee.com/openharmony/kernel_liteos_m.git
git@gitee.com:openharmony/kernel_liteos_m.git
openharmony
kernel_liteos_m
kernel_liteos_m

搜索帮助

14c37bed 8189591 565d56ea 8189591