今天帮个同学改个东西..突然出现个问题,求达人指点下..
    附件里边的关键代码如下:

Copy code

//就是按下OK按钮发生的事件
//分配一个空间->将一段字符串复制进去->计算检验和->释放空间
void CAddressQDlg::OnOK()
{

    //char *buff=new char[strlen("AT+SSCAN,*")+5];
    char *buff=new char[63];//我的机器上少于63就会报错 
    if(!buff) return;
    strcpy(buff,"AT+SSCAN,*");
    checkSum(buff,strlen(buff)+1);
                      //sendDataFunction(buff,strle(buff+1));
    delete []buff;
    //CDialog::OnOK();
}
//同学写的检验函数
char * CAddressQDlg::checkSum(char *str,int length)
{
    char p,q;               
    char k='\r';
    char j='\n';
    char *temp;
    temp=str;
    temp=temp+2;
    for(p=*(temp+1);*(temp+1)!='*';temp++)
      p=p^*(temp+1);
    q=p;
    p=p&(char)15;
    q=(q&(char)(15<<4));
    strcat(str,&q);
    strcat(str,&p);
    strcat(str,&k);
    strcat(str,&j);
    return str;
}

问题的关键就出在第一句...

Copy code

char *buff=new char[strlen("AT+SSCAN,*")+5]

这句分配出来的空间居然不够..然后发现不分配大于等于63的char空间都会出错,真的不明白..


[ 此帖被lxlcn在2007-12-25 21:58重新编辑 ]

老大，如果buff长度过小的话缓冲区要溢出。

    strcat(str,&q);
    strcat(str,&p);
    strcat(str,&k);
    strcat(str,&j);
这一类函数是专gate搞缓冲区溢出的，如果str串空间过小，上面这几个函数将会将checkSum函数返回地址冲刷掉，

Quote:

引用第7楼lxlcn于2007-12-25 22:25发表的  :

Buff不够大是明显的,但是我想问的就是为什么?..

strcpy(),strcat(),scanf()等一系列函数对参数实际长度都没有检查，而你的buff其实是在栈空间里分配的，单从地址上看，buff的地址与主函数的返回地址是相当靠近的

比如在main函数中，申明一个数组char buff[2];那么堆栈内容是：
byte1  <-----------ebp
byte2(buff[1])
byte3 (buff[0]) <-----------esp
如果你用strcat函数，它可能将把byte1或更上面的给冲掉，那么函数就反不回去了

Quote:

引用第10楼lxlcn于2007-12-25 22:38发表的  :


没有检查对分配由什么影响呢？
buff是在堆中分配的，不会很近吧？好像关系也不大吧？

你可以做个试验。