原创力文档- 1、原创力文档（book118）网站文档一经付费（服务费），不意味着购买了该文档的版权，仅供个人/单位学习、研究之用，不得用于商业用途，未经授权，严禁复制、发行、汇编、翻译或者网络传播等，侵权必究。。
- 2、本站所有内容均由合作方或网友上传，本站不对文档的完整性、权威性及其观点立场正确性做任何保证或承诺！文档内容仅供研究参考，付费前请自行鉴别。如您付费，意味着您自己接受本站规则且自行承担风险，本站不退款、不进行额外附加服务；查看《如何避免下载的几个坑》。如果您已付费下载过本站文档，您可以点击 这里二次下载。
- 3、如文档侵犯商业秘密、侵犯著作权、侵犯人身权等，请点击“版权申诉”（推荐），也可以打举报电话：400-050-0827(电话支持时间：9:00-18:30)。
- 4、该文档为VIP文档，如果想要下载，成为VIP会员后，下载免费。
- 5、成为VIP后，下载本文档将扣除1次下载权益。下载后，不支持退款、换文档。如有疑问请联系我们。
- 6、成为VIP后，您将拥有八大权益，权益包括：VIP文档下载权益、阅读免打扰、文档格式转换、高级专利检索、专属身份标志、高级客服、多端互通、版权登记。
- 7、VIP文档为合作方或网友上传，每下载1次， 网站将根据用户上传文档的质量评分、类型等，对文档贡献者给予高额补贴、流量扶持。如果你也想贡献VIP文档。上传文档
查看更多
华为代码审查技术-Daniel Lee
华为代码审查技术
By Daniel Lee
什么是代码审查
代码审查用眼睛遍历代码分支，用大脑运行程序，是
静态白盒测试方法的一种。
关键模块: 专人在了解特性、文档后细查
非关键模块: 多人快速扫描
内存篇
内存申请是否成功，需要判断
内存使用前是否初始化
申请释放接口是否匹配
申请空间大小是否合理
访问是否越界
是否存在泄漏
是否重复释放
中断篇
中断相关寄存器初始化(触发方式，屏蔽)
全局变量的使用是否正确
ISR中不能用信号量获取函数，不可用malloc/free
不能使用浮点处理的运算
不能用设备驱动设计的I/O操作，如printf
不可用耗时操作函数
代码审查九句真言
看见了If，就想Else。
看见malloc，就去找Free。
函数调用要小心，需要看看返回值。
看到for循环，就找边界值。
看见return要注意，要去前面找资源。
看见数组把神提，问题往往在下标。
不要小看字符串，长度是个大问题。
得到函数不要急，看看变量初始化，各种路径要小心。
赋值函数最危险，变量没有初始化。
九句句真言不孤立，相互结合显神威。
基站平台TOP15低级编码错误
TOP1:资源泄漏
TOP2:内存拷贝越界
TOP3:变量没有赋值
TOP4:数组下标越界
TOP5:边界错误
TOP6:数据溢出
TOP7:时间关键路径有耗时操作
TOP8:内存重复释放
基站平台TOP15低级编码错误
TOP9:字节对齐错误
TOP10:访问空指针
TOP11:指针释放后没有赋值为NULL
TOP12:临界区未保护
TOP13:数据类型不匹配
TOP14:除零错误
TOP15:堆栈溢出
文档评论（0）