精易论坛

标题: 关于《使用GDI+在内存转换图像格式》的BUG [打印本页]

作者: 热水哥 时间: 2014-7-30 10:31
标题: 关于《使用GDI+在内存转换图像格式》的BUG
http://125.confly.eu.org/thread-31530-1-1.html
http://bbs.eyuyan.com/read.php?tid=127301
《使用GDI+在内存转换图像格式》这是个很老的例子了.偶尔听到一些易友说有时候代码会出错。其实当时我修复过BUG，但当时已超过了可编辑帖子的时间，于是一直没有重新发布代码，后来时间久了甚至忘记修改过哪些地方了。
今天也有易友提到这个问题，于是我还是重新检查了一下代码。
貌似主要问题出在这一句：
CreateStreamOnHGlobal (hMem, 真, Stream)
MSDN有这么一段话

If the caller sets the fDeleteOnRelease parameter to FALSE,then the caller must also free the hGlobal after the final release. Ifthe caller sets the fDeleteOnRelease parameter to TRUE, the finalrelease will automatically free the hGlobal as shown in the followingexample code.
if(pData->fDeleteOnRelease)
{
Verify(0==GlobalFree(pData->hGlobal));
}

然而，那份代码已经手动调用了GlobalFree来释放资源，那么那行代码的第二个参数应该设置为“假”，也就是这样：
CreateStreamOnHGlobal (hMem, 假, Stream)

另外，根据MSDN对CreateStreamOnHGlobal第一个参数的说明，其中有这么一句“The handle must be allocated as movable and nondiscardable”，也就是说，GlobalAlloc应该使用GMEM_MOVEABLE和GMEM_NODISCARD常量。由于GMEM_NODISCARD已经过时所以被忽略了，那么应该只需要设置成GMEM_MOVEABLE就可以了。

下面附件是修复BUG后的代码

使用GDI 在内存转换图像格式[email protected] (4.94 KB, 下载次数: 141)

另一种方法：
当CreateStreamOnHGlobal (hMem, 真, Stream)时，把 GlobalFree (hMem) 这一行代码删除也是可以的。
既然fDeleteOnRelease选择了TRUE，那么对象释放的时候会自动Free掉HGLOBAL。
虽然这也是仅仅删除一行代码，但也上传一份源码吧