Fix NullReferenceException in TestResultsReader.Dispose when connecting the named pipe failed.

dgrunwald · dgrunwald · commit 4b4518db8698 · 2014-07-05T17:55:14.000+02:00
diff --git a/src/AddIns/Analysis/UnitTesting/TestRunner/TestResultsReader.cs b/src/AddIns/Analysis/UnitTesting/TestRunner/TestResultsReader.cs
@@ -61,8 +61,10 @@ public string PipeName {
 		
 		public void Dispose()
 		{
-			reader.Dispose();
-			namedPipe.Dispose();
+			if (reader != null)
+				reader.Dispose();
+			if (namedPipe != null)
+				namedPipe.Dispose();
 		}
 		
 		public event EventHandler<TestFinishedEventArgs> TestFinished;

Original file line number	Diff line number	Diff line change
`@@ -61,8 +61,10 @@ public string PipeName {`
`61`	`61`
`62`	`62`	`public void Dispose()`
`63`	`63`	`{`
`64`		`- reader.Dispose();`
`65`		`- namedPipe.Dispose();`
	`64`	`+ if (reader != null)`
	`65`	`+ reader.Dispose();`
	`66`	`+ if (namedPipe != null)`
	`67`	`+ namedPipe.Dispose();`
`66`	`68`	`}`
`67`	`69`
`68`	`70`	`public event EventHandler<TestFinishedEventArgs> TestFinished;`