fix(gosec): don't defer file Close()

pixelfactoryio · Nov 10, 2021 · 219ca50 · 219ca50
1 parent e66a1d1
commit 219ca50
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/internal/wrapper/git.go b/internal/wrapper/git.go
@@ -93,9 +93,10 @@ func isEmptyDirOrNotExist(name string) (found bool, err error) {
 		}
 		return false, err
 	}
-	defer func() {
-		err = f.Close()
-	}()
+
+	if err = f.Close(); err != nil {
+		return true, err
+	}
 
 	_, err = f.Readdirnames(1) // Or f.Readdir(1)
 	if err == io.EOF {