System.Net.NetworkInformation.Ping存在的“内存泄漏”问题

昨天写了一个监测网络状况的小工具，其实就是定时ping网关和一些地址。跑了一夜，今天发现电脑卡，昨天写的小工具占了4G多内存，明显内存泄露。小工具很简单，使用非托管资源的只有System.Net.NetworkInformation.Ping这个类。Ping调用的是异步方法SendAsync, 我在PingCompleted事件处理中已经Dispose了：

Code
private void OnPingCompleted (object sender, PingCompletedEventArgs e)
{
    
    (sender as Ping).Dispose();
}

于是调试，发现这句也确实执行了。不解，仔细研究代码。结果某次修改代码如下时，问题消失：

Code
private void OnPingCompleted (object sender, PingCompletedEventArgs e)
{
    using (sender as Ping)
    {
    
    }
}

于是思考两段代码的区别，难道是偶尔发生异常导致Dispose没有执行？将Dispose放到开头：

Code
private void OnPingCompleted (object sender, PingCompletedEventArgs e)
{
    (sender as Ping).Dispose ();
    
}

问题依旧。显式调用IDisposable.Dispose()，问题解决：

Code
private void OnPingCompleted (object sender, PingCompletedEventArgs e)
{
    (sender as IDisposable).Dispose ();
    
}

上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

转载于:https://www.cnblogs.com/Gildor/archive/2009/09/08/1562225.html