华为代码review准则.pptVIP

触发式检视13 UCHAR szOutString[204800]; … for (i = 0; i MAX_NUM; i++) { VOS_sprintf(szOutString…); } 204800够大了吧？ 太大的局部变量导致任务堆栈被破坏；建议申请动态内存 Page * 触发式检视14 if ( ulItemFlag FLAG_DELETED) { …} 是””而不是”” 此类还有”|”与”| |”等。同时注意什么情况下可以用”= =” for ( ucIndex = 0; ucIndex 256; ucIndex++) { …} 循环退出的条件可以满足吗？ 死循环问题 Page * 触发式检视15 UCHAR ucEncryptedPasswd[ 128 ]; UCHAR ucMacAddr[ 6 ]?; … vos_printf ( ucEncryptedPasswd ); vos_printf ( ucMacAddr ); ... 变量中存放的并不是字符串，这样输出的有不可预知的后果，重则死机 这种情况要用16进制打印ASCII码值 Page * 触发式检视16 VOS_StrCpy ( szDest, szSrc)?; VOS_StrNCpy ( szDest, szSrc, ulLen)?; VOS_MemCpy (pDest, pSrc,