Check the return value of Read

Fixes #1221

If Read returns 0 unexpectedly, we have reached the end of the stream.

Author: lukebakken

projects/RabbitMQ.Client/client/impl/Frame.cs

@@ -237,20 +237,31 @@ internal static InboundFrame ReadFrom(Stream reader, byte[] frameHeaderBuffer, A
                     break;
             }
 
-            reader.Read(frameHeaderBuffer, 0, frameHeaderBuffer.Length);
+            if (reader.Read(frameHeaderBuffer, 0, frameHeaderBuffer.Length) == 0)
+            {
+                throw new EndOfStreamException("Reached the end of the stream while reading the frame header length.");
+            }
+
             int channel = NetworkOrderDeserializer.ReadUInt16(new ReadOnlySpan<byte>(frameHeaderBuffer));
             int payloadSize = NetworkOrderDeserializer.ReadInt32(new ReadOnlySpan<byte>(frameHeaderBuffer, 2, 4)); // FIXME - throw exn on unreasonable value
 
             const int EndMarkerLength = 1;
             // Is returned by InboundFrame.Dispose in Connection.MainLoopIteration
             var readSize = payloadSize + EndMarkerLength;
             byte[] payloadBytes = pool.Rent(readSize);
+            int read = 0;
             int bytesRead = 0;
             try
             {
                 while (bytesRead < readSize)
                 {
-                    bytesRead += reader.Read(payloadBytes, bytesRead, readSize - bytesRead);
+                    read = reader.Read(payloadBytes, bytesRead, readSize - bytesRead);
+                    if (read == 0)
+                    {
+                        throw new EndOfStreamException("Reached the end of the stream while reading frame payload.");
+                    }
+
+                    bytesRead += read;
                 }
             }
             catch (Exception)