Proper handling of possible smaller arrays.

cincuranet · cincuranet · commit 50cfcf7ba0f7 · 2019-08-27T14:39:46.000+02:00
diff --git a/Provider/src/FirebirdSql.Data.FirebirdClient/FirebirdClient/FbDataReader.cs b/Provider/src/FirebirdSql.Data.FirebirdClient/FirebirdClient/FbDataReader.cs
@@ -412,12 +412,12 @@ public override int GetValues(object[] values)
 			CheckState();
 			CheckPosition();
 
-			for (var i = 0; i < _fields.Count; i++)
+			var count = Math.Min(_fields.Count, values.Length);
+			for (var i = 0; i < count; i++)
 			{
 				values[i] = CheckedGetValue(() => GetValue(i));
 			}
-
-			return values.Length;
+			return count;
 		}
 
 		public override bool GetBoolean(int i)

Original file line number	Diff line number	Diff line change
`@@ -412,12 +412,12 @@ public override int GetValues(object[] values)`
`412`	`412`	`CheckState();`
`413`	`413`	`CheckPosition();`
`414`	`414`
`415`		`- for (var i = 0; i < _fields.Count; i++)`
	`415`	`+ var count = Math.Min(_fields.Count, values.Length);`
	`416`	`+ for (var i = 0; i < count; i++)`
`416`	`417`	`{`
`417`	`418`	`values[i] = CheckedGetValue(() => GetValue(i));`
`418`	`419`	`}`
`419`		`-`
`420`		`- return values.Length;`
	`420`	`+ return count;`
`421`	`421`	`}`
`422`	`422`
`423`	`423`	`public override bool GetBoolean(int i)`