这里的内存delete该怎么用？

[muders]

代码：
long CCardPwdtestDlg::StoL_10(CString str)
{//取字符串所对应的数值
long temp,i;
char *str1;
temp=str.GetLength()+1;
str1=new char[temp];
strcpy(str1,str.GetBuffer(0));
temp=strtol(str1,&str1,10);
i=temp;
delete []str1; //<——这里出错，如下几种写法都出错
return temp;
}
出错句子写成delete str1; delete [] str1; delete[] str1;都是同样的错误：
内存不可读。
令temp=str.GetLength();也一样，但是去掉delete就没事了，这个内存该如何释放？请高手指点一下

[闻缺陷则喜何志丹]

strcpy(str1,str.GetBuffer(0));
=======================
改成
strcpy(str1,(LPCTSTR)str);

[闻缺陷则喜何志丹]

不如:
sscanf((LPCTSTR)str,"%ld",&temp);

[as2001]

有没有试过 delete str1？看看行不行！

[lzwcom]

问题出在 strtol(str1,&str1,10);
long strtol( const char *nptr, char **endptr, int base );
第二个参数 endptr不需要你进行分配空间，例子看MSDN
long CCardPwdtestDlg::StoL_10(CString str)
{//取字符串所对应的数值
long temp,i;
char *str1,*str2;
temp=str.GetLength()+1;
str1=new char[temp];
strcpy(str1,str.GetBuffer(0));
temp=strtol(str1,&str2,10); //str2不需要你分配空间
i=temp;
delete []str1;
str.ReleaseBuffer(); //-->这个很重要，否则会内存泄漏,或者改为
strcpy(str1,(LPCTSTR)str);
return temp;
}

[HongHuer]

str1=new char[temp];//temp = 32*n 这样，n为整数

[mumutouv]

up