From 73763b954f9df1514f53ebd09e9c79165c536e78 Mon Sep 17 00:00:00 2001 From: Derek McGowan Date: Mon, 5 Jun 2017 13:50:56 -0700 Subject: [PATCH] service/content: fix logic error on storing extra data Clarify logic that extra data is stored when the target buffer is full. Existing logic allows for extra data to be stored even when more data will be read into buffer when the remaining space is less than what was copied from the last receive. Signed-off-by: Derek McGowan --- services/content/reader.go | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/services/content/reader.go b/services/content/reader.go index e40150231..fd4200e50 100644 --- a/services/content/reader.go +++ b/services/content/reader.go @@ -34,11 +34,9 @@ func (rr *remoteReader) Read(p []byte) (n int, err error) { n += copied p = p[copied:] - if copied < len(p) { - continue + if len(p) == 0 { + rr.extra = append(rr.extra, resp.Data[copied:]...) } - - rr.extra = append(rr.extra, resp.Data[copied:]...) } return