gosec warnings

semaphoreci · Oct 17, 2023 · def44be · def44be
1 parent fa538ee
commit def44be
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/cache-cli/pkg/archive/native_archiver.go b/cache-cli/pkg/archive/native_archiver.go
@@ -207,7 +207,8 @@ func (a *NativeArchiver) Decompress(src string) (string, error) {
 			}
 
 			// #nosec
-			if _, err := io.Copy(outFile, tarReader); err != nil {
+			_, err = io.Copy(outFile, tarReader)
+			if err != nil {
 				log.Errorf("Error writing to file '%s': %v", header.Name, err)
 				hadError = true
 				_ = outFile.Close()

diff --git a/cache-cli/pkg/files/checksum.go b/cache-cli/pkg/files/checksum.go
@@ -7,15 +7,16 @@ import (
 	"os"
 )
 
-// #nosec
 func GenerateChecksum(filePath string) (string, error) {
+	// #nosec
 	file, err := os.Open(filePath)
 	if err != nil {
 		return "", err
 	}
 
 	defer file.Close()
 
+	// #nosec
 	hash := md5.New()
 	if _, err := io.Copy(hash, file); err != nil {
 		return "", err