1
0
Fork 1
mirror of https://github.com/vbatts/tar-split.git synced 2024-12-19 03:46:30 +00:00

Fix panic in Next

readHeader should never return nil with a tr.err also nil.
To correct this, ensure tr.err never gets reset to nil followed
by a nil return.
This commit is contained in:
Derek McGowan 2016-09-22 17:38:18 -07:00
parent 6810cedb21
commit e527e70d25

View file

@ -611,7 +611,8 @@ func (tr *Reader) readHeader() *Header {
if _, tr.err = io.ReadFull(tr.r, header); tr.err != nil {
// because it could read some of the block, but reach EOF first
if tr.err == io.EOF && tr.RawAccounting {
if _, tr.err = tr.rawBytes.Write(header); tr.err != nil {
if _, err := tr.rawBytes.Write(header); err != nil {
tr.err = err
return nil
}
}