检查 NULL 和不必要的分配

2006-10-9　网友评论 0 条　点击进入论坛

Static void CMyApp_HowManyMistakesCanIMakeInOneFunction(
CmyApp *pme,const AECHAR * pszError)
{
AECHAR * pMsg;
int i;

i = (WSTRLEN(pszError) + 1) * sizeof(AECHAR);
pMsg = (AECHAR *)MALLOC(i);
WSTRCPY(pMsg,pszError);
ISHELL_MessageBox(pme->a.m_pIShell, pMsg,...);
FREE(pMsg);
}
此代码有什么问题？

此函数中有几个非常严重的错误。它包括：
无法检查确保 pszError 在使用前为 NULL
无法检查 MALLOC 的 NULL 返回值
不需要时分配 pMsg。 “pszError”值可以直接传递给 ISHELL_MessageBox
所有这些错误都非常严重。简言之，它们将一个可用于显示消息框的简单函数转换成一个失效的函数。

BREW™ 应用程序必须检查内存分配例程的返回值 NULL、返回类实例的值等。

已有 0 位对此文章感兴趣的网友发布了看法　

我来评两句	登录邮箱：密码：
请您在这里发表您的个人看法，发言时请各位遵纪守法并注意语言文明！
	匿名发表

今日推荐

技术文库(共有 46732 篇文章)

产品技术答疑

重点推荐