msdn例子代码的一个潜在bug

早就听说msdn里的例子代码问题多,今天也看到一处。

例子代码url:
http://msdn2.microsoft.com/en-us/library/bb250436.aspx

问题的代码:
     //  Convert the text from Unicode to ANSI
    LPTSTR psz  =   new  TCHAR[SysStringLen(bstrHTMLText)];
    lstrcpy(psz, OLE2T(bstrHTMLText));

两个问题:

1、这里Unicode转换ANSI,如果工程没有定义为UNICODE的话,TCHAR就等于char,把实际上SysStringLen * 2大小的串往SysStringLen大小的堆内存里拷贝,就会造成一个堆溢出。

2、SysStringLen大小也不对,该申请的内存大小应该是SysStringLen + 1。

严谨的代码应该这样写:

DWORD dwLen  =  (SysStringLen(bstrHTMLText)  +   1 *  ( sizeof (WCHAR)  /   sizeof (TCHAR));
LPTSTR psz 
=   new  TCHAR[dwLen];
memset(psz, 
0 , dwLen  *   sizeof (TCHAR));
lstrcpy(psz, OLE2T(bstrHTMLText));

不知道实际情况下有没有有问题的程序,赫赫。

  • 0
    点赞
  • 0
    收藏
    觉得还不错? 一键收藏
  • 0
    评论
评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值