Skip to content

Commit 95cd0b1

Browse files
committed
PR feedback
1 parent 27079fe commit 95cd0b1

1 file changed

Lines changed: 7 additions & 2 deletions

File tree

src/Microsoft.Net.Http.Client/ChunkedReadStream.cs

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@ public override async Task<int> ReadAsync(byte[] buffer, int offset, int count,
6464
{
6565
if (_chunkBytesRemaining == 0)
6666
{
67-
var headerLine = await _inner.ReadLineAsync(memoryStream ??= new MemoryStream(), cancellationToken)
67+
var headerLine = await _inner.ReadLineAsync(memoryStream = new MemoryStream(), cancellationToken)
6868
.ConfigureAwait(false);
6969

7070
if (!int.TryParse(headerLine, NumberStyles.HexNumber, CultureInfo.InvariantCulture, out _chunkBytesRemaining))
@@ -95,7 +95,12 @@ public override async Task<int> ReadAsync(byte[] buffer, int offset, int count,
9595
var emptyLine = await _inner.ReadLineAsync(memoryStream ??= new MemoryStream(), cancellationToken)
9696
.ConfigureAwait(false);
9797

98-
if (!string.IsNullOrEmpty(emptyLine))
98+
if (emptyLine == null)
99+
{
100+
throw new EndOfStreamException();
101+
}
102+
103+
if (emptyLine.Length > 0)
99104
{
100105
throw new IOException($"Expected an empty line, but received: '{emptyLine}'.");
101106
}

0 commit comments

Comments
 (0)