If I kill a System.Diagnostics.Process with .Kill(), do I also need to call .Close()?

Chris Phillips picture Chris Phillips · Apr 13, 2011 · Viewed 8.3k times · Source

Using C# 4.0, I've created a System.Diagnostics.Process that I expect to take a short amount of time to run. If for some reason the process hasn't exited after some amount of time (e.g, I've called .WaitForExit(timeout) and the return value was false), I need to cleanup. I've decided that it's safe to use .Kill() in this particular situation (there are no data structures I'm worried about corrupting).

Given this setup, do I also need to call the .Close() method of the Process? If so, should I call .Close() before or after .Kill()?

Answer

Steve Guidi picture Steve Guidi · Apr 13, 2011

System.Diagnostics.Process implements IDisposable via the Component class, and the protected Dispose(bool) method calls Close(). It is generally considered good practice to dispose of disposable resources when you are done with them, as this will immediately release any resources associated with the class (not having to wait for the GC to run).

So to answer your question:

Yes, please call Close() by calling the Dispose() method directly or via the C# using construct as follows.

using(Process proc = CreateProcess())
{
    StartProcess(proc);
    if (!proc.WaitForExit(timeout))
    {
        proc.Kill();
    }
}