Skip to content

Commit 29fc55f

Browse files
authored
Merge pull request #6367 from tonistiigi/v0.26-picks
[v0.26] contentutil: avoid defaulting to ReadAt for fetch
2 parents a8e548f + 247e13c commit 29fc55f

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

util/contentutil/fetcher.go

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,17 +51,16 @@ type readerAt struct {
5151
}
5252

5353
func (r *readerAt) ReadAt(b []byte, off int64) (int, error) {
54-
if ra, ok := r.Reader.(io.ReaderAt); ok {
55-
return ra.ReadAt(b, off)
56-
}
57-
5854
if r.offset != off {
5955
if seeker, ok := r.Reader.(io.Seeker); ok {
6056
if _, err := seeker.Seek(off, io.SeekStart); err != nil {
6157
return 0, err
6258
}
6359
r.offset = off
6460
} else {
61+
if ra, ok := r.Reader.(io.ReaderAt); ok {
62+
return ra.ReadAt(b, off)
63+
}
6564
return 0, errors.Errorf("unsupported offset")
6665
}
6766
}

0 commit comments

Comments
 (0)