Skip to content

Commit b0a8b12

Browse files
author
Jay Conrod
committed
zip: fix staticcheck errors
Reported by dmitshur in a comment in CL 202042. Change-Id: I913bd87d9a17a9cf19ac869f8b7166b05e839758 Reviewed-on: https://go-review.googlesource.com/c/mod/+/218237 Run-TryBot: Jay Conrod <[email protected]> TryBot-Result: Gobot Gobot <[email protected]> Reviewed-by: Dmitri Shuralyov <[email protected]>
1 parent 76efd80 commit b0a8b12

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

zip/zip.go

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -247,6 +247,9 @@ func CreateFromDir(w io.Writer, m module.Version, dir string) (err error) {
247247

248248
var files []File
249249
err = filepath.Walk(dir, func(filePath string, info os.FileInfo, err error) error {
250+
if err != nil {
251+
return err
252+
}
250253
relPath, err := filepath.Rel(dir, filePath)
251254
if err != nil {
252255
return err

zip/zip_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -89,7 +89,7 @@ func (f fakeFile) Open() (io.ReadCloser, error) {
8989
if f.data != nil {
9090
return ioutil.NopCloser(bytes.NewReader(f.data)), nil
9191
}
92-
if f.size < 0 || f.size >= uint64(modzip.MaxZipFile<<1) {
92+
if f.size >= uint64(modzip.MaxZipFile<<1) {
9393
return nil, fmt.Errorf("cannot open fakeFile of size %d", f.size)
9494
}
9595
return ioutil.NopCloser(io.LimitReader(zeroReader{}, int64(f.size))), nil

0 commit comments

Comments
 (0)