Fix DNET749 (StackOverflow on CommandBehavior.CloseConnection)

cincuranet · cincuranet · commit 92292a3ec9c9 · 2017-04-05T14:27:09.000+02:00
diff --git a/Provider/src/FirebirdSql.Data.FirebirdClient/FirebirdClient/FbDataReader.cs b/Provider/src/FirebirdSql.Data.FirebirdClient/FirebirdClient/FbDataReader.cs
@@ -159,6 +159,7 @@ protected override void Dispose(bool disposing)
 			{
 				if (!IsClosed)
 				{
+					_isClosed = true;
 					if (_command != null && !_command.IsDisposed)
 					{
 						if (_command.CommandType == CommandType.StoredProcedure)
@@ -175,7 +176,6 @@ protected override void Dispose(bool disposing)
 					{
 						_connection.Close();
 					}
-					_isClosed = true;
 					_position = StartPosition;
 					_command = null;
 					_connection = null;

Original file line number	Diff line number	Diff line change
`@@ -159,6 +159,7 @@ protected override void Dispose(bool disposing)`
`159`	`159`	`{`
`160`	`160`	`if (!IsClosed)`
`161`	`161`	`{`
	`162`	`+ _isClosed = true;`
`162`	`163`	`if (_command != null && !_command.IsDisposed)`
`163`	`164`	`{`
`164`	`165`	`if (_command.CommandType == CommandType.StoredProcedure)`
`@@ -175,7 +176,6 @@ protected override void Dispose(bool disposing)`
`175`	`176`	`{`
`176`	`177`	`_connection.Close();`
`177`	`178`	`}`
`178`		`- _isClosed = true;`
`179`	`179`	`_position = StartPosition;`
`180`	`180`	`_command = null;`
`181`	`181`	`_connection = null;`