System.Net.NetworkInformation.Ping存在的“内存泄漏”问题
昨天写了一个监测网络状况的小工具,其实就是定时ping网关和一些地址。跑了一夜,今天发现电脑卡,昨天写的小工具占了4G多内存,明显内存泄露。小工具很简单,使用非托管资源的只有System.Net.NetworkInformation.Ping这个类。Ping调用的是异步方法SendAsync, 我在PingCompleted事件处理中已经Dispose了:
![](/Images/OutliningIndicators/ContractedBlock.gif)
于是调试,发现这句也确实执行了。不解,仔细研究代码。结果某次修改代码如下时,问题消失:
![](/Images/OutliningIndicators/ContractedBlock.gif)
于是思考两段代码的区别,难道是偶尔发生异常导致Dispose没有执行?将Dispose放到开头:
![](/Images/OutliningIndicators/ContractedBlock.gif)
问题依旧。显式调用IDisposable.Dispose(),问题解决:
![](/Images/OutliningIndicators/ContractedBlock.gif)
上MSDN查,发现Ping.Dispose()并非实现了IDisposable.Dispose()方法,而是直接继承了Component.Dispose(),换言之,这个方法根本没有对Ping对象的非托管资源进行释放。反而Ping类显示实现了IDisposable接口,必须将其强转为IDisposable才能正确调用真正的Dispose()方法。
再看Component类,Component.Dispose()方法是一个非虚方法,而Component.Dispose(bool)是一个虚方法,Component.Dispose()方法只是对Component.Dispose(bool)的一个简单调用。按照微软的设计思想,Ping类应该重写Component.Dispose(bool)方法,实现资源释放,而不应显式实现IDisposable接口。因此,这应是一个bug。微软已声明将修复它,但从.NET 2.0开始,直到目前.NET 4.0 Beta 1经实验依然没有解决。
为避免遇到类似问题,再次强烈建议使用using语句块!
国外探讨此问题的两篇文章:
http://blog.mbcharbonneau.com/2006/11/14/using-the-ping-class-in-net-20-without-memory-leaks/
http://www.julmar.com/blog/mark/PermaLink,guid,fc83197b-ce2d-47ab-b70a-db352085b370.aspx
微软关于此bug的报告及回复:
http://connect.microsoft.com/VisualStudio/feedback/ViewFeedback.aspx?FeedbackID=98421
posted on 2009-09-08 03:55 Gildor Wang 阅读(3014) 评论(5) 编辑 收藏 举报