【图文】华为代码review准则_第1页
【图文】华为代码review准则_第2页
【图文】华为代码review准则_第3页
【图文】华为代码review准则_第4页
【图文】华为代码review准则_第5页
已阅读5页，还剩14页未读， 继续免费阅读

下载本文档

版权说明：本文档由用户提供并上传，收益归属内容提供方，若内容存在侵权，请进行举报或认领

文档简介

1、 触发式检视19 l l l l l /*如果用户认证不成功*/ if (VOS_ERR = = AAA_UserAuth (ulCid /error; l l 应该是” VOS_OK != ”吧？ 注意这两种写法并不是等价的，因为可能还有其它的错误类型 VOS_sprintf ( szOutString, “%s”, szTemp ; l l l l VOS_sprintf ( szOutString, szTemp ; 这两种写法有区别吗？ 第1种写法是安全的，第2种写法时，如果szTemp含有转义字符(如”%s”，则极度 危险，严重至死机 HUAWEI TECHNOLOGIES CO.,

2、 LTD. Huawei Confidential Page 31 触发式检视20 l l l l l #define MAX_NAME_LEN 64 UCHAR szUserName MAX_NAME_LEN ； 64这个长度包不包括最后的0呢？那就是说名字最长是63咯？ l 这样定义通常会让人迷惑到底是多长，建议的做法是定义成(64 + 1。参考 aaa_pub.h l l l l l pMsg = VOS_Malloc(; if ( VOS_OK != AAA_SendMsg ( pMsg else 发送失败情况下pMsg在哪里释放的？发送成功情况下pMsg在哪里释放的？ 内存泄漏问题

3、Huawei Confidential Page 32 HUAWEI TECHNOLOGIES CO., LTD. 触发式检视21 l l if ( g_stVar.ulTimerId != 0 VOS_Timer_Delete ( g_stVar.ulTimerId ; l l l l 删除定时器前判断合法性了吗？ 删除定时器后，控制句柄清0了吗？ 其它类型资源的控制句柄都要这样处理 l l l if ( pMsp = = NULL VOS_Assert ( pMsg = = NULL ; l l l l return VOS_ERR; 断言的“= =” 应该是 ”!=” 经常碰到这类错误。断言在条件为“假”的时候工作 Huawei Confidential Page 33 HUAWEI TECHNOLOGIES CO., LTD. 触发式检视22 l l l l l if ( g_stVar.ulMemPtr != NULL VOS_Free( g_stVar.ulMemPtr ; 释放内存前判断指针合法性了吗？ l l 释放内存后把指针清空了吗？ 内存重复释放问题，空闲内存被改写问题等严重后果

人人文库> 全部分类> 教育资料 > 辅导培训

温馨提示

  • 1. 本站所有资源如无特殊说明，都需要本地电脑安装OFFICE2007和PDF阅读器。图纸软件为CAD,CAXA,PROE,UG,SolidWorks等.压缩文件请下载最新的WinRAR软件解压。
  • 2. 本站的文档不包含任何第三方提供的附件图纸等，如果需要附件，请联系上传者。文件的所有权益归上传用户所有。
  • 3. 本站RAR压缩包中若带图纸，网页内容里面会有图纸预览，若没有图纸预览就没有图纸。
  • 4. 未经权益所有人同意不得将文件中的内容挪作商业或盈利用途。
  • 5. 人人文库网仅提供信息存储空间，仅对用户上传内容的表现方式做保护处理，对用户上传分享的文档内容本身不做任何修改或编辑，并不能对任何下载内容负责。
  • 6. 下载文件中如有侵权或不适当内容，请与我们联系，我们立即纠正。
  • 7. 本站不保证下载资源的准确性、安全性和完整性, 同时也不承担用户因使用这些下载资源对自己和他人造成任何形式的伤害或损失。

最新文档

评论

0/150

提交评论