2009-01-18 Zoltan Varga <vargaz@gmail.com>

	* Process.cs: Avoid closing the duplicated process handle in
	ProcessWaitHandle, it is done automatically by the WaitHandle.Dispose ().
	Fixes #464628.

svn path=/trunk/mcs/; revision=123738

1 files changed, 2 insertions, 20 deletions

diff --git a/mcs/class/System/System.Diagnostics/Process.cs b/mcs/class/System/System.Diagnostics/Process.cs
index d2d510c1c08..a26719e84a5 100644
--- a/mcs/class/System/System.Diagnostics/Process.cs
+++ b/mcs/class/System/System.Diagnostics/Process.cs
@@ -1665,27 +1665,9 @@ namespace System.Diagnostics {
 				// Need to keep a reference to this handle,
 				// in case the Process object is collected
 				Handle = ProcessHandle_duplicate (handle);
-			}
-
-			[MethodImplAttribute (MethodImplOptions.InternalCall)]
-			private extern static void ProcessHandle_close (IntPtr handle);
-			
-			private bool disposed = false;
-			
-			protected override void Dispose (bool explicitDisposing)
-			{
-				if (this.disposed == false) {
-					this.disposed = true;
-					
-					ProcessHandle_close (Handle);
-					Handle = IntPtr.Zero;
-				}
-				base.Dispose (explicitDisposing);
-			}
 
-			~ProcessWaitHandle ()
-			{
-				Dispose (false);
+				// When the wait handle is disposed, the duplicated handle will be
+				// closed, so no need to override dispose (bug #464628).
 			}
 		}
 	}