MySqlDataReader.GetStream() throws IndexOutOfRangeException

Question

So I'm using the MySql.Data package for my .NET Core project and store password hashes and salts (of type byte[]) as varbinary() to the database. When selecting the password hash and salt from a user I need a way to convert the result back to a byte array. Given this sample code

Stream passwordHashStream = dbDataReader.GetStream(0);
byte[] passwordHash;
                        
using (MemoryStream memoryStream = new MemoryStream())
{
    await passwordHashStream.CopyToAsync(memoryStream);
    passwordHash = memoryStream.ToArray();
}

the first line will throw this exception

System.IndexOutOfRangeException: Data index must be a valid index in the field at MySql.Data.MySqlClient.Interceptors.ExceptionInterceptor.Throw(Exception exception) at MySql.Data.MySqlClient.MySqlConnection.Throw(Exception ex) at MySql.Data.MySqlClient.MySqlDataReader.Throw(Exception ex) at MySql.Data.MySqlClient.MySqlDataReader.GetBytes(Int32 i, Int64 fieldOffset, Byte[] buffer, Int32 bufferoffset, Int32 length) at System.Data.Common.DbDataReader.GetStream(Int32 ordinal) at Infrastructure.Persistence.Repositories.UsersRepository.<>c.<b__1_0>d.MoveNext() in /.../Infrastructure/Persistence/Repositories/UsersRepository.cs:line 60

although the reader contains the correct database results as you can see here down below (password is 32 bytes, hash is 16 bytes)

The error was reported in 2018

https://bugs.mysql.com/bug.php?id=93374

and has a verified status. Are there any good solutions / workarounds on how to solve it? I don't want to use the GetBytes method because it requires me to pass in the length of the byte array and I would like to stay independent from that.

sorry, this method does not exist. dbDataReader.GetItem(0) — Question3r
– Question3r, Commented Jul 11, 2020 at 13:05

Bradley Grainger · Accepted Answer · 2021-01-17 05:52:15Z

2

Uninstall MySql.Data and replace it with MySqlConnector.

(Disclosure: I'm the reporter of the MySql bug you found and the primary contributor to MySqlConnector.)

As well as fixing that issue and many other bugs, MySqlConnector adds true async I/O support and performance improvements.

I don't want to use the GetBytes method because it requires me to pass in the length of the byte array and I would like to stay independent from that.

If you don't want to switch libraries, a little-known feature of GetBytes (supported by both MySql.Data and MySqlConnector) is that passing in a null buffer returns the needed length, so you don't need to hard-code it:

// get the length of the column and allocate a buffer
var length = dbDataReader.GetBytes(0, 0, null, 0, 0);
var passwordHash = new byte[length];

// fill the buffer from the column
dbDataReader.GetBytes(0, 0, passwordHash, 0, passwordHash.Length);

edited Jan 17, 2021 at 5:52

answered Jul 11, 2020 at 17:24

Bradley Grainger

28.3k4 gold badges95 silver badges117 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

LDS · Accepted Answer · 2020-07-11 12:33:07Z

0

public static async Task<MemoryStream> ToMemoryStream(this Stream stream)
    {
      var memStream = stream as MemoryStream;
      if (memStream == null)
      {
        memStream = new MemoryStream();
        await stream.CopyToAsync(memStream);
      }
      return memStream;
    }

answered Jul 11, 2020 at 12:33

LDS

3623 silver badges9 bronze badges

2 Comments

Question3r Over a year ago

sorry, I don't understand your solution. Could you explain it a little bit?

LDS Over a year ago

Asy function should call from asy function .I am not aware the function where you call CopyToAsync

Collectives™ on Stack Overflow

MySqlDataReader.GetStream() throws IndexOutOfRangeException

2 Answers 2

Comments

2 Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Comments

2 Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related