Windows Embedded CE 6 msmqadm工具的一个bug

在使用C++进行MSMQ开发时,发现了reg.cxx文件中的一个写注册表失败时的Bug,并提出了修复建议。该Bug可能是由于代码复制过程中未完全调整参数导致。

最近继续使用C++做MSMQ的开发,最简单的方法就是看微软的源代码,在做MSMQ管理部分,参考reg.cxx,发现了一个bug,其实这个bug没什么大不了,为了证明自己“眼光敏捷”,吹毛求疵的把这个bug报告一下。

 

这bug在reg.cxx文件的273行,如果执行msmqadm register install 的时候,写注册表失败就会进入这段代码。我估计这个bug的出现是因为拷贝复制导致的,因为前后的代码都非常类似,如果把这段代码封装起来,那么就不会出现这个bug了。

dwValue = 1;
if (ERROR_SUCCESS != (hr = RegSetValueEx (hKey, L"Index", 0, REG_DWORD, (BYTE *)&dwValue, sizeof(DWORD)))) {
RegCloseKey (hKey);
rsc_wprintf (IDS_CANT_WRITE_KEY, L"Keep", hr);
return 1;
}

应该改为

dwValue = 1;
if (ERROR_SUCCESS != (hr = RegSetValueEx (hKey, L"Index", 0, REG_DWORD, (BYTE *)&dwValue, sizeof(DWORD)))) {
RegCloseKey (hKey);
rsc_wprintf (IDS_CANT_WRITE_KEY, L"Index", hr);
return 1;
}

 

image

我想这段代码没有走Unit Test,至少没有做写注册表失败的Unit Test。

其实Unit Test全面覆盖真的很难,最近自己也写很多Test cases,写的代码中,有一半为测试代码。其实还远远不够,因为要覆盖全部异常情况,测试代码应该是功能代码的好几倍。任重而道远呀。


    本文转自Jake Lin博客园博客,原文链接:http://www.cnblogs.com/procoder/archive/2010/08/24/Windows_CE_6_msmqadm_Bug.html,如需转载请自行联系原作者


评论
添加红包

请填写红包祝福语或标题

红包个数最小为10个

红包金额最低5元

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

抵扣说明:

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

余额充值