切忌!不要用int和sizeof()返回的无符号类型比较

今天现场一个程序挂掉,查了很久,是一个之前人的代码出的问题,很简单,也很容易忽略。

代码如下:

result = m_rd.GetCurRecordVal(SYSTEM_TYPE_NSA_CD,
TABLE_CMENU_INFO,
"menuName",pMenuName,MENU_NAME_LEN,XTRUE,
"",0,0,XFALSE,
"",0,0,XFALSE,
"",0,0,XFALSE,
"",0,0,XFALSE,
buff,XTRUE);
if(pMenuName)
{
delete []pMenuName;
pMenuName = NULL;
}
buff_len = result;
// if (result>0)
// {
if(buff_len>=sizeof(cMenu_Info))
{
pfix = new cMenu_Info[buff_len/sizeof(cMenu_Info)];

......

.......


result 是读数据库函数返回值,之前一直没发现这个问题是因为result 一直是正数,今天现场的模型发生了变化,result是-275,而此程序之前并未判断if (result>0)就把result和sizeof()进行比较,结果就是-275>=168返回true。sizeof返回是unsigned int,而result是int;

编译器里面有标准的转换,这个是在整形运算的时候出现。标准转换的规则是:短的的向长的转;有符号的向无符号的转。如果被转换的数据比转换后的数据要长的话,转换可能会丢失bit数据。

此处int 转换为有符号的数。造成了下面一行的数组下标是负数的低级错误,程序挂掉。

再小的地方都有很多学问,任重道远。

评论 1
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

当前余额3.43前往充值 >
需支付:10.00
成就一亿技术人!
领取后你会自动成为博主和红包主的粉丝 规则
hope_wisdom
发出的红包
实付
使用余额支付
点击重新获取
扫码支付
钱包余额 0

抵扣说明:

1.余额是钱包充值的虚拟货币,按照1:1的比例进行支付金额的抵扣。
2.余额无法直接购买下载,可以购买VIP、付费专栏及课程。

余额充值