diff options
author | Joe Tsai <joetsai@digital-static.net> | 2017-09-25 14:23:38 -0700 |
---|---|---|
committer | Joe Tsai <thebrokentoaster@gmail.com> | 2017-09-25 23:05:58 +0000 |
commit | 57f7bc3a057a4e104349e72f2834d5e47e8518bd (patch) | |
tree | 4e6ba0412ad29529f6f32a1881d332c2e6b6ae6c /src/compress | |
parent | 4d3d333468b6a36106550bf90d93b3cc88b1efa4 (diff) | |
download | go-57f7bc3a057a4e104349e72f2834d5e47e8518bd.tar.gz go-57f7bc3a057a4e104349e72f2834d5e47e8518bd.zip |
compress/bzip2: fix checksum mismatch on empty reads
Previously, the read method checked whether the current block
was fully consumed or not based on whether the buffer could be filled
with a non-zero number of bytes. This check is problematic because
zero bytes could be read if the provided buffer is empty.
We fix this case by simply checking for whether the input buffer
provided by the user was empty or not. If empty, we assume that
we could not read any bytes because the buffer was too small,
rather than indicating that the current block was fully exhausted.
This check causes bzip2.Reader to be unable to make progress
on the next block unless a non-empty buffer is provided.
However, that is an entirely reasonable expectation since a
non-empty buffer needs to be provided eventually anyways to
read the actual contents of subsequent blocks.
Fixes #22028
Change-Id: I2bb1b2d54e78567baf2bf7b490a272c0853d7bfe
Reviewed-on: https://go-review.googlesource.com/66110
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Diffstat (limited to 'src/compress')
-rw-r--r-- | src/compress/bzip2/bzip2.go | 2 | ||||
-rw-r--r-- | src/compress/bzip2/bzip2_test.go | 8 |
2 files changed, 9 insertions, 1 deletions
diff --git a/src/compress/bzip2/bzip2.go b/src/compress/bzip2/bzip2.go index 42788443bc..f07c7e81e8 100644 --- a/src/compress/bzip2/bzip2.go +++ b/src/compress/bzip2/bzip2.go @@ -163,7 +163,7 @@ func (bz2 *reader) readFromBlock(buf []byte) int { func (bz2 *reader) read(buf []byte) (int, error) { for { n := bz2.readFromBlock(buf) - if n > 0 { + if n > 0 || len(buf) == 0 { bz2.blockCRC = updateCRC(bz2.blockCRC, buf[:n]) return n, nil } diff --git a/src/compress/bzip2/bzip2_test.go b/src/compress/bzip2/bzip2_test.go index a6c3080db3..3848603e0d 100644 --- a/src/compress/bzip2/bzip2_test.go +++ b/src/compress/bzip2/bzip2_test.go @@ -204,6 +204,14 @@ func TestMTF(t *testing.T) { } } +func TestZeroRead(t *testing.T) { + b := mustDecodeHex("425a6839314159265359b5aa5098000000600040000004200021008283177245385090b5aa5098") + r := NewReader(bytes.NewReader(b)) + if n, err := r.Read(nil); n != 0 || err != nil { + t.Errorf("Read(nil) = (%d, %v), want (0, nil)", n, err) + } +} + var ( digits = mustLoadFile("testdata/e.txt.bz2") twain = mustLoadFile("testdata/Mark.Twain-Tom.Sawyer.txt.bz2") |