今天现场一个程序挂掉,查了很久,是一个之前人的代码出的问题,很简单,也很容易忽略。
代码如下:
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 转换为有符号的数。造成了下面一行的数组下标是负数的低级错误,程序挂掉。
再小的地方都有很多学问,任重道远。